From: simonbrandhof <simon.brandhof@gmail.com>
Date: Fri, 24 Sep 2010 09:49:34 +0000 (+0000)
Subject: do not keep in cache violations in DecoratorContext
X-Git-Tag: 2.6~956
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=60ae8203486a7eed8a4e3b209820c19851b8c309;p=sonarqube.git

do not keep in cache violations in DecoratorContext
---

diff --git a/sonar-batch/src/main/java/org/sonar/batch/DefaultDecoratorContext.java b/sonar-batch/src/main/java/org/sonar/batch/DefaultDecoratorContext.java
index 05314d3040e..f812bd0e8fe 100644
--- a/sonar-batch/src/main/java/org/sonar/batch/DefaultDecoratorContext.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/DefaultDecoratorContext.java
@@ -43,7 +43,6 @@ public class DefaultDecoratorContext implements DecoratorContext {
   private boolean readOnly = false;
 
   private List<DecoratorContext> childrenContexts;
-  private List<Violation> violations;
   private ViolationsDao violationsDao;
 
   public DefaultDecoratorContext(Resource resource,
@@ -60,7 +59,6 @@ public class DefaultDecoratorContext implements DecoratorContext {
 
   public DefaultDecoratorContext setReadOnly(boolean b) {
     readOnly = b;
-    violations = null;
     childrenContexts = null;
     return this;
   }
@@ -125,13 +123,12 @@ public class DefaultDecoratorContext implements DecoratorContext {
 
 
   public List<Violation> getViolations() {
-    if (violations == null) {
       Bucket bucket = index.getBucket(resource);
       if (bucket != null && bucket.getSnapshotId() != null) {
-        violations = violationsDao.getViolations(resource, bucket.getSnapshotId());
+        return violationsDao.getViolations(resource, bucket.getSnapshotId());
       }
-    }
-    return violations;
+
+    return null;
   }
 
   public Dependency saveDependency(Dependency dependency) {