[jboss-svn-commits] JBL Code SVN: r35300 - labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/modeldriven/ui.

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

Author: nheron
Date: 2010-09-26 10:14:41 -0400 (Sun, 26 Sep 2010)
New Revision: 35300

Modified:
   labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/modeldriven/ui/ConstraintValueEditor.java
Log:
[GUVNOR-647]
Guided rule Editor does not allow user to proceed when variable binding is shared by two LHS Constraints
It now works by setting constraints on variables


Modified: labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/modeldriven/ui/ConstraintValueEditor.java
===================================================================
--- labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/modeldriven/ui/ConstraintValueEditor.java	2010-09-26 11:26:22 UTC (rev 35299)
+++ labs/jbossrules/trunk/drools-guvnor/src/main/java/org/drools/guvnor/client/modeldriven/ui/ConstraintValueEditor.java	2010-09-26 14:14:41 UTC (rev 35300)
@@ -381,9 +381,8 @@
                 FactPattern f = model.getBoundFact(var);
                 String fieldConstraint = model.getBindingType(var);
 
-                if ((f != null && f.factType != null && this.fieldType != null)
-                        && (f.factType.equals(this.fieldType)
-                        || this.fieldType.equals(fieldConstraint))) {
+                if ((f != null && f.factType != null && f.factType.equals(this.fieldType))
+                        || (this.fieldType != null && this.fieldType.equals(fieldConstraint))) {
                     foundABouncVariableThatMatches = true;
                     break;
                 } else {