From 523eed9319e8ef2e08f5098c5dd9367486687a90 Mon Sep 17 00:00:00 2001
From: Jeremy Long <jeremy.long@gmail.com>
Date: Sat, 13 May 2017 08:38:43 -0400
Subject: [PATCH] resolved issue #686 - reports are generated even if no
 dependencies were analyzed

---
 .../dependencycheck/maven/CheckMojo.java      | 31 +++++++++----------
 1 file changed, 15 insertions(+), 16 deletions(-)

diff --git a/dependency-check-maven/src/main/java/org/owasp/dependencycheck/maven/CheckMojo.java b/dependency-check-maven/src/main/java/org/owasp/dependencycheck/maven/CheckMojo.java
index b566cc306..cb95628db 100644
--- a/dependency-check-maven/src/main/java/org/owasp/dependencycheck/maven/CheckMojo.java
+++ b/dependency-check-maven/src/main/java/org/owasp/dependencycheck/maven/CheckMojo.java
@@ -108,24 +108,23 @@ public class CheckMojo extends BaseDependencyCheckMojo {
                     }
                     exCol = ex;
                 }
-                if (exCol == null || !exCol.isFatal()) {
-                    try {
-                        writeReports(engine, getProject(), getCorrectOutputDirectory());
-                    } catch (ReportException ex) {
-                        if (this.isFailOnError()) {
-                            if (exCol != null) {
-                                exCol.addException(ex);
-                            } else {
-                                exCol = new ExceptionCollection("Unable to write the dependency-check report", ex);
-                            }
+            }
+            if (exCol == null || !exCol.isFatal()) {
+                try {
+                    writeReports(engine, getProject(), getCorrectOutputDirectory());
+                } catch (ReportException ex) {
+                    if (this.isFailOnError()) {
+                        if (exCol != null) {
+                            exCol.addException(ex);
+                        } else {
+                            exCol = new ExceptionCollection("Unable to write the dependency-check report", ex);
                         }
                     }
-                    //writeDataFile(getProject(), null, engine.getDependencies());
-                    showSummary(getProject(), engine.getDependencies());
-                    checkForFailure(engine.getDependencies());
-                    if (exCol != null && this.isFailOnError()) {
-                        throw new MojoExecutionException("One or more exceptions occurred during dependency-check analysis", exCol);
-                    }
+                }
+                showSummary(getProject(), engine.getDependencies());
+                checkForFailure(engine.getDependencies());
+                if (exCol != null && this.isFailOnError()) {
+                    throw new MojoExecutionException("One or more exceptions occurred during dependency-check analysis", exCol);
                 }
             }
             engine.cleanup();