Author: jm
Date: 2012-05-02 12:24:47 -0700 (Wed, 02 May 2012)
New Revision: 29080

Modified:
   
core3/impl/trunk/swing-application-impl/src/main/java/org/cytoscape/internal/layout/ui/LayoutSettingsDialog.java
Log:
Fixes #936: Removed isReady() check since it wasn't really appropriate.  
Doesn't make sense to perform check before tunables are set.

Modified: 
core3/impl/trunk/swing-application-impl/src/main/java/org/cytoscape/internal/layout/ui/LayoutSettingsDialog.java
===================================================================
--- 
core3/impl/trunk/swing-application-impl/src/main/java/org/cytoscape/internal/layout/ui/LayoutSettingsDialog.java
    2012-05-02 19:24:19 UTC (rev 29079)
+++ 
core3/impl/trunk/swing-application-impl/src/main/java/org/cytoscape/internal/layout/ui/LayoutSettingsDialog.java
    2012-05-02 19:24:47 UTC (rev 29080)
@@ -397,10 +397,6 @@
                                Object context = 
currentLayout.getDefaultLayoutContext();
 
                                TaskFactory provisioner = 
factoryProvisioner.createFor(wrapWithContext(currentLayout, context));
-                               if (!provisioner.isReady()) {
-                                       throw new 
IllegalArgumentException("Layout is not fully configured");
-                               }
-                               
                                JPanel tunablePanel = 
taskManager.getConfiguration(provisioner, context);
 
                                int row = 0;

-- 
You received this message because you are subscribed to the Google Groups 
"cytoscape-cvs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/cytoscape-cvs?hl=en.

Reply via email to