diff options
author | Fabrice Bellingard <bellingard@gmail.com> | 2011-04-21 16:56:21 +0200 |
---|---|---|
committer | Fabrice Bellingard <bellingard@gmail.com> | 2011-04-21 18:40:48 +0200 |
commit | e8bf534ec680e583b0e7683c92f1be31ab66a74e (patch) | |
tree | 69a2281fa2f3b036e945c1b27d41c9afd2043a75 /sonar-plugin-api | |
parent | 5c66787f6e31f2931f28b78d3eb056503cdfe57a (diff) | |
download | sonarqube-e8bf534ec680e583b0e7683c92f1be31ab66a74e.tar.gz sonarqube-e8bf534ec680e583b0e7683c92f1be31ab66a74e.zip |
SONAR-2380 The "violations" web service API must not return
"false-positive" violation
Also modified the batch side ViolationQuery class that I created so
that they have the same API.
Diffstat (limited to 'sonar-plugin-api')
-rw-r--r-- | sonar-plugin-api/src/main/java/org/sonar/api/batch/SonarIndex.java | 2 | ||||
-rw-r--r-- | sonar-plugin-api/src/main/java/org/sonar/api/violations/ViolationQuery.java | 14 |
2 files changed, 8 insertions, 8 deletions
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/SonarIndex.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/SonarIndex.java index 89133b1c6b8..4c82198ffdd 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/SonarIndex.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/SonarIndex.java @@ -138,7 +138,7 @@ public abstract class SonarIndex implements DirectedGraphAccessor<Resource, Depe * @return the list of violations */ public final List<Violation> getViolations(Resource resource) { - return getViolations(ViolationQuery.create().forResource(resource).ignoreSwitchedOff(true)); + return getViolations(ViolationQuery.create().forResource(resource)); } /** diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/violations/ViolationQuery.java b/sonar-plugin-api/src/main/java/org/sonar/api/violations/ViolationQuery.java index 10be756b640..28b9082693c 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/violations/ViolationQuery.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/violations/ViolationQuery.java @@ -28,7 +28,7 @@ import org.sonar.api.resources.Resource; */ public final class ViolationQuery { - private boolean ignoreSwitchedOff; + private boolean isSwitchedOff; private Resource resource; /** @@ -50,21 +50,21 @@ public final class ViolationQuery { * Specifies if the query should returned switched-off violations or not. * * @param ignore - * if true, the query will return only active violations. + * if true, the query will return only switched-off violations. if false, it will return only active violations. * @return the current violation query */ - public ViolationQuery ignoreSwitchedOff(boolean ignore) { - this.ignoreSwitchedOff = ignore; + public ViolationQuery setSwitchedOff(boolean ignore) { + this.isSwitchedOff = ignore; return this; } /** - * Tells if the query should returned switched-off violations or not. + * Tells if the query should returned switched-off violations or active violations. * * @return */ - public boolean ignoreSwitchedOff() { - return ignoreSwitchedOff; + public boolean isSwitchedOff() { + return isSwitchedOff; } /** |