]> source.dussan.org Git - sonarqube.git/commitdiff
Fix quality flaws
authorJulien Lancelot <julien.lancelot@sonarsource.com>
Fri, 29 Jan 2016 14:21:56 +0000 (15:21 +0100)
committerJulien Lancelot <julien.lancelot@sonarsource.com>
Fri, 29 Jan 2016 14:21:56 +0000 (15:21 +0100)
12 files changed:
server/sonar-server/src/main/java/org/sonar/server/computation/formula/AverageFormula.java
server/sonar-server/src/main/java/org/sonar/server/computation/issue/TrackerRawInputFactory.java
server/sonar-server/src/main/java/org/sonar/server/computation/measure/api/MeasureComputerDefinitionImpl.java
server/sonar-server/src/main/java/org/sonar/server/computation/step/ComputeMeasureVariationsStep.java
server/sonar-server/src/main/java/org/sonar/server/computation/step/PersistComponentsStep.java
server/sonar-server/src/main/java/org/sonar/server/computation/step/PersistFileSourcesStep.java
server/sonar-server/src/main/java/org/sonar/server/computation/step/PersistProjectLinksStep.java
server/sonar-server/src/main/java/org/sonar/server/computation/step/PersistSnapshotsStep.java
server/sonar-server/src/main/java/org/sonar/server/measure/MeasureFilterFactory.java
server/sonar-server/src/main/java/org/sonar/server/permission/PermissionFinder.java
server/sonar-server/src/main/java/org/sonar/server/qualityprofile/QProfileLookup.java
sonar-plugin-api/src/main/java/org/sonar/api/ce/measure/test/TestMeasureComputerDefinition.java

index 48632600d8d4b2c274d6580c76a79885390d1e0d..f188506ce3d7ddcf8e2ce6e23d5aaa50d4097121 100644 (file)
@@ -153,25 +153,24 @@ public class AverageFormula implements Formula<AverageFormula.AverageCounter> {
       }
       return Optional.absent();
     }
-  }
 
-  private static Optional<Double> getDoubleValue(Optional<Measure> measureOptional) {
-    if (!measureOptional.isPresent()) {
-      return Optional.absent();
-    }
-    Measure measure = measureOptional.get();
-    switch (measure.getValueType()) {
-      case DOUBLE:
-        return Optional.of(measure.getDoubleValue());
-      case LONG:
-        return Optional.of((double) measure.getLongValue());
-      case INT:
-        return Optional.of((double) measure.getIntValue());
-      case NO_VALUE:
+    private Optional<Double> getDoubleValue(Optional<Measure> measureOptional) {
+      if (!measureOptional.isPresent()) {
         return Optional.absent();
-      default:
-        throw new IllegalArgumentException(String.format("Measure of type '%s' are not supported", measure.getValueType().name()));
+      }
+      Measure measure = measureOptional.get();
+      switch (measure.getValueType()) {
+        case DOUBLE:
+          return Optional.of(measure.getDoubleValue());
+        case LONG:
+          return Optional.of((double) measure.getLongValue());
+        case INT:
+          return Optional.of((double) measure.getIntValue());
+        case NO_VALUE:
+          return Optional.absent();
+        default:
+          throw new IllegalArgumentException(String.format("Measure of type '%s' are not supported", measure.getValueType().name()));
+      }
     }
   }
-
 }
index 1247466e20f39e619976d9df105eba94bafca621..8c403dc705a4cd7717db49a8e5b4285e8f894b4a 100644 (file)
@@ -170,22 +170,22 @@ public class TrackerRawInputFactory {
       }
       return target.build();
     }
-  }
 
-  private static DbCommons.TextRange.Builder convertTextRange(BatchReport.TextRange sourceRange) {
-    DbCommons.TextRange.Builder targetRange = DbCommons.TextRange.newBuilder();
-    if (sourceRange.hasStartLine()) {
-      targetRange.setStartLine(sourceRange.getStartLine());
-    }
-    if (sourceRange.hasStartOffset()) {
-      targetRange.setStartOffset(sourceRange.getStartOffset());
-    }
-    if (sourceRange.hasEndLine()) {
-      targetRange.setEndLine(sourceRange.getEndLine());
-    }
-    if (sourceRange.hasEndOffset()) {
-      targetRange.setEndOffset(sourceRange.getEndOffset());
+    private DbCommons.TextRange.Builder convertTextRange(BatchReport.TextRange sourceRange) {
+      DbCommons.TextRange.Builder targetRange = DbCommons.TextRange.newBuilder();
+      if (sourceRange.hasStartLine()) {
+        targetRange.setStartLine(sourceRange.getStartLine());
+      }
+      if (sourceRange.hasStartOffset()) {
+        targetRange.setStartOffset(sourceRange.getStartOffset());
+      }
+      if (sourceRange.hasEndLine()) {
+        targetRange.setEndLine(sourceRange.getEndLine());
+      }
+      if (sourceRange.hasEndOffset()) {
+        targetRange.setEndOffset(sourceRange.getEndOffset());
+      }
+      return targetRange;
     }
-    return targetRange;
   }
 }
index 18f8719d4be792ba249fd825c8aa1de9d9c30178..e2001f414c28e9baf699c140de082f605f32d622 100644 (file)
@@ -117,11 +117,11 @@ public class MeasureComputerDefinitionImpl implements MeasureComputer.MeasureCom
       checkNotNull(outputMetrics);
       return outputMetrics;
     }
-  }
 
-  private static void checkNotNull(String[] metrics) {
-    for (String metric : metrics) {
-      requireNonNull(metric, "Null metric is not allowed");
+    private static void checkNotNull(String[] metrics) {
+      for (String metric : metrics) {
+        requireNonNull(metric, "Null metric is not allowed");
+      }
     }
   }
 }
index 29dd8a0f43ff5948b314b4334b15ce01b42649c8..4930fe92fa9d145cdf6a85c90c2a3331fc873136 100644 (file)
@@ -153,20 +153,20 @@ public class ComputeMeasureVariationsStep implements ComputationStep {
         }
       }
     }
-  }
 
-  private static double computeVariation(Measure measure, Double pastValue) {
-    switch (measure.getValueType()) {
-      case INT:
-        return measure.getIntValue() - pastValue;
-      case LONG:
-        return measure.getLongValue() - pastValue;
-      case DOUBLE:
-        return measure.getDoubleValue() - pastValue;
-      case BOOLEAN:
-        return (measure.getBooleanValue() ? 1d : 0d) - pastValue;
-      default:
-        throw new IllegalArgumentException("Unsupported Measure.ValueType " + measure.getValueType());
+    private double computeVariation(Measure measure, Double pastValue) {
+      switch (measure.getValueType()) {
+        case INT:
+          return measure.getIntValue() - pastValue;
+        case LONG:
+          return measure.getLongValue() - pastValue;
+        case DOUBLE:
+          return measure.getDoubleValue() - pastValue;
+        case BOOLEAN:
+          return (measure.getBooleanValue() ? 1d : 0d) - pastValue;
+        default:
+          throw new IllegalArgumentException("Unsupported Measure.ValueType " + measure.getValueType());
+      }
     }
   }
 
index 8ed39412676731e5d8f6aa6cb2580e0e3a385777..972a1ce620e1ed85470920fd5a29a8775ae13ca9 100644 (file)
@@ -177,6 +177,10 @@ public class PersistComponentsStep implements ComputationStep {
         return existingComponent;
       }
     }
+
+    private void addToCache(Component component, ComponentDto componentDto) {
+      dbIdsRepository.setComponentId(component, componentDto.getId());
+    }
   }
 
   public ComponentDto createForProject(Component project) {
@@ -320,10 +324,6 @@ public class PersistComponentsStep implements ComputationStep {
     componentDto.setModuleUuidPath(parentModule.moduleUuidPath());
   }
 
-  private void addToCache(Component component, ComponentDto componentDto) {
-    dbIdsRepository.setComponentId(component, componentDto.getId());
-  }
-
   private static boolean updateExisting(ComponentDto existingComponent, ComponentDto newComponent) {
     boolean modified = false;
     if (!StringUtils.equals(existingComponent.name(), newComponent.name())) {
index 83476932c9c14ebaed42f3b8953303e8d6ba3984..47827c044b711281b8b3a39c4632959b2685b7fd 100644 (file)
@@ -180,22 +180,22 @@ public class PersistFileSourcesStep implements ComputationStep {
         }
       }
     }
-  }
 
-  @CheckForNull
-  private static String computeRevision(FileSourceDto previousDto, @Nullable Changeset latestChange) {
-    if (latestChange == null) {
-      return previousDto.getRevision();
+    @CheckForNull
+    private String computeRevision(FileSourceDto previousDto, @Nullable Changeset latestChange) {
+      if (latestChange == null) {
+        return previousDto.getRevision();
+      }
+      return latestChange.getRevision();
     }
-    return latestChange.getRevision();
-  }
 
-  @CheckForNull
-  private static String computeRevision(@Nullable Changeset latestChange) {
-    if (latestChange == null) {
-      return null;
+    @CheckForNull
+    private String computeRevision(@Nullable Changeset latestChange) {
+      if (latestChange == null) {
+        return null;
+      }
+      return latestChange.getRevision();
     }
-    return latestChange.getRevision();
   }
 
   private static class LineReaders {
index 4c69a5b11086fd2f6519e93447f128d03b7ca9c6..a25ccdc9f3790dddfa084e73ed4109a84e47c186 100644 (file)
@@ -144,14 +144,14 @@ public class PersistProjectLinksStep implements ComputationStep {
         }
       }
     }
-  }
 
-  private static String convertType(Constants.ComponentLinkType reportType) {
-    String type = typesConverter.get(reportType);
-    if (type != null) {
-      return type;
-    } else {
-      throw new IllegalArgumentException(String.format("Unsupported type %s", reportType.name()));
+    private String convertType(Constants.ComponentLinkType reportType) {
+      String type = typesConverter.get(reportType);
+      if (type != null) {
+        return type;
+      } else {
+        throw new IllegalArgumentException(String.format("Unsupported type %s", reportType.name()));
+      }
     }
   }
 
index 228ca609dc9cee2796257857fb3bea09d667ddaf..afb0bf5baa19cb0799c43502ef09983f186f0ea5 100644 (file)
@@ -139,6 +139,15 @@ public class PersistSnapshotsStep implements ComputationStep {
       commonForAnyVisit(projectView, path, snapshot);
     }
 
+    private void updateSnapshotPeriods(SnapshotDto snapshotDto) {
+      for (Period period : periodsHolder.getPeriods()) {
+        int index = period.getIndex();
+        snapshotDto.setPeriodMode(index, period.getMode());
+        snapshotDto.setPeriodParam(index, period.getModeParameter());
+        snapshotDto.setPeriodDate(index, period.getSnapshotDate());
+      }
+    }
+
     private void commonForAnyVisit(Component project, Path<SnapshotDtoHolder> path, SnapshotDto snapshot) {
       persist(snapshot, dbSession);
       addToCache(project, snapshot);
@@ -186,15 +195,6 @@ public class PersistSnapshotsStep implements ComputationStep {
     dbIdsRepository.setSnapshotId(component, snapshotDto.getId());
   }
 
-  private void updateSnapshotPeriods(SnapshotDto snapshotDto) {
-    for (Period period : periodsHolder.getPeriods()) {
-      int index = period.getIndex();
-      snapshotDto.setPeriodMode(index, period.getMode());
-      snapshotDto.setPeriodParam(index, period.getModeParameter());
-      snapshotDto.setPeriodDate(index, period.getSnapshotDate());
-    }
-  }
-
   private static String getFileQualifier(Component component) {
     return component.getFileAttributes().isUnitTest() ? Qualifiers.UNIT_TEST_FILE : Qualifiers.FILE;
   }
index 3877f9ab5f018c1b3edfab13c954c1f5b61d82e2..2364d1e3309f3b75897ae62fcf329ade14be4f39 100644 (file)
@@ -40,6 +40,7 @@ import org.sonar.api.utils.System2;
 
 import static com.google.common.collect.Lists.newArrayList;
 import static com.google.common.collect.Lists.transform;
+import static java.util.Collections.singletonList;
 
 @ServerSide
 public class MeasureFilterFactory {
@@ -164,11 +165,11 @@ public class MeasureFilterFactory {
     return null;
   }
 
-  private List<String> toList(@Nullable Object obj) {
+  private static List<String> toList(@Nullable Object obj) {
     List<String> result = null;
     if (obj != null) {
       if (obj instanceof String) {
-        result = Arrays.asList((String) obj);
+        result = singletonList((String) obj);
       } else {
         result = (List<String>) obj;
       }
@@ -177,7 +178,7 @@ public class MeasureFilterFactory {
   }
 
   @CheckForNull
-  private Date toDate(@Nullable String date) {
+  private static Date toDate(@Nullable String date) {
     if (date != null) {
       return DateUtils.parseDate(date);
     }
index d45b286f11e50d18696cb401feea1ad9bff74f7f..bbe04729bec6605dcf43cde291d6186bdb6b7824 100644 (file)
@@ -107,7 +107,7 @@ public class PermissionFinder {
    * As the anyone group does not exists in db, it's not returned when it has not the permission.
    * We have to manually add it at the begin of the list, if it matched the search text
    */
-  private void addAnyoneGroup(List<GroupWithPermissionDto> groups, PermissionQuery query) {
+  private static void addAnyoneGroup(List<GroupWithPermissionDto> groups, PermissionQuery query) {
     boolean hasAnyoneGroup = Iterables.any(groups, IsAnyoneGroup.INSTANCE);
     if (!hasAnyoneGroup
       && !GlobalPermissions.SYSTEM_ADMIN.equals(query.permission())
index cba8b46707c252b36235e94a6be85436bb0f5b84..13ed6c46c65ccd3c379219666ae739d2cce546f4 100644 (file)
@@ -25,9 +25,9 @@ import java.util.List;
 import javax.annotation.CheckForNull;
 import javax.annotation.Nonnull;
 import org.sonar.api.server.ServerSide;
+import org.sonar.db.DbClient;
 import org.sonar.db.DbSession;
 import org.sonar.db.qualityprofile.QualityProfileDto;
-import org.sonar.db.DbClient;
 
 import static com.google.common.collect.Lists.newArrayList;
 
@@ -144,7 +144,7 @@ public class QProfileLookup {
     }
   }
 
-  private List<QProfile> toQProfiles(List<QualityProfileDto> dtos) {
+  private static List<QProfile> toQProfiles(List<QualityProfileDto> dtos) {
     return newArrayList(Iterables.transform(dtos, ToQProfile.INSTANCE));
   }
 
index 2a11c1df180d7bfb90b8d0f647ea7775bbb5b026..97c9146653b5f421c03193e4819f0d25c8543f82 100644 (file)
@@ -92,11 +92,11 @@ public class TestMeasureComputerDefinition implements MeasureComputer.MeasureCom
       checkNotNull(outputMetrics);
       return outputMetrics;
     }
-  }
 
-  private static void checkNotNull(String[] metrics) {
-    for (String metric : metrics) {
-      requireNonNull(metric, "Null metric is not allowed");
+    private static void checkNotNull(String[] metrics) {
+      for (String metric : metrics) {
+        requireNonNull(metric, "Null metric is not allowed");
+      }
     }
   }