Merge pull request #13 from olivergondza/JENKINS-13554

[FIXED JENKINS-13554] Delete configurations from MatrixBuild#delete

src/main/java/hudson/matrix/MatrixBuild.java

@@ -90,35 +90,30 @@ public Object readResolve() {
         return this;
     }
 
+    /**
+     * Delete build and its child builds.
+     *
+     * @since TODO
+     */
+    @Override
+    public void delete() throws IOException {
+        super.delete();
+
+        for(MatrixRun run : getExactRuns()) {
+            run.delete();
+        }
+    }
+
     /**
      * Deletes the build and all matrix configurations in this build when the button is pressed.
+     *
+     * @deprecated since TODO, kept not to break REST clients.
      */
     @RequirePOST
+    @Deprecated
     public void doDoDeleteAll( StaplerRequest req, StaplerResponse rsp ) throws IOException, ServletException {
-        checkPermission(DELETE);
-
-        // We should not simply delete the build if it has been explicitly
-        // marked to be preserved, or if the build should not be deleted
-        // due to dependencies!
-        String why = getWhyKeepLog();
-        if (why!=null) {
-            sendError(hudson.model.Messages.Run_UnableToDelete(getFullDisplayName(), why), req, rsp);
-            return;
-        }
-
-        List<MatrixRun> runs = getExactRuns();
-        for(MatrixRun run : runs){
-        	why = run.getWhyKeepLog();
-            if (why!=null) {
-                sendError(hudson.model.Messages.Run_UnableToDelete(getFullDisplayName(), why), req, rsp);
-                return;
-            }
-        	run.delete();
-        }
-        delete();
-        rsp.sendRedirect2(req.getContextPath()+'/' + getParent().getUrl());
+        doDoDelete(req, rsp);
     }
-
 
     /**
      * Used by view to render a ball for {@link MatrixRun}.