From 404a10668a63b7ec7a3f88f72a89660cbb546eff Mon Sep 17 00:00:00 2001
From: Simon Brandhof <simon.brandhof@sonarsource.com>
Date: Fri, 23 Jan 2015 13:15:31 +0100
Subject: [PATCH] Computation stack must not update issues in db if no fields
 were changed

---
 .../computation/step/PersistIssuesStep.java   | 24 ++++++++++---------
 1 file changed, 13 insertions(+), 11 deletions(-)

diff --git a/server/sonar-server/src/main/java/org/sonar/server/computation/step/PersistIssuesStep.java b/server/sonar-server/src/main/java/org/sonar/server/computation/step/PersistIssuesStep.java
index e64ce2c590f..d081874346a 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/computation/step/PersistIssuesStep.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/computation/step/PersistIssuesStep.java
@@ -67,34 +67,36 @@ public class PersistIssuesStep implements ComputationStep {
     try {
       while (issues.hasNext()) {
         DefaultIssue issue = issues.next();
-
+        boolean saved = false;
         if (issue.isNew()) {
           Integer ruleId = ruleCache.get(issue.ruleKey()).getId();
           mapper.insert(IssueDto.toDtoForBatchInsert(issue, issue.componentId(), context.getProject().getId(), ruleId, system2.now()));
           count++;
-        } else {
+          saved = true;
+        } else if (issue.isChanged()) {
           IssueDto dto = IssueDto.toDtoForUpdate(issue, context.getProject().getId(), system2.now());
           if (Issue.STATUS_CLOSED.equals(issue.status()) || issue.selectedAt() == null) {
             // Issue is closed by scan or changed by end-user
             mapper.update(dto);
-
+            count++;
           } else {
             int updateCount = mapper.updateIfBeforeSelectedDate(dto);
+            count++;
             if (updateCount == 0) {
               // End-user and scan changed the issue at the same time.
               // See https://jira.codehaus.org/browse/SONAR-4309
               conflictResolver.resolve(issue, mapper);
             }
           }
-
-          count++;
+          saved = true;
         }
-        count += insertChanges(changeMapper, issue);
-
-        if (count > BatchSession.MAX_BATCH_SIZE) {
-          session.flushStatements();
-          session.commit();
-          count = 0;
+        if (saved) {
+          count += insertChanges(changeMapper, issue);
+          if (count > BatchSession.MAX_BATCH_SIZE) {
+            session.flushStatements();
+            session.commit();
+            count = 0;
+          }
         }
       }
       session.flushStatements();
-- 
2.39.5