[jbosstools-commits] JBoss Tools SVN: r23005 - trunk/seam/plugins/org.jboss.tools.seam.xml/src/org/jboss/tools/seam/xml/components/model/handlers.

jbosstools-commits at lists.jboss.org jbosstools-commits at lists.jboss.org
Thu Jun 24 14:36:28 EDT 2010


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();
+		}
 	}
 
 }



More information about the jbosstools-commits mailing list