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; --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
