[jbosstools-commits] JBoss Tools SVN: r42439 - trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators.

[jbosstools-commits at lists.jboss.org]

Author: fbricon
Date: 2012-07-06 08:57:02 -0400 (Fri, 06 Jul 2012)
New Revision: 42439

Modified:
   trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators/HibernateProjectConfigurator.java
Log:
Remove unnecessary loop

Modified: trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators/HibernateProjectConfigurator.java
===================================================================
--- trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators/HibernateProjectConfigurator.java	2012-07-06 12:10:17 UTC (rev 42438)
+++ trunk/maven/plugins/org.jboss.tools.maven.hibernate/src/org/jboss/tools/maven/hibernate/configurators/HibernateProjectConfigurator.java	2012-07-06 12:57:02 UTC (rev 42439)
@@ -86,23 +86,18 @@
 	}
 
 	private boolean isHibernateProject(MavenProject mavenProject) {
-		List<Artifact> artifacts = new ArrayList<Artifact>();
-		ArtifactFilter filter = new ScopeArtifactFilter(
-				Artifact.SCOPE_TEST);
+		ArtifactFilter filter = new ScopeArtifactFilter(Artifact.SCOPE_TEST);
 		for (Artifact artifact : mavenProject.getArtifacts()) {
 			if (filter.include(artifact)) {
-				artifacts.add(artifact);
+				String groupId = artifact.getGroupId();
+				if (HIBERNATE_GROUP_ID.equals(groupId)) {
+					String artifactId = artifact.getArtifactId();
+					if (artifactId != null && artifactId.startsWith(HIBERNATE_ARTIFACT_ID_PREFIX)) {
+						return true;
+					} 
+				}
 			}
 		}
-        for (Artifact artifact:artifacts) {
-	    	String groupId = artifact.getGroupId();
-    		if (HIBERNATE_GROUP_ID.equals(groupId)) {
-    			String artifactId = artifact.getArtifactId();
-    			if (artifactId != null && artifactId.startsWith(HIBERNATE_ARTIFACT_ID_PREFIX)) {
-	    			return true;
-	    		} 
-	    	}
-	    }
 	    return false;
 	}
 }