[jboss-svn-commits] JBL Code SVN: r10843 - labs/jbossrules/trunk/drools-core/src/main/java/org/drools/base.

[jboss-svn-commits at lists.jboss.org]

Author: mark.proctor at jboss.com
Date: 2007-04-09 22:47:37 -0400 (Mon, 09 Apr 2007)
New Revision: 10843

Modified:
   labs/jbossrules/trunk/drools-core/src/main/java/org/drools/base/DefaultKnowledgeHelper.java
Log:
JBRULES-778 column fact handle used in consequences even for field bindings
-now uses eager rather than lazy facthandle retrieval

Modified: labs/jbossrules/trunk/drools-core/src/main/java/org/drools/base/DefaultKnowledgeHelper.java
===================================================================
--- labs/jbossrules/trunk/drools-core/src/main/java/org/drools/base/DefaultKnowledgeHelper.java	2007-04-10 02:47:32 UTC (rev 10842)
+++ labs/jbossrules/trunk/drools-core/src/main/java/org/drools/base/DefaultKnowledgeHelper.java	2007-04-10 02:47:37 UTC (rev 10843)
@@ -92,19 +92,11 @@
 
     public void modifyObject(FactHandle handle,
                              final Object newObject) throws FactException {
-    	if ( handle == null ) {
-    		// no fact handle exists for this object, so it must be a bound field
-    		// check if that bound field is still a fact in the WM
-    		handle = this.workingMemory.getFactHandle( newObject );
-    	}
-    	
-    	if (  handle != null ) {
-    		// only modify if this fact exists in the wm
-	        this.workingMemory.modifyObject( handle,
-	                                         newObject,
-	                                         this.rule,
-	                                         this.activation );
-    	}
+        // only modify if this fact exists in the wm
+        this.workingMemory.modifyObject( handle,
+                                         newObject,
+                                         this.rule,
+                                         this.activation );
     }
 
     public void retractObject(final FactHandle handle) throws FactException {