SONAR-6012 Checksum can be null

diff --git a/server/sonar-server/src/main/java/org/sonar/server/batch/IssuesAction.java b/server/sonar-server/src/main/java/org/sonar/server/batch/IssuesAction.java
index c08b203edd52b69839d72951e93e02cc52c96c08..147d7d899f2df3be8e5d6b39789b14a3d2a83240 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/batch/IssuesAction.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/batch/IssuesAction.java
@@ -100,7 +100,10 @@ public class IssuesAction implements RequestHandler {
     }
     issueBuilder.setRuleRepository(issue.ruleKey().repository());
     issueBuilder.setRuleKey(issue.ruleKey().rule());
-    issueBuilder.setChecksum(issue.checksum());
+    String checksum = issue.checksum();
+    if (checksum != null) {
+      issueBuilder.setChecksum(checksum);
+    }
     String assigneeLogin = issue.assignee();
     if (assigneeLogin != null) {
       issueBuilder.setAssigneeLogin(assigneeLogin);

diff --git a/server/sonar-server/src/main/java/org/sonar/server/issue/index/IssueDoc.java b/server/sonar-server/src/main/java/org/sonar/server/issue/index/IssueDoc.java
index f8f6e2d40b68ce114c8d2dcac1aac25ba659286d..fdfedca9850f7e97922ca1eef636b77ebd558900 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/issue/index/IssueDoc.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/issue/index/IssueDoc.java
@@ -97,8 +97,9 @@ public class IssueDoc extends BaseDoc implements Issue {
     return BooleanUtils.isTrue((Boolean) getField(IssueIndexDefinition.FIELD_ISSUE_MANUAL_SEVERITY));
   }
 
+  @Nullable
   public String checksum() {
-    return getField(IssueIndexDefinition.FIELD_ISSUE_CHECKSUM);
+    return getNullableField(IssueIndexDefinition.FIELD_ISSUE_CHECKSUM);
   }
 
   @Override

diff --git a/server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java b/server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java
index 9d1fb293013a11d10f2a42dfdadd4da3ae123890..ed1efae697c5ea6ffe07973ac053742fcffb2687 100644 (file)
--- a/server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java
@@ -123,7 +123,7 @@ public class IssuesActionTest {
       .setManualSeverity(false)
       .setMessage(null)
       .setLine(null)
-      .setChecksum("123456")
+      .setChecksum(null)
       .setAssignee(null));
 
     MockUserSession.set().setLogin("henry").setGlobalPermissions(GlobalPermissions.PREVIEW_EXECUTION);
@@ -142,7 +142,7 @@ public class IssuesActionTest {
     assertThat(serverIssue.getStatus()).isEqualTo("RESOLVED");
     assertThat(serverIssue.getSeverity()).isEqualTo(Severity.BLOCKER);
     assertThat(serverIssue.getManualSeverity()).isFalse();
-    assertThat(serverIssue.getChecksum()).isEqualTo("123456");
+    assertThat(serverIssue.hasChecksum()).isFalse();
     assertThat(serverIssue.hasAssigneeLogin()).isFalse();
   }