#504 mlDeleteForestReplicas now requires confirmation

Author: rjrudin

src/main/groovy/com/marklogic/gradle/MarkLogicPlugin.groovy

@@ -208,7 +208,7 @@ class MarkLogicPlugin implements Plugin<Project> {
 
 		String forestGroup = "ml-gradle Forest"
 		project.task("mlConfigureForestReplicas", type: ConfigureForestReplicasTask, group: forestGroup, description: "Deprecated - configure forest replicas via the command.forestNamesAndReplicaCounts map")
-		project.task("mlDeleteForestReplicas", type: DeleteForestReplicasTask, group: forestGroup, description: "Deprecated - delete forest replicas via the command.forestNamesAndReplicaCounts map")
+		project.task("mlDeleteForestReplicas", type: DeleteForestReplicasTask, group: forestGroup, description: "Deprecated - delete forest replicas via the command.forestNamesAndReplicaCounts map; requires -Pconfirm=true to be set so this isn't accidentally executed")
 		project.task("mlDeployCustomForests", type: DeployCustomForestsTask, group: forestGroup, description: "Deploy custom forests as defined in subdirectories of the forests configuration directory")
 		project.task("mlDeployForestReplicas", type: DeployForestReplicasTask, group: forestGroup, description: "Prefer this over mlConfigureForestReplicas; it does the same thing, but uses the ConfigureForestReplicasCommand that is used by mlDeploy")
 		project.task("mlPrintForestPlan", type: PrintForestPlanTask, group: forestGroup, description: "Print a list of primary forests to be created for a database specified by -Pdatabase=(name of database) when the database is next deployed")
@@ -313,7 +313,7 @@ class MarkLogicPlugin implements Plugin<Project> {
 		project.task("mlUnitTest", type: UnitTestTask, group: unitTestGroup, description: "Run tests found under /test/suites in the modules database. " +
 			"Connects to MarkLogic via the REST API server defined by mlTestRestPort (or by mlRestPort if mlTestRestPort is not set), and uses mlRest* properties for authentication. " +
 			"Use -PunitTestResultsPath to override where test result files are written, which defaults to build/test-results/marklogic-unit-test. " +
-			"Use -PrunCodeCoverage to enable code coverage support when running the tests. " + 
+			"Use -PrunCodeCoverage to enable code coverage support when running the tests. " +
 			"Use -PrunTeardown and -PrunSuiteTeardown to control whether teardown and suite teardown scripts are run; these default to 'true' and can be set to 'false' instead. ")
 
 		logger.info("Finished initializing ml-gradle\n")

src/main/groovy/com/marklogic/gradle/task/forests/DeleteForestReplicasTask.groovy

@@ -1,20 +1,18 @@
 package com.marklogic.gradle.task.forests
 
-import org.gradle.api.tasks.TaskAction
-
 import com.marklogic.appdeployer.command.forests.ConfigureForestReplicasCommand
-import com.marklogic.gradle.task.MarkLogicTask
+import com.marklogic.gradle.task.AbstractConfirmableTask
 
 /**
- * Task for invoking "undo" on an instance of ConfigureForestReplicasCommand. The command is exposed as a task 
- * attribute so that its map of forest names and replica counts and be configured easily in a Gradle build file. 
+ * Task for invoking "undo" on an instance of ConfigureForestReplicasCommand. The command is exposed as a task
+ * attribute so that its map of forest names and replica counts and be configured easily in a Gradle build file.
  */
-class DeleteForestReplicasTask extends MarkLogicTask {
+class DeleteForestReplicasTask extends AbstractConfirmableTask {
 
-    ConfigureForestReplicasCommand command = new ConfigureForestReplicasCommand()
+	ConfigureForestReplicasCommand command = new ConfigureForestReplicasCommand()
 
-    @TaskAction
-    void deleteForestReplicas() {
-        command.undo(getCommandContext())
-    }
+	@Override
+	void executeIfConfirmed() {
+		command.undo(getCommandContext())
+	}
 }