SONAR-5056 Improve characteristic search

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 852273d40096634e7158a968bf232733414f029e..8a6bf727407c70b5b6302fedbffef2ae25f7e3b6 100644 (file)
--- a/sonar-server/src/main/java/org/sonar/server/rule/RuleRegistration.java
+++ b/sonar-server/src/main/java/org/sonar/server/rule/RuleRegistration.java
@@ -538,6 +538,9 @@ public class RuleRegistration implements Startable {
   @CheckForNull
   private CharacteristicDto findCharacteristic(List<CharacteristicDto> characteristicDtos, RuleDefinitions.Rule ruleDef) {
     final String key = ruleDef.characteristicKey();
+    if (key == null) {
+      return null;
+    }
     CharacteristicDto characteristicDto = Iterables.find(characteristicDtos, new Predicate<CharacteristicDto>() {
       @Override
       public boolean apply(CharacteristicDto input) {
@@ -547,7 +550,8 @@ public class RuleRegistration implements Startable {
     }, null);
     // TODO check not root characteristic
     if (characteristicDto == null) {
-      LOG.warn(String.format("Characteristic : '%s' has not been found, Technical debt definitions on rule '%s:%s' will be ignored", key, ruleDef.repository(), ruleDef.key()));
+      LOG.warn(String.format("Characteristic : '%s' has not been found, Technical debt definitions on rule '%s:%s' will be ignored", key,
+        ruleDef.repository().name(), ruleDef.key()));
     }
     return characteristicDto;
   }