[ https://issues.jboss.org/browse/JBRULES-3051?page=com.atlassian.jira.plug... ] Mark Proctor commented on JBRULES-3051: --------------------------------------- Previously ALL collections where cloned when bound: {code} public Object getNonShadowedValue(InternalWorkingMemory workingMemory, final Object object) { Object result = this.readAccessor.getValue( workingMemory, object ); if ( this.isInternalFact() && result instanceof Collection ) { try { Collection newCol = (Collection) result.getClass().newInstance(); for ( Iterator it = ((Collection) result).iterator(); it.hasNext(); ) { Object element = it.next(); newCol.add( element ); } return newCol; } catch ( InstantiationException e ) { // nothing we can do, so just return the resulting object } catch ( IllegalAccessException e ) { // TODO Auto-generated catch block } } return result; } {code} This was left over from shadow facts and actually still existed today. I have removed the code but now we have a failing test in FirstOrderLogicTest.testCollectResultConstraints. {code} rule "Collect Test" salience 70 when $cheeseList : ArrayList(size == 1) from collect( Cheese( ) ); then results.add($cheeseList); end {code} The reason why this passed before was the collection was cloned, so the accumulate's evaulations would not impact it. Now it is no longer cloned and while the rule does not fire the Collection is updated. I think the later is the correct behaviour and I've updated the test as expected. -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/JBRULES-3051?page=com.atlassian.jira.plug... ] Mark Proctor resolved JBRULES-3051. ----------------------------------- Fix Version/s: 5.2.0.CR1 Resolution: Done -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira

[ https://issues.jboss.org/browse/JBRULES-3051?page=com.atlassian.jira.plug... ] Mark Proctor resolved JBRULES-3051. ----------------------------------- Resolution: Done -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira