svn commit: r17204 - trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigClassifierBox.java

Author: bobtarling
Date: 2009-08-01 08:17:38-0700
New Revision: 17204

Modified:
   trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigClassifierBox.java

Log:
Questionable use of SelectionClass

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigClassifierBox.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigClassifierBox.java?view=diff&pathrev=17204&r1=17203&r2=17204
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigClassifierBox.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/static_structure/ui/FigClassifierBox.java	2009-08-01 08:17:38-0700
@@ -244,6 +244,8 @@
         Editor ce = Globals.curEditor();
         if (ce != null) {
             Selection sel = ce.getSelectionManager().findSelectionFor(this);
+            // TODO" What is the purpose of this? Why do we hide buttons here?
+            // Presumably if so we should not assume SelectionClass
             if (sel instanceof SelectionClass) {
                 ((SelectionClass) sel).hideButtons();
             }

------------------------------------------------------
http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=2377821

To unsubscribe from this discussion, e-mail: [[email protected]].