Author: paperwing
Date: 2012-02-14 07:44:55 -0800 (Tue, 14 Feb 2012)
New Revision: 28258

Modified:
   
csplugins/trunk/toronto/yuedong/multi_renderer/impl/presentation-impl/src/main/java/org/cytoscape/view/presentation/internal/ExternalRendererManagerImpl.java
Log:
Updated ExternalRendererManagerImpl to set its single renderer as the default 
if there is only 1

Modified: 
csplugins/trunk/toronto/yuedong/multi_renderer/impl/presentation-impl/src/main/java/org/cytoscape/view/presentation/internal/ExternalRendererManagerImpl.java
===================================================================
--- 
csplugins/trunk/toronto/yuedong/multi_renderer/impl/presentation-impl/src/main/java/org/cytoscape/view/presentation/internal/ExternalRendererManagerImpl.java
       2012-02-14 00:30:30 UTC (rev 28257)
+++ 
csplugins/trunk/toronto/yuedong/multi_renderer/impl/presentation-impl/src/main/java/org/cytoscape/view/presentation/internal/ExternalRendererManagerImpl.java
       2012-02-14 15:44:55 UTC (rev 28258)
@@ -34,9 +34,8 @@
                
                installedRenderers.put(externalRenderer.getRendererID(), 
externalRenderer);
                
-               if (installedRenderers.size() == 1) {
-                       defaultRendererID = new 
String(externalRenderer.getRendererID());
-               }
+               // If we only have 1 renderer, set it as the default
+               manageDefaultRenderer();
        }
 
        @Override
@@ -51,7 +50,8 @@
        
                installedRenderers.remove(externalRenderer.getRendererID());
                
-               // TODO: Change default renderer ID if only 1 renderer left
+               // If we only have 1 renderer, set it as the default
+               manageDefaultRenderer();
        }
 
        @Override
@@ -65,7 +65,7 @@
                // Let the renderer know
                removed.dispose();
        
-               installedRenderers.remove(rendererID);
+               manageDefaultRenderer();
        }
 
        @Override
@@ -103,4 +103,15 @@
                
                // TODO: Check if necessary to return defensively copied 
Strings this way
        }
+       
+       /**
+        * Checks if there is only 1 renderer currently installed, and if so, 
sets it to be the default renderer
+        */
+       private void manageDefaultRenderer() {
+               if (installedRenderers.size() == 1) {
+                       for (String id : installedRenderers.keySet()) {
+                               defaultRendererID = new String(id);
+                       }
+               }
+       }
 }

-- 
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