add overwrite option. set overwrite as default option, and prompt if not overwrite and output already exists.

Author: johnsteele

plugins/org.jboss.tools.windup.model/src/org/jboss/tools/windup/model/domain/ModelService.java

@@ -348,9 +348,10 @@ protected void doExecute() {
 					configuration.setName(name);
 					configuration.setWindupHome(WindupRuntimePlugin.computeWindupHome().toString());
 					Pair pair = WindupFactory.eINSTANCE.createPair();
-					pair.setKey("output");
-//					pair.setValue();
+					pair.setKey("overwrite");
+					pair.setValue("true");
 //					configuration.getOptions().add(pair);
+					configuration.getOptions().add(pair);
 					configuration.setOutputLocation(getDefaultOutputLocation(configuration));
 					configuration.setSourceMode(true);
 					configuration.setGenerateReport(true);

plugins/org.jboss.tools.windup.runtime/src/org/jboss/tools/windup/runtime/WindupRuntimePlugin.java

@@ -171,6 +171,12 @@ public static Help findWindupHelpCache() {
 	private static String getOptions() {
 		return "<?xml version=\"1.0\" encoding=\"UTF-8\"?>\n"
 		+ "<help>\n"
+		+ " <option name=\"overwrite\">\n"
+		+ "		<description>If set, overwrite the output directory.</description>\n"
+		+ "		<type>Boolean</type>\n"
+		+ " 	<ui-type>SINGLE</ui-type>\n"
+		+ " 	<required>false</required>\n"
+		+ " </option>\n"
 		+ "	<option name=\"target\">\n"
 		+ "		<description>The target technology to consider for analysis.</description>\n"
 		+ "		<type>String</type>\n"

plugins/org.jboss.tools.windup.ui/src/org/jboss/tools/windup/ui/internal/launch/WindupLaunchDelegate.java

@@ -14,6 +14,7 @@
 import java.io.File;
 import java.io.FileInputStream;
 import java.util.List;
+import java.util.Optional;
 import java.util.Set;
 import java.util.function.Consumer;
 import java.util.stream.Collectors;
@@ -86,8 +87,11 @@ public void launch(ILaunchConfiguration config, String mode, ILaunch launch, IPr
 			this.runKantra(configuration);
 		}
 	}
+	
+	private boolean invalidConfig = false; 
 
 	private void runKantra(ConfigurationElement configuration) {
+		this.invalidConfig = false;
 		if (WindupLaunchDelegate.activeRunner != null) {
 			WindupLaunchDelegate.activeRunner.kill();
 		}
@@ -127,6 +131,7 @@ protected IStatus run(IProgressMonitor monitor) {
 	          	return Status.OK_STATUS;
 	      }
 		};
+		
 		kantraJob.addJobChangeListener(new JobChangeAdapter() {
 	    	@Override
 	    	public void done(IJobChangeEvent event) {
@@ -150,10 +155,12 @@ public void done(IJobChangeEvent event) {
 					if (WindupLaunchDelegate.activeRunner != null) {
 	    				WindupLaunchDelegate.activeRunner.kill();
 	    			}
-		    		org.jboss.tools.windup.model.domain.KantraRulesetParser.parseRulesetForKantraConfig(IssueExplorer.current.modelService.getKantraDelegate(configuration));
-		    		IssueExplorer.current.modelService.save();
-		    		IssueExplorer.current.markerService.generateMarkersForConfiguration(configuration);
-		    		IssueExplorer.current.viewService.renderReport(configuration);
+					if (!WindupLaunchDelegate.this.invalidConfig) {
+			    		org.jboss.tools.windup.model.domain.KantraRulesetParser.parseRulesetForKantraConfig(IssueExplorer.current.modelService.getKantraDelegate(configuration));
+			    		IssueExplorer.current.modelService.save();
+			    		IssueExplorer.current.markerService.generateMarkersForConfiguration(configuration);
+			    		IssueExplorer.current.viewService.renderReport(configuration);
+					}
 				}
 	    		kantraMonitor.done();
 	    	}
@@ -172,6 +179,37 @@ public void done(IJobChangeEvent event) {
 		String cli = WindupRuntimePlugin.computeWindupHome();
 
 		File outputFile = new File(output);
+		
+		Optional<Pair> overwriteOption = configuration.getOptions().stream().filter(option -> option.getKey().equals("overwrite")).findFirst();
+		if (overwriteOption.isPresent()) {
+			Pair pair = overwriteOption.get();
+			if (!Boolean.valueOf(pair.getValue())) {
+				if (outputFile.exists()) {
+					Display.getDefault().asyncExec(() -> {
+						MessageDialog.openInformation(Display.getDefault().getActiveShell(), 
+								Messages.launchErrorTitle, "Output location already exists. `--overwrite` option is required.");
+						WindupUIPlugin.logErrorMessage("Output location already exists. `--overwrite` option is required."); //$NON-NLS-1$
+					});
+					this.invalidConfig = true;
+					kantraJob.cancel();
+					return;
+				}
+			}
+			
+		}
+		else {
+			if (outputFile.exists()) {
+				Display.getDefault().asyncExec(() -> {
+					MessageDialog.openInformation(Display.getDefault().getActiveShell(), 
+							Messages.launchErrorTitle, "Output location already exists. `--overwrite` option is required.");
+					WindupUIPlugin.logErrorMessage("Output location already exists. `--overwrite` option is required."); //$NON-NLS-1$
+				});
+				this.invalidConfig = true;
+				kantraJob.cancel();
+				return;
+			}
+		}
+		
 		if (!outputFile.exists()) {
 			try {
 				outputFile.mkdirs();
@@ -183,6 +221,8 @@ public void done(IJobChangeEvent event) {
 				return;
 			}
 		}
+		
+		
 
     	for (Pair pair : configuration.getOptions()) {
 			String name = pair.getKey();