Fixes issue philmander#14

philmander#14 was reported by me. I specified the code change in a comment and then decided to try to fix it.
ojintoad · Sep 6, 2013 · be4df69 · be4df69
1 parent ede60c0
commit be4df69
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/src/main/java/com/philmander/jshint/report/PlainJsHintReporter.java b/src/main/java/com/philmander/jshint/report/PlainJsHintReporter.java
@@ -30,12 +30,13 @@ public String createReport() {
 		if (report.getTotalErrors() > 0) {
 			output.append("----------------------------------------------------------------------\n");
 			for(JsHintResult result : report.getResults()) { 
-				output.append("\n");
-				output.append(getFileFailureMessage((result.getFile()) + "\n"));
-				for(JsHintError error : result.getErrors()) {
-					output.append(getIssueMessage(error.getReason(), error.getEvidence(), error.getLine(), error.getCharacter()) + "\n");
+				if(result.getNumErrors() > 0) {
+					output.append("\n");
+					output.append(getFileFailureMessage((result.getFile()) + "\n"));
+					for(JsHintError error : result.getErrors()) {
+						output.append(getIssueMessage(error.getReason(), error.getEvidence(), error.getLine(), error.getCharacter()) + "\n");
+					}
 				}
-
 			}
 			output.append("----------------------------------------------------------------------\n");
 		}