[jboss-svn-commits] JBL Code SVN: r31760 - in labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow: common/editor/core/command and 1 other directories.

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

Author: KrisVerlaenen
Date: 2010-02-21 12:09:11 -0500 (Sun, 21 Feb 2010)
New Revision: 31760

Modified:
   labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ElementContainer.java
   labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ElementContainerElementWrapper.java
   labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ProcessWrapper.java
   labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/command/AddElementCommand.java
   labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/ruleflow/core/RuleFlowProcessWrapper.java
Log:
JBIDE-5883: Several exceptions in Error log view with drools
 - fixed exception when adding multiple start nodes

Modified: labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ElementContainer.java
===================================================================
--- labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ElementContainer.java	2010-02-21 16:36:02 UTC (rev 31759)
+++ labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ElementContainer.java	2010-02-21 17:09:11 UTC (rev 31760)
@@ -13,5 +13,7 @@
     List<ElementWrapper> getElements();
     
     ProcessWrapper getProcessWrapper();
+    
+    boolean canAddElement(ElementWrapper element);
 
 }

Modified: labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ElementContainerElementWrapper.java
===================================================================
--- labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ElementContainerElementWrapper.java	2010-02-21 16:36:02 UTC (rev 31759)
+++ labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ElementContainerElementWrapper.java	2010-02-21 17:09:11 UTC (rev 31760)
@@ -40,5 +40,9 @@
     public ProcessWrapper getProcessWrapper() {
         return getParent().getProcessWrapper();
     }
+    
+    public boolean canAddElement(ElementWrapper element) {
+    	return true;
+    }
 
 }

Modified: labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ProcessWrapper.java
===================================================================
--- labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ProcessWrapper.java	2010-02-21 16:36:02 UTC (rev 31759)
+++ labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/ProcessWrapper.java	2010-02-21 17:09:11 UTC (rev 31760)
@@ -295,4 +295,8 @@
                 ExceptionScope.EXCEPTION_SCOPE)).setExceptionHandlers((Map<String, ExceptionHandler>) value);
         }
     }
+
+    public boolean canAddElement(ElementWrapper element) {
+    	return true;
+    }
 }

Modified: labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/command/AddElementCommand.java
===================================================================
--- labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/command/AddElementCommand.java	2010-02-21 16:36:02 UTC (rev 31759)
+++ labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/common/editor/core/command/AddElementCommand.java	2010-02-21 17:09:11 UTC (rev 31760)
@@ -28,6 +28,10 @@
 
     private ElementWrapper child;
     private ElementContainer parent;
+    
+    public boolean canExecute() {
+    	return parent.canAddElement(child);
+    }
 
     public void execute() {
         parent.addElement(child);

Modified: labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/ruleflow/core/RuleFlowProcessWrapper.java
===================================================================
--- labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/ruleflow/core/RuleFlowProcessWrapper.java	2010-02-21 16:36:02 UTC (rev 31759)
+++ labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/flow/ruleflow/core/RuleFlowProcessWrapper.java	2010-02-21 17:09:11 UTC (rev 31760)
@@ -54,4 +54,11 @@
         getRuleFlowProcess().removeNode(((NodeWrapper) element).getNode()); 
     }
     
+    public boolean canAddElement(ElementWrapper element) {
+    	if (element instanceof StartNodeWrapper && getRuleFlowProcess().getStart() != null) {
+    		return false;
+    	}
+    	return true;
+    }
+    
 }