Bug 996976 - Guided Decision Table: empty cells are ignored in the source
Summary: Guided Decision Table: empty cells are ignored in the source
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: JBoss BRMS Platform 6
Classification: Retired
Component: Business Central
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: ---
Assignee: manstis
QA Contact: Zuzana Krejčová
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-08-14 11:57 UTC by Zuzana Krejčová
Modified: 2016-08-01 01:07 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-10-24 12:26:47 UTC
Type: Bug
Embargoed:
manstis: needinfo+


Attachments (Terms of Use)
ignored cell, invalid rule (18.36 KB, image/png)
2013-08-14 11:57 UTC, Zuzana Krejčová
no flags Details

Description Zuzana Krejčová 2013-08-14 11:57:08 UTC
Created attachment 786512 [details]
ignored cell, invalid rule

Description of problem:
If you don't fill a value into a cell, that fact pattern is ignored in the source of that row. This doesn't really make sense and creates problems once you try to update that fact in an action column.

As you can see in the screenshot, I have a Fact [f], field [==] condition column and a f, field action column updating the field on the fact from the condition column. I have one row with only the action column cell filled in.

The source code then shows this:
1. | //from row number: 1
2. | rule "Row 1 gdt"
3. |     dialect "mvel"
4. |     when
5. |     then
6. |         f.setField( "aaa" );
7. | end

As you can see, this creates an invalid rule.

This happens for any empty cell, not just those in condition columns. 

The only understandable occurrence of this is for the attribute columns, where e.g. salience without a value would again create an invalid rule.


Version-Release number of selected component (if applicable):
kie-wb 6.0 CR1

Comment 1 Edson Tirelli 2013-10-01 00:27:44 UTC
Not sure if I am understanding this correctly, but decision tables are supposed to not create conditions for empty cells. I am assigning to Michael for him to double check that is the case. If so, it is not a bug.

Comment 2 manstis 2013-10-16 11:14:33 UTC
The only scenario where this is potentially valid is where a field's data-type is String and the value is a space or spaces. Can you confirm this? Obviously the other scenario where there's an action to update an existing fact where the pattern is excluded remains a valid - and flawed - scenario.

Comment 3 manstis 2013-10-16 12:25:07 UTC
*** Bug 996964 has been marked as a duplicate of this bug. ***

Comment 4 Zuzana Krejčová 2013-10-24 12:26:47 UTC
You are right.

Based on comment 1 and comment 2, I'll be closing this as not a bug and reopening the other BZ (bug 996964), now marked as duplicate of this, since that is an issue.


Note You need to log in before you can comment on or make changes to this bug.