Author: bobtarling Date: 2011-03-14 19:39:31-0700 New Revision: 19115 Modified: trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java
Log: Use the handy local variable instead of calling the same class method repeatedly. Modified: trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java?view=diff&pathrev=19115&r1=19114&r2=19115 ============================================================================== --- trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java (original) +++ trunk/src/argouml-core-umlpropertypanels/src/org/argouml/core/propertypanels/ui/SwingUIFactory.java 2011-03-14 19:39:31-0700 @@ -215,19 +215,19 @@ final String propertyName = prop.getPropertyName(); final Class<?> type = prop.getType(); - if ("initialValue".equals(prop.getPropertyName())) { + if ("initialValue".equals(propertyName)) { UMLExpressionModel model = new UMLInitialValueExpressionModel(target); - p = new UMLExpressionPanel(model, prop.getPropertyName()); + p = new UMLExpressionPanel(model, propertyName); control = p; - } else if ("defaultValue".equals(prop.getPropertyName())) { + } else if ("defaultValue".equals(propertyName)) { UMLExpressionModel model = new UMLDefaultValueExpressionModel(target); - p = new UMLExpressionPanel(model, prop.getPropertyName()); + p = new UMLExpressionPanel(model, propertyName); control = p; - } else if ("specification".equals(prop.getPropertyName())) { + } else if ("specification".equals(propertyName)) { UMLPlainTextDocument document = - new UMLOperationSpecificationDocument(prop.getPropertyName(), target); + new UMLOperationSpecificationDocument(propertyName, target); UMLTextArea osta = new UMLTextArea(document); osta.setRows(3); control = new JScrollPane(osta); @@ -237,34 +237,34 @@ text.setLineWrap(true); text.setRows(5); control = new JScrollPane(text); - } else if ("condition".equals(prop.getPropertyName())) { + } else if ("condition".equals(propertyName)) { UMLExpressionModel conditionModel = new UMLConditionExpressionModel(target); JTextArea conditionArea = new UMLExpressionBodyField(conditionModel, true); conditionArea.setRows(5); control = new JScrollPane(conditionArea); - } else if ("script".equals(prop.getPropertyName())) { + } else if ("script".equals(propertyName)) { UMLExpressionModel scriptModel = new UMLScriptExpressionModel(target); p = new UMLExpressionPanel(scriptModel, prop.getPropertyName()); control = p; - } else if ("recurrence".equals(prop.getPropertyName())) { + } else if ("recurrence".equals(propertyName)) { UMLExpressionModel recurrenceModel = new UMLRecurrenceExpressionModel(target); - p = new UMLExpressionPanel(recurrenceModel, prop.getPropertyName()); + p = new UMLExpressionPanel(recurrenceModel, propertyName); control = p; - } else if ("expression".equals(prop.getPropertyName())) { + } else if ("expression".equals(propertyName)) { UMLExpressionModel model = new UMLExpressionExpressionModel(target); - p = new UMLExpressionPanel(model, prop.getPropertyName()); + p = new UMLExpressionPanel(model, propertyName); control = p; - } else if ("changeExpression".equals(prop.getPropertyName())) { + } else if ("changeExpression".equals(propertyName)) { UMLExpressionModel model = new UMLChangeExpressionModel(target); - p = new UMLExpressionPanel(model, prop.getPropertyName()); + p = new UMLExpressionPanel(model, propertyName); control = p; - } else if ("when".equals(prop.getPropertyName())) { + } else if ("when".equals(propertyName)) { UMLExpressionModel model = new UMLTimeExpressionModel(target); - p = new UMLExpressionPanel(model, prop.getPropertyName()); + p = new UMLExpressionPanel(model, propertyName); control = p; } ------------------------------------------------------ http://argouml.tigris.org/ds/viewMessage.do?dsForumId=5905&dsMessageId=2711524 To unsubscribe from this discussion, e-mail: [[email protected]].
