SONAR-5912 fix notification of issue changes during analysis

SonarSource · Jan 26, 2015 · 31726f3 · 31726f3
1 parent 3de9100
commit 31726f3
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/server/sonar-server/src/main/java/org/sonar/server/computation/AnalysisReportService.java b/server/sonar-server/src/main/java/org/sonar/server/computation/AnalysisReportService.java
@@ -123,6 +123,7 @@ private DefaultIssue toIssue(ComputationContext context, ReportIssue issue) {
     defaultIssue.setChanged(issue.isChanged());
     defaultIssue.setNew(issue.isNew());
     defaultIssue.setSelectedAt(issue.selectedAt());
+    defaultIssue.setSendNotifications(issue.mustSendNotifications());
     return defaultIssue;
   }
 

diff --git a/sonar-batch-protocol/src/main/java/org/sonar/batch/protocol/output/issue/ReportIssue.java b/sonar-batch-protocol/src/main/java/org/sonar/batch/protocol/output/issue/ReportIssue.java
@@ -54,6 +54,7 @@ public class ReportIssue {
   private Long selectedAt;
   private String diffFields;
   private boolean isChanged;
+  private boolean mustSendNotification;
 
 
   public ReportIssue setKey(String key) {
@@ -286,4 +287,12 @@ public boolean isChanged() {
     return isChanged;
   }
 
+  public ReportIssue setMustSendNotification(boolean mustSendNotification) {
+    this.mustSendNotification = mustSendNotification;
+    return this;
+  }
+
+  public boolean mustSendNotifications() {
+    return mustSendNotification;
+  }
 }
diff --git a/sonar-batch/src/main/java/org/sonar/batch/report/IssuesPublisher.java b/sonar-batch/src/main/java/org/sonar/batch/report/IssuesPublisher.java
@@ -82,6 +82,7 @@ private ReportIssue toReportIssue(DefaultIssue issue) {
       .setSelectedAt(issue.selectedAt())
       .setDiffFields(toString(issue.currentChange()))
       .setTags(issue.tags())
+      .setMustSendNotification(issue.mustSendNotifications())
       .setChanged(issue.isChanged());
   }