aboutsummaryrefslogtreecommitdiffstats
path: root/sonar-server/src
diff options
context:
space:
mode:
authorJulien Lancelot <julien.lancelot@sonarsource.com>2014-03-06 09:53:44 +0100
committerJulien Lancelot <julien.lancelot@sonarsource.com>2014-03-06 12:15:22 +0100
commitfdab089af9a50ab912cbf0061e61f00075a6f264 (patch)
tree82cfc1040a14f0dcaa8ccd0b7d63c1711298415a /sonar-server/src
parent61445847aa308e79cf92ed308a43e15d165eb788 (diff)
downloadsonarqube-fdab089af9a50ab912cbf0061e61f00075a6f264.tar.gz
sonarqube-fdab089af9a50ab912cbf0061e61f00075a6f264.zip
SONAR-5056 Set Rule remediation factor and offset as String
Diffstat (limited to 'sonar-server/src')
-rw-r--r--sonar-server/src/main/java/org/sonar/server/rule/RuleRegistration.java21
1 files changed, 13 insertions, 8 deletions
diff --git a/sonar-server/src/main/java/org/sonar/server/rule/RuleRegistration.java b/sonar-server/src/main/java/org/sonar/server/rule/RuleRegistration.java
index 3356625a869..2ab48037f8e 100644
--- a/sonar-server/src/main/java/org/sonar/server/rule/RuleRegistration.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/RuleRegistration.java
@@ -225,6 +225,19 @@ public class RuleRegistration implements Startable {
dto.setStatus(status);
changed = true;
}
+ if (!StringUtils.equals(dto.getLanguage(), def.repository().language())) {
+ dto.setLanguage(def.repository().language());
+ changed = true;
+ }
+ changed = mergeDebtRule(def, dto) || changed;
+ if (changed) {
+ dto.setUpdatedAt(buffer.now());
+ }
+ return changed;
+ }
+
+ private boolean mergeDebtRule(RuleDefinitions.Rule def, RuleDto dto){
+ boolean changed = false;
// TODO add characteristic id change verification
@@ -246,14 +259,6 @@ public class RuleRegistration implements Startable {
dto.setEffortToFixL10nKey(def.effortToFixL10nKey());
changed = true;
}
-
- if (!StringUtils.equals(dto.getLanguage(), def.repository().language())) {
- dto.setLanguage(def.repository().language());
- changed = true;
- }
- if (changed) {
- dto.setUpdatedAt(buffer.now());
- }
return changed;
}