diff options
author | Duarte Meneses <duarte.meneses@sonarsource.com> | 2018-04-13 15:59:23 +0200 |
---|---|---|
committer | SonarTech <sonartech@sonarsource.com> | 2018-04-26 20:20:51 +0200 |
commit | 6c5aad9a2d48cd783993a3a31911e307bf6827f5 (patch) | |
tree | e12910dde39fb187663a7f9bf6c93d73bb45efef /sonar-plugin-api/src/main/java | |
parent | 4cbf46ae604e6619c5b1fdaab81f5dd28f464a0e (diff) | |
download | sonarqube-6c5aad9a2d48cd783993a3a31911e307bf6827f5.tar.gz sonarqube-6c5aad9a2d48cd783993a3a31911e307bf6827f5.zip |
SONAR-10543 Update test to v7.2 and remove descriptionUrl and ruleTitle from scanner
Diffstat (limited to 'sonar-plugin-api/src/main/java')
3 files changed, 1 insertions, 45 deletions
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/ExternalIssue.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/ExternalIssue.java index ec72c49de1a..b151a6582aa 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/ExternalIssue.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/ExternalIssue.java @@ -33,12 +33,6 @@ public interface ExternalIssue extends IIssue { Severity severity(); /** - * Link to a page describing more details about the rule that triggered this issue. - */ - @CheckForNull - String descriptionUrl(); - - /** * Effort to fix the issue, in minutes. */ @CheckForNull @@ -48,9 +42,5 @@ public interface ExternalIssue extends IIssue { * Type of the issue. */ RuleType type(); - - /** - * Short description of the rule. Should not depend on the issue being raised. - */ - String ruleTitle(); + } diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/NewExternalIssue.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/NewExternalIssue.java index 8049c44e1dc..d2e55a999ee 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/NewExternalIssue.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/NewExternalIssue.java @@ -63,11 +63,6 @@ public interface NewExternalIssue { NewExternalIssue addLocation(NewIssueLocation secondaryLocation); /** - * Add a URL pointing to more information about the rule triggering this issue. - */ - NewExternalIssue descriptionUrl(String url); - - /** * Register a flow for this issue. A flow is an ordered list of issue locations that help to understand the issue. * It should be a <b>path that backtracks the issue from its primary location to the start of the flow</b>. * Several flows can be registered. @@ -80,11 +75,6 @@ public interface NewExternalIssue { NewIssueLocation newLocation(); /** - * Add the description of the rule. Should not depend on the issue being raised. - */ - NewExternalIssue ruleTitle(String title); - - /** * Save the issue. If rule key is unknown or rule not enabled in the current quality profile then a warning is logged but no exception * is thrown. */ diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/internal/DefaultExternalIssue.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/internal/DefaultExternalIssue.java index 6ffcb5055e8..a0bd77b53c7 100644 --- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/internal/DefaultExternalIssue.java +++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/internal/DefaultExternalIssue.java @@ -34,9 +34,7 @@ import static java.util.Objects.requireNonNull; public class DefaultExternalIssue extends AbstractDefaultIssue<DefaultExternalIssue> implements ExternalIssue, NewExternalIssue { private Long effort; private Severity severity; - private String url; private RuleType type; - private String ruleTitle; public DefaultExternalIssue() { super(null); @@ -74,24 +72,12 @@ public class DefaultExternalIssue extends AbstractDefaultIssue<DefaultExternalIs requireNonNull(this.ruleKey, "Rule key is mandatory on external issue"); checkState(primaryLocation != null, "Primary location is mandatory on every external issue"); checkState(primaryLocation.inputComponent().isFile(), "External issues must be located in files"); - checkState(ruleTitle != null, "Rule title is mandatory on every external issue"); checkState(severity != null, "Severity is mandatory on every external issue"); checkState(type != null, "Type is mandatory on every external issue"); storage.store(this); } @Override - public DefaultExternalIssue descriptionUrl(String url) { - this.url = url; - return this; - } - - @Override - public String descriptionUrl() { - return url; - } - - @Override public RuleType type() { return type; } @@ -102,14 +88,4 @@ public class DefaultExternalIssue extends AbstractDefaultIssue<DefaultExternalIs return this; } - @Override - public String ruleTitle() { - return ruleTitle; - } - - @Override - public DefaultExternalIssue ruleTitle(String ruleTitle) { - this.ruleTitle = ruleTitle; - return this; - } } |