387521 : proper way to prevent SWTException

instead of passing a null progress monitor, which prevents the user from
cancelling archetype downloads, forking the process
(getContainer().run(true, true, new IRunnableWithProgress() {}) does not
produce the dreaded 'SWTException: Invalid thread access' error

Signed-off-by: Fred Bricon <fbricon@gmail.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/wizards/MavenProjectWizardArchetypeParametersPage.java b/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/wizards/MavenProjectWizardArchetypeParametersPage.java
index f3eaeba5..1c612d77 100644
--- a/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/wizards/MavenProjectWizardArchetypeParametersPage.java
+++ b/org.eclipse.m2e.core.ui/src/org/eclipse/m2e/core/ui/internal/wizards/MavenProjectWizardArchetypeParametersPage.java
@@ -383,7 +383,7 @@ public class MavenProjectWizardArchetypeParametersPage extends AbstractMavenWiza
     final String archetypeName = groupId + ":" + artifactId + ":" + version; //$NON-NLS-1$ //$NON-NLS-2$
     
     try {
-      getContainer().run(false, true, new IRunnableWithProgress() {
+      getContainer().run(true, true, new IRunnableWithProgress() {
         public void run(IProgressMonitor monitor) {
           monitor.beginTask(NLS.bind(org.eclipse.m2e.core.ui.internal.Messages.MavenProjectWizardArchetypeParametersPage_task, archetypeName), IProgressMonitor.UNKNOWN);
           
@@ -393,7 +393,7 @@ public class MavenProjectWizardArchetypeParametersPage extends AbstractMavenWiza
             
             ArtifactRepository remoteArchetypeRepository = archetypeManager.getArchetypeRepository(archetype);
             
-            List<?> properties = archetypeManager.getRequiredProperties(archetype, remoteArchetypeRepository, null);
+            List<?> properties = archetypeManager.getRequiredProperties(archetype, remoteArchetypeRepository, monitor);
             
             if(properties != null) {
               for(Object o : properties) {