fix default assignee when property is null - SONAR-6154

diff --git a/server/sonar-server/src/main/java/org/sonar/server/computation/issue/IssueComputation.java b/server/sonar-server/src/main/java/org/sonar/server/computation/issue/IssueComputation.java
index 6e4a6c44349d4356669310fbb6d804f96e4d2090..6400979ffb46fa28d600f4fbc7a6f3d22ee232e4 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/computation/issue/IssueComputation.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/computation/issue/IssueComputation.java
@@ -19,6 +19,7 @@
  */
 package org.sonar.server.computation.issue;
 
+import com.google.common.base.Strings;
 import com.google.common.collect.Sets;
 import org.sonar.api.CoreProperties;
 import org.sonar.api.issue.internal.DefaultIssue;
@@ -143,17 +144,19 @@ public class IssueComputation {
     issue.setTags(Sets.union(rule.getTags(), rule.getSystemTags()));
   }
 
-  private void computeDefaultAssignee(String login) {
+  private void computeDefaultAssignee(@Nullable String login) {
     if (hasAssigneeBeenComputed) {
       return;
     }
 
     hasAssigneeBeenComputed = true;
-    UserDoc user = userIndex.getNullableByLogin(login);
-    if (user == null) {
-      LOG.info("the {} property was set with an unknown login: {}", CoreProperties.DEFAULT_ISSUE_ASSIGNEE, login);
-    } else {
-      defaultAssignee = login;
+    if (!Strings.isNullOrEmpty(login)) {
+      UserDoc user = userIndex.getNullableByLogin(login);
+      if (user == null) {
+        LOG.info("the {} property was set with an unknown login: {}", CoreProperties.DEFAULT_ISSUE_ASSIGNEE, login);
+      } else {
+        defaultAssignee = login;
+      }
     }
   }
 }