Bug 1034094 - Evaluation of the RHS in a rule with multiple matches using noloop and update gets cut short
Summary: Evaluation of the RHS in a rule with multiple matches using noloop and update...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss BRMS Platform 6
Classification: Retired
Component: BRE
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
urgent
high
Target Milestone: ER6
: 6.0.0
Assignee: Mario Fusco
QA Contact: Radovan Synek
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-25 09:14 UTC by Greg Kable
Modified: 2014-08-06 20:18 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-08-06 20:18:12 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Some code to demonstrate the issue. (30.00 KB, application/x-tar)
2013-11-25 09:14 UTC, Greg Kable
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker DROOLS-353 0 Major Resolved Evaluation of the RHS in a rule with multiple matches using noloop and update gets cut short 2015-06-03 12:19:09 UTC

Description Greg Kable 2013-11-25 09:14:01 UTC
Created attachment 828572 [details]
Some code to demonstrate the issue.

Description of problem:

If you have a rule with both noloop and update where there are multiple matches on the LHS, the RHS only runs for first match instead of running for all matches.  This is completely different to the behaviour under 5.3 and will break lots of existing code.


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


How reproducible:

Steps to Reproduce:
1. Build the attached code against BRMS6.0 and run - test will fail
2. Build the code against BRMS5.3 and the test will pass ...OR...
2. Or remove the uodate($order) call from the RHS under BRMS6.0 and the test will pass


Actual results:
Evaluation of the RHS stops after the update call


Expected results:
The RHS should be evaluated for all matches


Additional info:

Comment 4 Mario Fusco 2013-11-25 18:54:48 UTC
Fixed by https://github.com/droolsjbpm/drools/commit/a558125a2

Comment 5 Radovan Synek 2014-01-02 16:51:45 UTC
Verified on BRMS-6.0.0.ER7

Comment 6 Radovan Synek 2014-03-18 16:33:26 UTC
qe_test_coverage: covered by community unit tests


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