Author: rob.stryker(a)jboss.com Date: 2011-08-18 03:57:07 -0400 (Thu, 18 Aug 2011) New Revision: 34034 Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.wtp.core/src/org/jboss/ide/eclipse/as/wtp/core/modules/JBTFlatProjectModuleFactory.java Log: https://issues.jboss.org/browse/JBIDE-9016 - Handling of code review Modified: trunk/as/plugins/org.jboss.ide.eclipse.as.wtp.core/src/org/jboss/ide/eclipse/as/wtp/core/modules/JBTFlatProjectModuleFactory.java =================================================================== --- trunk/as/plugins/org.jboss.ide.eclipse.as.wtp.core/src/org/jboss/ide/eclipse/as/wtp/core/modules/JBTFlatProjectModuleFactory.java 2011-08-18 04:08:06 UTC (rev 34033) +++ trunk/as/plugins/org.jboss.ide.eclipse.as.wtp.core/src/org/jboss/ide/eclipse/as/wtp/core/modules/JBTFlatProjectModuleFactory.java 2011-08-18 07:57:07 UTC (rev 34034) @@ -97,25 +97,15 @@ } protected IModule[] createModuleDelegates(IVirtualComponent component) { - if(component == null) - return null; - - List<IModule> projectModules = new ArrayList<IModule>(); - try { - if (canHandleProject(component.getProject())) { - String type = getModuleType(component.getProject()); - String version = getModuleVersion(component.getProject()); - IModule module = createModule(component.getName(), component.getName(), type, version, component.getProject()); - FlatComponentDeployable moduleDelegate = createModuleDelegate(component.getProject(), component); - moduleDelegates.put(module, moduleDelegate); - projectModules.add(module); - } else { - return new IModule[]{}; - } - } catch (Exception e) { - J2EEPlugin.logError(e); + if(component != null && canHandleProject(component.getProject())) { + String type = getModuleType(component.getProject()); + String version = getModuleVersion(component.getProject()); + IModule module = createModule(component.getName(), component.getName(), type, version, component.getProject()); + FlatComponentDeployable moduleDelegate = createModuleDelegate(component.getProject(), component); + moduleDelegates.put(module, moduleDelegate); + return new IModule[]{module}; } - return projectModules.toArray(new IModule[projectModules.size()]); + return null; }