]> source.dussan.org Git - sonarqube.git/commitdiff
Fix quality flaws
authorTeryk Bellahsene <teryk.bellahsene@sonarsource.com>
Mon, 21 Sep 2015 16:28:06 +0000 (18:28 +0200)
committerDuarte Meneses <duarte.meneses@sonarsource.com>
Wed, 30 Sep 2015 14:27:12 +0000 (16:27 +0200)
server/sonar-server/src/main/java/org/sonar/server/scanner/ProjectAction.java
server/sonar-server/src/main/java/org/sonar/server/scanner/ProjectDataLoader.java
server/sonar-server/src/main/java/org/sonar/server/util/ObjectUtils.java
sonar-batch/src/main/java/org/sonar/batch/repository/DefaultProjectRepositoriesLoader.java

index 31a4973a53d2ae729436dde775d0540bfb15a74f..2877759304056c5affbf928aae01520f40c7868f 100644 (file)
@@ -20,6 +20,7 @@
 
 package org.sonar.server.scanner;
 
+import java.util.Date;
 import java.util.HashMap;
 import java.util.Map;
 import org.sonar.api.server.ws.Request;
@@ -93,8 +94,9 @@ public class ProjectAction implements ScannerWsAction {
   }
 
   private static void setLastAnalysisDate(WsProjectResponse.Builder response, ProjectRepositories data) {
-    if (data.lastAnalysisDate() != null) {
-      response.setLastAnalysisDate(data.lastAnalysisDate().getTime());
+    Date lastAnalysisDate = data.lastAnalysisDate();
+    if (lastAnalysisDate != null) {
+      response.setLastAnalysisDate(lastAnalysisDate.getTime());
     }
   }
 
index 313022d32c65e0f041d81669a84b7299be8db14e..06ba815719e9100eb9f9808dd01fdc7b83ad4db6 100644 (file)
@@ -28,8 +28,6 @@ import java.util.Date;
 import java.util.List;
 import java.util.Map;
 import org.sonar.api.server.ServerSide;
-import org.sonar.api.utils.log.Logger;
-import org.sonar.api.utils.log.Loggers;
 import org.sonar.api.web.UserRole;
 import org.sonar.batch.protocol.input.FileData;
 import org.sonar.batch.protocol.input.ProjectRepositories;
@@ -51,8 +49,6 @@ import static org.sonar.server.ws.WsUtils.checkFound;
 @ServerSide
 public class ProjectDataLoader {
 
-  private static final Logger LOG = Loggers.get(ProjectDataLoader.class);
-
   private final DbClient dbClient;
   private final UserSession userSession;
 
index 09b57e70df2ea27b0a4dda61a3994dd3d2e9aedd..f5f7542b314419854cc060664642f21fba582a8c 100644 (file)
 package org.sonar.server.util;
 
 public class ObjectUtils {
+  private ObjectUtils() {
+    // static utility methods only
+  }
 
   /**
    * Taken from http://commons.apache.org/proper/commons-lang/javadocs/api-3.1/src-html/org/apache/commons/lang3/ObjectUtils.html#line.119
+   * 
    * <p>Returns the first value in the array which is not {@code null}.
     * If all the values are {@code null} or the array is {@code null}
     * or empty then {@code null} is returned.</p>
index add0f3d157eb5d492344171c1948c3cb9e7cd162..ab0f960c6a1580d5d4436e258904e7d8e26942dd 100644 (file)
@@ -21,8 +21,8 @@ package org.sonar.batch.repository;
 
 import javax.annotation.Nullable;
 import org.apache.commons.lang.mutable.MutableBoolean;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import org.sonar.api.utils.log.Logger;
+import org.sonar.api.utils.log.Loggers;
 import org.sonar.batch.analysis.DefaultAnalysisMode;
 import org.sonar.batch.cache.WSLoader;
 import org.sonar.batch.cache.WSLoaderResult;
@@ -32,7 +32,7 @@ import org.sonar.batch.util.BatchUtils;
 
 public class DefaultProjectRepositoriesLoader implements ProjectRepositoriesLoader {
 
-  private static final Logger LOG = LoggerFactory.getLogger(DefaultProjectRepositoriesLoader.class);
+  private static final Logger LOG = Loggers.get(DefaultProjectRepositoriesLoader.class);
   private static final String BATCH_PROJECT_URL = "/scanner/project";
 
   private final WSLoader wsLoader;
@@ -53,13 +53,12 @@ public class DefaultProjectRepositoriesLoader implements ProjectRepositoriesLoad
     }
     url += "&preview=" + analysisMode.isIssues();
 
-    ProjectRepositories projectRepositories = load(url, fromCache);
-    return projectRepositories;
+    return load(url, fromCache);
   }
 
   private ProjectRepositories load(String resource, @Nullable MutableBoolean fromCache) {
     WSLoaderResult<String> result = wsLoader.loadString(resource);
-    if(fromCache != null) {
+    if (fromCache != null) {
       fromCache.setValue(result.isFromCache());
     }
     return ProjectRepositories.fromJson(result.get());