diff options
author | Julien HENRY <julien.henry@sonarsource.com> | 2014-11-03 15:59:10 +0100 |
---|---|---|
committer | Julien HENRY <julien.henry@sonarsource.com> | 2014-11-03 15:59:10 +0100 |
commit | 0b4f1ffbd4231942dd9d536a99f77411a17301c1 (patch) | |
tree | 2f23f2a8663bcee27d04dec4b80479c4a27517fe /plugins | |
parent | 38a55b31c34ae84caa107935337cd147de2c2ec0 (diff) | |
download | sonarqube-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.java | 11 | ||||
-rw-r--r-- | plugins/sonar-xoo-plugin/src/main/java/org/sonar/xoo/scm/XooScmProvider.java | 7 |
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"; } |