From 6c5aad9a2d48cd783993a3a31911e307bf6827f5 Mon Sep 17 00:00:00 2001
From: Duarte Meneses <duarte.meneses@sonarsource.com>
Date: Fri, 13 Apr 2018 15:59:23 +0200
Subject: SONAR-10543 Update test to v7.2 and remove descriptionUrl and
 ruleTitle from scanner

---
 .../api/batch/sensor/issue/ExternalIssue.java      | 12 +----------
 .../api/batch/sensor/issue/NewExternalIssue.java   | 10 ---------
 .../issue/internal/DefaultExternalIssue.java       | 24 ----------------------
 3 files changed, 1 insertion(+), 45 deletions(-)

(limited to 'sonar-plugin-api/src/main/java')

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
@@ -32,12 +32,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.
    */
@@ -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
@@ -62,11 +62,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>. 
@@ -79,11 +74,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,23 +72,11 @@ 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;
-  }
 }
-- 
cgit v1.2.3