Bug 1052313 - Restrictions on Number that is result of 'accumulate' CE are not shown in GRE when the rule is reopened
Summary: Restrictions on Number that is result of 'accumulate' CE are not shown in GRE...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss BRMS Platform 6
Classification: Retired
Component: Business Central
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ER1
: 6.0.1
Assignee: Toni Rikkola
QA Contact: Jiri Locker
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-01-13 15:45 UTC by Jiri Locker
Modified: 2014-08-06 19:57 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2014-08-06 19:57:12 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Jiri Locker 2014-01-13 15:45:43 UTC
Description of problem:
Save and close a Guided rule with this fragment:
> $total : Number( intValue > 0 ) from accumulate ( Applicant( age < 30 ), count() )

When it is reopened it looks like
> Number( ) from accumulate ( Applicant( age < 30 ), count() )

Variable binding and field restrictions are gone.

Version-Release number of selected component (if applicable):
ER7

How reproducible:
-

Steps to Reproduce:
See description.

Actual results:
GRE doesn't recognize parts of the DRL source.

Expected results:
After reopening the rule should look exactly the same as it was saved.

Additional info:

Comment 1 Toni Rikkola 2014-01-27 10:46:27 UTC
Test in master: https://github.com/droolsjbpm/drools/commit/619b14877

Comment 2 Mario Fusco 2014-01-27 17:01:34 UTC
Fixed by https://github.com/droolsjbpm/drools/commit/5b650c7ac

Comment 3 Mario Fusco 2014-01-27 17:09:14 UTC
I fixed the drl unmarshalling. Reassigning to Toni to check if also the editor is now ok.

Comment 4 Toni Rikkola 2014-01-29 09:39:56 UTC
Looks like there was actually two bugs here. First one, no constraints shown for Number, is fixed.

Second is that the variable for Number is not picked up. Here is the test for it.
https://github.com/droolsjbpm/drools/commit/5a38cb402

Comment 5 Mario Fusco 2014-01-29 11:09:03 UTC
Fixed by https://github.com/droolsjbpm/drools/commit/21dd2b838

Reassigning to Toni for a further check.

Comment 6 Toni Rikkola 2014-01-29 11:41:02 UTC
Commits for 6.0.x:
https://github.com/droolsjbpm/drools/compare/7376025...227ac58


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