From 32986b9e625c58ca8e5cf774d32c19eb061bcbe5 Mon Sep 17 00:00:00 2001
From: Julien HENRY <julien.henry@sonarsource.com>
Date: Wed, 7 Oct 2015 10:27:39 +0200
Subject: SONAR-6913 Stop support of issue attributes on batch side

---
 .../main/java/org/sonar/api/batch/sensor/issue/Issue.java |  7 -------
 .../java/org/sonar/api/batch/sensor/issue/NewIssue.java   |  6 ------
 .../api/batch/sensor/issue/internal/DefaultIssue.java     | 15 ---------------
 .../src/main/java/org/sonar/api/issue/Issuable.java       |  3 +++
 4 files changed, 3 insertions(+), 28 deletions(-)

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

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/Issue.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/Issue.java
index 98a5eb32fd8..8ac79f4a924 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/Issue.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/Issue.java
@@ -21,7 +21,6 @@ package org.sonar.api.batch.sensor.issue;
 
 import com.google.common.annotations.Beta;
 import java.util.List;
-import java.util.Map;
 import javax.annotation.CheckForNull;
 import org.sonar.api.batch.rule.Severity;
 import org.sonar.api.batch.sensor.Sensor;
@@ -71,10 +70,4 @@ public interface Issue {
    */
   List<Flow> flows();
 
-  /**
-   * Key/value pair of attributes that are attached to the issue.
-   * @since 5.2
-   */
-  Map<String, String> attributes();
-
 }
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/NewIssue.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/NewIssue.java
index b6b2e068835..8fc8156d3ca 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/NewIssue.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/NewIssue.java
@@ -74,12 +74,6 @@ public interface NewIssue {
    */
   NewIssueLocation newLocation();
 
-  /**
-   * Attach a new attribute to the issue. Not used by SQ but can be reused later for integration needs (for example it is returned by WS).
-   * @since 5.2
-   */
-  NewIssue addAttribute(String key, String value);
-
   /**
    * 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/DefaultIssue.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/internal/DefaultIssue.java
index 10f3e83706c..7fc35eabe3a 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/internal/DefaultIssue.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/sensor/issue/internal/DefaultIssue.java
@@ -22,13 +22,10 @@ package org.sonar.api.batch.sensor.issue.internal;
 import com.google.common.base.Function;
 import com.google.common.base.Preconditions;
 import com.google.common.collect.ImmutableList;
-import com.google.common.collect.ImmutableMap;
 import com.google.common.collect.Lists;
 import java.util.ArrayList;
 import java.util.Arrays;
-import java.util.LinkedHashMap;
 import java.util.List;
-import java.util.Map;
 import javax.annotation.Nullable;
 import org.sonar.api.batch.rule.Severity;
 import org.sonar.api.batch.sensor.internal.DefaultStorable;
@@ -60,7 +57,6 @@ public class DefaultIssue extends DefaultStorable implements Issue, NewIssue {
   private Severity overriddenSeverity;
   private IssueLocation primaryLocation;
   private List<List<IssueLocation>> flows = new ArrayList<>();
-  private final Map<String, String> attributes = new LinkedHashMap<>();
 
   public DefaultIssue() {
     super(null);
@@ -118,17 +114,6 @@ public class DefaultIssue extends DefaultStorable implements Issue, NewIssue {
     return this;
   }
 
-  @Override
-  public DefaultIssue addAttribute(String key, String value) {
-    attributes.put(key, value);
-    return this;
-  }
-
-  @Override
-  public Map<String, String> attributes() {
-    return ImmutableMap.copyOf(attributes);
-  }
-
   @Override
   public RuleKey ruleKey() {
     return this.ruleKey;
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/issue/Issuable.java b/sonar-plugin-api/src/main/java/org/sonar/api/issue/Issuable.java
index de822ecb824..007ba81a117 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/issue/Issuable.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/issue/Issuable.java
@@ -119,6 +119,9 @@ public interface Issuable extends Perspective {
 
     IssueBuilder effortToFix(@Nullable Double d);
 
+    /**
+     * No more supported from batch side since 5.2
+     */
     IssueBuilder attribute(String key, @Nullable String value);
 
     Issue build();
-- 
cgit v1.2.3