svn commit: r14532 - trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DiagramPropPanelFactory.java

Author: bobtarling
Date: 2008-04-30 10:39:51-0700
New Revision: 14532

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

Log:
Remove my previous hack - modules can now register their specific panels

Modified: trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DiagramPropPanelFactory.java
Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DiagramPropPanelFactory.java?view=diff&rev=14532&p1=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DiagramPropPanelFactory.java&p2=trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DiagramPropPanelFactory.java&r1=14531&r2=14532
==============================================================================
--- trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DiagramPropPanelFactory.java	(original)
+++ trunk/src/argouml-app/src/org/argouml/uml/diagram/ui/DiagramPropPanelFactory.java	2008-04-30 10:39:51-0700
@@ -40,8 +40,6 @@
  * TODO: This class should be split up, to have a different
  * factory for each package. 
  * That would solve the current wrong package dependencies.
- * However, the statement added by Bob to return PropPanelDiagram
- * needs a different solution then - or can it be removed?
  *
  * @author Michiel
  */
@@ -63,10 +61,6 @@
                 return new PropPanelUMLStateDiagram();
             } else if (object instanceof UMLUseCaseDiagram) {
                 return new PropPanelUMLUseCaseDiagram();
-            } else {
-                // If we get here then presumably a plugin has provided a
-                // different diagram type. For now lets show something.
-                return new PropPanelDiagram();
             }
         }
         return null;