SONAR-5007 cleanup

diff --git a/sonar-server/src/main/java/org/sonar/server/rule/ws/AbstractUpdateTagsWsHandler.java b/sonar-server/src/main/java/org/sonar/server/rule/ws/AbstractUpdateTagsWsHandler.java
index 50a2b56312bcdc891a27a10d7c8be9884d77c51c..7c64895f12ed15a06efb7bce23b81033f4a5633a 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule/ws/AbstractUpdateTagsWsHandler.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/ws/AbstractUpdateTagsWsHandler.java
@@ -32,6 +32,10 @@ import org.sonar.server.rule.Rules;
 
 import java.util.Set;
 
+/**
+ * @deprecated to be dropped in 4.4
+ */
+@Deprecated
 public abstract class AbstractUpdateTagsWsHandler implements RequestHandler {
 
   private final Rules rules;

diff --git a/sonar-server/src/main/java/org/sonar/server/rule/ws/AddTagsWsHandler.java b/sonar-server/src/main/java/org/sonar/server/rule/ws/AddTagsWsHandler.java
index f20552b213d9a490d21c57b6b3f08a8cbe82b17f..38d2d2674414314b6cf32045b9ac4b777fcc3f9c 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule/ws/AddTagsWsHandler.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/ws/AddTagsWsHandler.java
@@ -24,6 +24,10 @@ import org.sonar.server.rule.Rules;
 import java.util.Arrays;
 import java.util.Set;
 
+/**
+ * @deprecated to be dropped in 4.4
+ */
+@Deprecated
 public class AddTagsWsHandler extends AbstractUpdateTagsWsHandler {
 
   public AddTagsWsHandler(Rules rules) {

diff --git a/sonar-server/src/main/java/org/sonar/server/rule/ws/RemoveTagsWsHandler.java b/sonar-server/src/main/java/org/sonar/server/rule/ws/RemoveTagsWsHandler.java
index 552db68e9bdb997573a520e45e64f7cd8f6709fe..9fe9c5ad77da3526ec676070cb1d84adce8d4c38 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule/ws/RemoveTagsWsHandler.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/ws/RemoveTagsWsHandler.java
@@ -24,6 +24,10 @@ import org.sonar.server.rule.Rules;
 import java.util.Arrays;
 import java.util.Set;
 
+/**
+ * @deprecated to be dropped in 4.4
+ */
+@Deprecated
 public class RemoveTagsWsHandler extends AbstractUpdateTagsWsHandler {
 
   public RemoveTagsWsHandler(Rules rules) {

diff --git a/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleSearchWsHandler.java b/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleSearchWsHandler.java
index 672bb91c6a9e11a7b2aa19e4cf9c3a279f3481ab..9aea4cb16b291ddbf8c1f7c6406f77d406c936fb 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleSearchWsHandler.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleSearchWsHandler.java
@@ -39,6 +39,10 @@ import javax.annotation.CheckForNull;
 import java.util.Collection;
 import java.util.Collections;
 
+/**
+ * @deprecated to be dropped in 4.4
+ */
+@Deprecated
 public class RuleSearchWsHandler implements RequestHandler {
 
   private final Rules rules;

diff --git a/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleShowWsHandler.java b/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleShowWsHandler.java
index 6a76e65d384136ff5a8597083cdc031b7d577178..c250950f5aa1358f47d408845c7928211910bca4 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleShowWsHandler.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleShowWsHandler.java
@@ -44,6 +44,10 @@ import javax.annotation.Nullable;
 
 import java.util.Date;
 
+/**
+ * @deprecated to be dropped in 4.4
+ */
+@Deprecated
 public class RuleShowWsHandler implements RequestHandler {
 
   private final Rules rules;

diff --git a/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleTagsWs.java b/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleTagsWs.java
index 69801e05e433bf9138d0f40e89098fa7ad8a1854..2628ee3653e8580829eb6be3abbf0b514d3e0887 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleTagsWs.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/ws/RuleTagsWs.java
@@ -27,6 +27,10 @@ import org.sonar.api.utils.text.JsonWriter;
 import org.sonar.core.rule.RuleTagDto;
 import org.sonar.server.rule.RuleTags;
 
+/**
+ * @deprecated to be dropped in 4.4
+ */
+@Deprecated
 public class RuleTagsWs implements WebService {
 
   private final RuleTags ruleTags;

diff --git a/sonar-server/src/main/java/org/sonar/server/rule/ws/RulesWs.java b/sonar-server/src/main/java/org/sonar/server/rule/ws/RulesWs.java
index 332cfc93d6f59a8167831033f9a59f37d839201e..75a4e13b64e9b0248ab98150357b30c0dfd01da4 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule/ws/RulesWs.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/ws/RulesWs.java
@@ -21,6 +21,10 @@ package org.sonar.server.rule.ws;
 
 import org.sonar.api.server.ws.WebService;
 
+/**
+ * @deprecated to be dropped in 4.4
+ */
+@Deprecated
 public class RulesWs implements WebService {
 
   private final RuleSearchWsHandler searchHandler;

diff --git a/sonar-server/src/main/java/org/sonar/server/rule2/RegisterRules.java b/sonar-server/src/main/java/org/sonar/server/rule2/RegisterRules.java
index 577ec2b67dbe4b73270970d0ebb0e1969a516427..afe1111ceacb8c7e7f36a7c828e8888b6819d2da 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule2/RegisterRules.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule2/RegisterRules.java
@@ -21,6 +21,7 @@ package org.sonar.server.rule2;
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Function;
+import com.google.common.collect.ImmutableSet;
 import com.google.common.collect.Iterables;
 import org.apache.commons.lang.ObjectUtils;
 import org.apache.commons.lang.StringUtils;
@@ -50,7 +51,6 @@ import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.Set;
 
 import static com.google.common.collect.Lists.newArrayList;
 
@@ -308,15 +308,14 @@ public class RegisterRules implements Startable {
   private boolean mergeTags(RulesDefinition.Rule ruleDef, RuleDto dto) {
     boolean changed = false;
 
-    //the Rule is not active and dto has tags
     if (Rule.STATUS_REMOVED.equals(ruleDef.status())) {
       dto.setSystemTags(Collections.EMPTY_SET);
       dto.setTags(Collections.EMPTY_SET);
       changed = true;
     } else if (!dto.getSystemTags().containsAll(ruleDef.tags())) {
-      Set<String> tags = dto.getTags();
-      tags.addAll(ruleDef.tags());
-      dto.setSystemTags(tags);
+      dto.setSystemTags(ruleDef.tags());
+      // remove end-user tags that are now declared as system
+      RuleTagHelper.applyTags(dto, ImmutableSet.copyOf(dto.getTags()));
       changed = true;
     }
     return changed;
@@ -382,7 +381,6 @@ public class RegisterRules implements Startable {
     for (RuleDto rule : removedRules) {
       // SONAR-4642 Remove active rules only when repository still exists
       if (repositoryKeys.contains(rule.getRepositoryKey())) {
-        // TODO
         profilesManager.removeActivatedRules(rule.getId());
       }
     }