Fix some quality flaws

diff --git a/plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/lang/XooTokenizer.java b/plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/lang/XooTokenizer.java
index 760249e92351b65f82c33976cc84421870715ba5..2d5325bcbb09c0f5439ec34793bc31f04eba9e1a 100644 (file)
--- a/plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/lang/XooTokenizer.java
+++ b/plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/lang/XooTokenizer.java
@@ -25,13 +25,14 @@ import net.sourceforge.pmd.cpd.TokenEntry;
 import net.sourceforge.pmd.cpd.Tokenizer;
 import net.sourceforge.pmd.cpd.Tokens;
 import org.apache.commons.io.FileUtils;
-import org.sonar.api.BatchComponent;
+import org.sonar.api.BatchSide;
 import org.sonar.api.batch.fs.FileSystem;
 
 import java.io.File;
 import java.io.IOException;
 
-public class XooTokenizer implements Tokenizer, BatchComponent {
+@BatchSide
+public class XooTokenizer implements Tokenizer {
 
   private FileSystem fs;
 

diff --git a/sonar-batch/src/main/java/org/sonar/batch/maven/MavenProjectConverter.java b/sonar-batch/src/main/java/org/sonar/batch/maven/MavenProjectConverter.java
index 754472f67224c0a683007285913ddc4ceed3095a..21623db784df97749c5c3289dca21e749dc66ee4 100644 (file)
--- a/sonar-batch/src/main/java/org/sonar/batch/maven/MavenProjectConverter.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/maven/MavenProjectConverter.java
@@ -38,6 +38,7 @@ import org.sonar.api.task.TaskExtension;
 import org.sonar.api.utils.MessageException;
 import org.sonar.java.api.JavaUtils;
 
+import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
 import java.io.File;
@@ -292,7 +293,7 @@ public class MavenProjectConverter implements TaskExtension {
   private static String[] toPaths(Collection<File> dirs) {
     Collection<String> paths = Collections2.transform(dirs, new Function<File, String>() {
       @Override
-      public String apply(File dir) {
+      public String apply(@Nonnull File dir) {
         return dir.getAbsolutePath();
       }
     });

diff --git a/sonar-batch/src/main/java/org/sonar/batch/report/TestExecutionAndCoveragePublisher.java b/sonar-batch/src/main/java/org/sonar/batch/report/TestExecutionAndCoveragePublisher.java
index 9144de126bfda391c03e3d2070c55e4a2cb6ed72..344ba437ce99c5c45692be2e1cb643afc64be9a5 100644 (file)
--- a/sonar-batch/src/main/java/org/sonar/batch/report/TestExecutionAndCoveragePublisher.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/report/TestExecutionAndCoveragePublisher.java
@@ -36,6 +36,8 @@ import org.sonar.batch.protocol.output.BatchReport.Test;
 import org.sonar.batch.protocol.output.BatchReportWriter;
 import org.sonar.core.test.TestPlanBuilder;
 
+import javax.annotation.Nonnull;
+
 import java.util.HashSet;
 import java.util.Set;
 
@@ -50,7 +52,7 @@ public class TestExecutionAndCoveragePublisher implements ReportPublisherStep {
     }
 
     @Override
-    public Test apply(MutableTestCase testCase) {
+    public Test apply(@Nonnull MutableTestCase testCase) {
       builder.clear();
       builder.setName(testCase.name());
       if (testCase.doesCover()) {
@@ -86,7 +88,7 @@ public class TestExecutionAndCoveragePublisher implements ReportPublisherStep {
     }
 
     @Override
-    public CoverageDetail apply(String testName) {
+    public CoverageDetail apply(@Nonnull String testName) {
       // Take first test with provided name
       MutableTestCase testCase = testPlan.testCasesByName(testName).iterator().next();
       builder.clear();

diff --git a/sonar-batch/src/main/java/org/sonar/batch/rule/RuleFinderCompatibility.java b/sonar-batch/src/main/java/org/sonar/batch/rule/RuleFinderCompatibility.java
index dc497d31886fe345c868636fb632a308e0c73d04..6dab1413350f6c57e13fc9518275cda0a2ea642e 100644 (file)
--- a/sonar-batch/src/main/java/org/sonar/batch/rule/RuleFinderCompatibility.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/rule/RuleFinderCompatibility.java
@@ -32,6 +32,7 @@ import org.sonar.api.rules.RuleFinder;
 import org.sonar.api.rules.RuleQuery;
 
 import javax.annotation.CheckForNull;
+import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
 import java.util.Arrays;
@@ -82,25 +83,37 @@ public class RuleFinderCompatibility implements RuleFinder {
   public Collection<Rule> findAll(RuleQuery query) {
     if (query.getConfigKey() != null) {
       if (query.getRepositoryKey() != null && query.getKey() == null) {
-        Rule rule = toRule(activeRules.findByInternalKey(query.getRepositoryKey(), query.getConfigKey()));
-        return rule != null ? Arrays.asList(rule) : Collections.<Rule>emptyList();
+        return byInternalKey(query);
       }
     } else if (query.getRepositoryKey() != null) {
       if (query.getKey() != null) {
-        Rule rule = toRule(activeRules.find(RuleKey.of(query.getRepositoryKey(), query.getKey())));
-        return rule != null ? Arrays.asList(rule) : Collections.<Rule>emptyList();
+        return byKey(query);
       } else {
-        return Collections2.transform(activeRules.findByRepository(query.getRepositoryKey()), new Function<ActiveRule, Rule>() {
-          @Override
-          public Rule apply(ActiveRule input) {
-            return toRule(input);
-          }
-        });
+        return byRepository(query);
       }
     }
     throw new UnsupportedOperationException("Unable to find rule by query");
   }
 
+  private Collection<Rule> byRepository(RuleQuery query) {
+    return Collections2.transform(activeRules.findByRepository(query.getRepositoryKey()), new Function<ActiveRule, Rule>() {
+      @Override
+      public Rule apply(@Nonnull ActiveRule input) {
+        return toRule(input);
+      }
+    });
+  }
+
+  private Collection<Rule> byKey(RuleQuery query) {
+    Rule rule = toRule(activeRules.find(RuleKey.of(query.getRepositoryKey(), query.getKey())));
+    return rule != null ? Arrays.asList(rule) : Collections.<Rule>emptyList();
+  }
+
+  private Collection<Rule> byInternalKey(RuleQuery query) {
+    Rule rule = toRule(activeRules.findByInternalKey(query.getRepositoryKey(), query.getConfigKey()));
+    return rule != null ? Arrays.asList(rule) : Collections.<Rule>emptyList();
+  }
+
   @CheckForNull
   private Rule toRule(@Nullable ActiveRule rule) {
     DefaultActiveRule ar = (DefaultActiveRule) rule;