aboutsummaryrefslogtreecommitdiffstats
path: root/plugins
diff options
context:
space:
mode:
authorJulien HENRY <julien.henry@sonarsource.com>2014-11-03 15:59:10 +0100
committerJulien HENRY <julien.henry@sonarsource.com>2014-11-03 15:59:10 +0100
commit0b4f1ffbd4231942dd9d536a99f77411a17301c1 (patch)
tree2f23f2a8663bcee27d04dec4b80479c4a27517fe /plugins
parent38a55b31c34ae84caa107935337cd147de2c2ec0 (diff)
downloadsonarqube-0b4f1ffbd4231942dd9d536a99f77411a17301c1.tar.gz
sonarqube-0b4f1ffbd4231942dd9d536a99f77411a17301c1.zip
Fix some quality flaws
Diffstat (limited to 'plugins')
-rw-r--r--plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/IssueTrackingResult.java11
-rw-r--r--plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/scm/XooScmProvider.java7
2 files changed, 15 insertions, 3 deletions
diff --git a/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/IssueTrackingResult.java b/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/IssueTrackingResult.java
index dd69da8e23c..68e24d97bae 100644
--- a/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/IssueTrackingResult.java
+++ b/plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/IssueTrackingResult.java
@@ -87,8 +87,7 @@ class IssueTrackingResult {
}
unmatchedByRuleAndKey.get(ruleKey).put(i.getKee(), i);
Map<Integer, Multimap<String, IssueDto>> unmatchedForRule = unmatchedByRuleAndLineAndChecksum.get(ruleKey);
- Integer line = i.getLine();
- Integer lineNotNull = line != null ? line : 0;
+ Integer lineNotNull = lineNotNull(i);
if (!unmatchedForRule.containsKey(lineNotNull)) {
unmatchedForRule.put(lineNotNull, HashMultimap.<String, IssueDto>create());
}
@@ -97,12 +96,18 @@ class IssueTrackingResult {
unmatchedForRuleAndLine.put(checksumNotNull, i);
}
+ private Integer lineNotNull(IssueDto i) {
+ Integer line = i.getLine();
+ Integer lineNotNull = line != null ? line : 0;
+ return lineNotNull;
+ }
+
void setMatch(DefaultIssue issue, IssueDto matching) {
matched.put(issue, matching);
RuleKey ruleKey = RuleKey.of(matching.getRuleRepo(), matching.getRule());
unmatchedByRuleAndKey.get(ruleKey).remove(matching.getKee());
unmatchedByKey.remove(matching.getKee());
- Integer lineNotNull = matching.getLine() != null ? matching.getLine() : 0;
+ Integer lineNotNull = lineNotNull(matching);
String checksumNotNull = StringUtils.defaultString(matching.getChecksum(), "");
unmatchedByRuleAndLineAndChecksum.get(ruleKey).get(lineNotNull).get(checksumNotNull).remove(matching);
}
diff --git a/plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/scm/XooScmProvider.java b/plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/scm/XooScmProvider.java
index 5807d820043..1dbcb2c04fd 100644
--- a/plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/scm/XooScmProvider.java
+++ b/plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/scm/XooScmProvider.java
@@ -22,6 +22,8 @@ package org.sonar.xoo.scm;
import org.sonar.api.batch.scm.BlameCommand;
import org.sonar.api.batch.scm.ScmProvider;
+import java.io.File;
+
public class XooScmProvider extends ScmProvider {
private final XooBlameCommand blame;
@@ -31,6 +33,11 @@ public class XooScmProvider extends ScmProvider {
}
@Override
+ public boolean supports(File baseDir) {
+ return new File(baseDir, ".xoo").exists();
+ }
+
+ @Override
public String key() {
return "xoo";
}