[jbpm-commits] JBoss JBPM SVN: r3012 - projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/task.

[do-not-reply at jboss.org]

Author: heiko.braun at jboss.com
Date: 2008-11-20 09:06:49 -0500 (Thu, 20 Nov 2008)
New Revision: 3012

Modified:
   projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/task/AssignmentFormPanel.java
Log:
Tasks with neither pooled actors nor specific actor will be treated as an exception

Modified: projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/task/AssignmentFormPanel.java
===================================================================
--- projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/task/AssignmentFormPanel.java	2008-11-20 13:13:45 UTC (rev 3011)
+++ projects/gwt-console/trunk/war/src/main/java/org/jboss/bpm/console/client/task/AssignmentFormPanel.java	2008-11-20 14:06:49 UTC (rev 3012)
@@ -121,16 +121,23 @@
       selectedInstance= (TaskReference)changeEvent;
       availableActors = null;
 
-      if(selectedInstance.getPooledActors().isEmpty())
+      boolean hasPooledActors = selectedInstance.getPooledActors().size()>0;
+      if(hasPooledActors)
       {
-         // no pooled actors -> can be assigned to any user
-         // of current users groups
+         // has pooled actors -> can be asigned to pooled actors only
+         availableActors = selectedInstance.getPooledActors();
+      }
+      else if(!hasPooledActors &&
+            TaskReference.STATE.ASSIGNED ==selectedInstance.getCurrentState())
+      {
+         // no pooled actors but an assigned actor
          loadActors();
       }
       else
       {
-         // has pooled actors -> can be asigned to pooled actors only
-         availableActors = selectedInstance.getPooledActors();
+         // no pooled actors and no assignee
+         // should be covered by assignment handler
+         throw new IllegalStateException(selectedInstance + " has no pooled actors and no assignee");
       }
 
       // finally refresh the combobox