diff options
author | Julien Lancelot <julien.lancelot@sonarsource.com> | 2014-03-06 09:53:44 +0100 |
---|---|---|
committer | Julien Lancelot <julien.lancelot@sonarsource.com> | 2014-03-06 12:15:22 +0100 |
commit | fdab089af9a50ab912cbf0061e61f00075a6f264 (patch) | |
tree | 82cfc1040a14f0dcaa8ccd0b7d63c1711298415a /sonar-server/src | |
parent | 61445847aa308e79cf92ed308a43e15d165eb788 (diff) | |
download | sonarqube-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.java | 21 |
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; } |