svn commit: r1629060 - in /sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np: AbstractNewMavenBasedSlingApplicationWizard.java ArchetypeParametersWizardPage.java

[rombert]

Author: rombert
Date: Thu Oct  2 19:59:50 2014
New Revision: 1629060

URL: http://svn.apache.org/r1629060
Log:
SLING-3999 - Bundle creation wizard should warn early if a project with
the same name exists

Modified:
    
sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/AbstractNewMavenBasedSlingApplicationWizard.java
    
sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/ArchetypeParametersWizardPage.java

Modified: 
sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/AbstractNewMavenBasedSlingApplicationWizard.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/AbstractNewMavenBasedSlingApplicationWizard.java?rev=1629060&r1=1629059&r2=1629060&view=diff
==============================================================================
--- 
sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/AbstractNewMavenBasedSlingApplicationWizard.java
 (original)
+++ 
sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/AbstractNewMavenBasedSlingApplicationWizard.java
 Thu Oct  2 19:59:50 2014
@@ -84,7 +84,7 @@ public abstract class AbstractNewMavenBa
 
         IProject existingProject = 
ResourcesPlugin.getWorkspace().getRoot().getProject(artifactId);
         if (existingProject!=null && existingProject.exists()) {
-            throw new IllegalStateException("Project already exists with name 
"+artifactId);
+            throw new IllegalStateException("A project with the name " + 
artifactId + " already exists.");
         }
 
         advance(monitor, 1);

Modified: 
sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/ArchetypeParametersWizardPage.java
URL: 
http://svn.apache.org/viewvc/sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/ArchetypeParametersWizardPage.java?rev=1629060&r1=1629059&r2=1629060&view=diff
==============================================================================
--- 
sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/ArchetypeParametersWizardPage.java
 (original)
+++ 
sling/trunk/tooling/ide/eclipse-m2e-ui/src/org/apache/sling/ide/eclipse/ui/wizards/np/ArchetypeParametersWizardPage.java
 Thu Oct  2 19:59:50 2014
@@ -24,6 +24,8 @@ import java.util.StringTokenizer;
 import org.apache.maven.archetype.catalog.Archetype;
 import org.apache.maven.archetype.metadata.RequiredProperty;
 import org.apache.maven.artifact.repository.ArtifactRepository;
+import org.eclipse.core.resources.IProject;
+import org.eclipse.core.resources.ResourcesPlugin;
 import org.eclipse.jface.dialogs.IDialogPage;
 import org.eclipse.jface.viewers.CellEditor;
 import org.eclipse.jface.viewers.CellNavigationStrategy;
@@ -313,6 +315,14 @@ public class ArchetypeParametersWizardPa
                        updateStatus("package must be specified");
                        return;
                }
+
+        IProject existingProject = 
ResourcesPlugin.getWorkspace().getRoot().getProject(artifactId.getText());
+
+        if (existingProject.exists()) {
+            updateStatus("A project with the name " + artifactId.getText() + " 
already exists.");
+            return;
+        }
+
                int cnt = propertiesTable.getItemCount();
                for(int i=0; i<cnt; i++) {
                        TableItem item = propertiesTable.getItem(i);