Fix issue when violations are ignored

diff --git a/sonar-batch/src/main/java/org/sonar/batch/index/DefaultIndex.java b/sonar-batch/src/main/java/org/sonar/batch/index/DefaultIndex.java
index ea0746d6b47f34ff3698e5c344bad1313d8638bd..8f37854061f426a1b3becd5a5daf991031329743 100644 (file)
--- a/sonar-batch/src/main/java/org/sonar/batch/index/DefaultIndex.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/index/DefaultIndex.java
@@ -367,14 +367,15 @@ public class DefaultIndex extends SonarIndex {
     }
 
     violation.setResource(bucket.getResource());
-    addViolation(violation, bucket, force);
-    deprecatedViolations.add(violation);
+    if (addViolation(violation, bucket, force)){
+      deprecatedViolations.add(violation);
+    }
   }
 
-  private void addViolation(Violation violation, Bucket bucket, boolean force) {
+  private boolean addViolation(Violation violation, Bucket bucket, boolean force) {
     boolean isIgnored = !force && violationFilters != null && violationFilters.isIgnored(violation);
     if (isIgnored) {
-      return;
+      return false;
     }
 
     // TODO this code is not the responsibility of this index. It should be moved somewhere else.
@@ -386,13 +387,15 @@ public class DefaultIndex extends SonarIndex {
         violation.setSeverity(violation.getRule().getSeverity());
       } else {
         LoggerFactory.getLogger(getClass()).debug("Rule is not activated, ignoring violation {}", violation);
-        return;
+        return false;
       }
     }
 
     bucket.addViolation(violation);
+    return true;
   }
 
+
   //
   //
   //