Author: dgolovin Date: 2010-06-24 14:36:27 -0400 (Thu, 24 Jun 2010) New Revision: 23005 Modified: trunk/seam/plugins/org.jboss.tools.seam.xml/src/org/jboss/tools/seam/xml/components/model/handlers/RenameComponentHandler.java Log: code cleanup instead of using if(!condition) return; code; better style is if(condition) { code; } Modified: trunk/seam/plugins/org.jboss.tools.seam.xml/src/org/jboss/tools/seam/xml/components/model/handlers/RenameComponentHandler.java =================================================================== --- trunk/seam/plugins/org.jboss.tools.seam.xml/src/org/jboss/tools/seam/xml/components/model/handlers/RenameComponentHandler.java 2010-06-24 18:24:15 UTC (rev 23004) +++ trunk/seam/plugins/org.jboss.tools.seam.xml/src/org/jboss/tools/seam/xml/components/model/handlers/RenameComponentHandler.java 2010-06-24 18:36:27 UTC (rev 23005) @@ -18,11 +18,11 @@ } public void executeHandler(XModelObject object, Properties p) throws XModelException { - if(!isEnabled(object)) return; - - SpecialWizard wizard = SpecialWizardFactory.createSpecialWizard("org.jboss.tools.seam.ui.views.actions.RenameComponentAction"); - wizard.setObject(object); - wizard.execute(); + if(isEnabled(object)) { + SpecialWizard wizard = SpecialWizardFactory.createSpecialWizard("org.jboss.tools.seam.ui.views.actions.RenameComponentAction"); + wizard.setObject(object); + wizard.execute(); + } } }