]> source.dussan.org Git - sonarqube.git/commitdiff
Fix some quality flaws
authorSimon Brandhof <simon.brandhof@sonarsource.com>
Tue, 1 Jul 2014 11:43:04 +0000 (13:43 +0200)
committerSimon Brandhof <simon.brandhof@sonarsource.com>
Tue, 1 Jul 2014 11:51:23 +0000 (13:51 +0200)
23 files changed:
plugins/sonar-core-plugin/src/main/java/org/sonar/plugins/core/issue/notification/NewIssuesEmailTemplate.java
sonar-batch/src/main/java/org/sonar/batch/bootstrap/DefaultPluginsReferential.java
sonar-batch/src/main/java/org/sonar/batch/languages/package-info.java
sonar-batch/src/main/java/org/sonar/batch/phases/GraphPersister.java
sonar-batch/src/main/java/org/sonar/batch/rules/package-info.java
sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/DefaultModuleFileSystem.java
sonar-batch/src/main/java/org/sonar/batch/scan2/package-info.java
sonar-batch/src/main/java/org/sonar/batch/settings/package-info.java
sonar-core/src/main/java/org/sonar/core/graph/graphson/GraphsonUtil.java
sonar-core/src/main/java/org/sonar/core/persistence/BatchSession.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/analyzer/internal/package-info.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/analyzer/issue/internal/package-info.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/analyzer/issue/package-info.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/analyzer/measure/internal/package-info.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/analyzer/measure/package-info.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/analyzer/package-info.java
sonar-plugin-api/src/main/java/org/sonar/api/batch/fs/internal/DefaultFileSystem.java
sonar-server/src/main/java/org/sonar/server/activity/index/ActivityNormalizer.java
sonar-server/src/main/java/org/sonar/server/db/migrations/v43/NotResolvedIssuesOnRemovedComponentsMigration.java
sonar-server/src/main/java/org/sonar/server/platform/ServerComponents.java
sonar-server/src/main/java/org/sonar/server/rule/index/RuleDoc.java
sonar-server/src/main/java/org/sonar/server/search/ws/BaseMapping.java
sonar-ws-client/src/main/java/org/sonar/wsclient/unmarshallers/ServerSetupUnmarshaller.java

index b6d547d6d4c54fa3b091f09c279c0469e8e063b4..d8e94277b2a349fd8f50d60fa3c0f5ffdc38515e 100644 (file)
@@ -64,10 +64,10 @@ public class NewIssuesEmailTemplate extends EmailTemplate {
     sb.append("Project: ").append(projectName).append("\n\n");
     sb.append(notification.getFieldValue("count")).append(" new issues").append("\n\n");
     sb.append("   ");
-    for (Iterator<String> severityIterator = Lists.reverse(Severity.ALL).iterator(); severityIterator.hasNext(); ) {
+    for (Iterator<String> severityIterator = Lists.reverse(Severity.ALL).iterator(); severityIterator.hasNext();) {
       String severity = severityIterator.next();
-      String severityLabel = i18n.message(getLocale(), "severity."+ severity, severity);
-      sb.append(severityLabel).append(": ").append(notification.getFieldValue("count-"+ severity));
+      String severityLabel = i18n.message(getLocale(), "severity." + severity, severity);
+      sb.append(severityLabel).append(": ").append(notification.getFieldValue("count-" + severity));
       if (severityIterator.hasNext()) {
         sb.append("   ");
       }
@@ -76,12 +76,10 @@ public class NewIssuesEmailTemplate extends EmailTemplate {
 
     appendFooter(sb, notification);
 
-    EmailMessage message = new EmailMessage()
+    return new EmailMessage()
       .setMessageId("new-issues/" + notification.getFieldValue(FIELD_PROJECT_KEY))
       .setSubject(projectName + ": new issues")
       .setMessage(sb.toString());
-
-    return message;
   }
 
   private void appendFooter(StringBuilder sb, Notification notification) {
index 4d3c8d30d6470d2889bc5743ec709044fa16d385..1332f59b06235832e502cf367673cbc15d5ad101 100644 (file)
@@ -55,7 +55,7 @@ public class DefaultPluginsReferential implements PluginsReferential {
   public File pluginFile(final RemotePlugin remote) {
     try {
       final RemotePluginFile file = remote.file();
-      File cachedFile = fileCache.get(file.getFilename(), file.getHash(), new FileCache.Downloader() {
+      return fileCache.get(file.getFilename(), file.getHash(), new FileCache.Downloader() {
         public void download(String filename, File toFile) throws IOException {
           String url = "/deploy/plugins/" + remote.getKey() + "/" + file.getFilename();
           if (LOG.isDebugEnabled()) {
@@ -66,7 +66,6 @@ public class DefaultPluginsReferential implements PluginsReferential {
           server.download(url, toFile);
         }
       });
-      return cachedFile;
 
     } catch (Exception e) {
       throw new IllegalStateException("Fail to download plugin: " + remote.getKey(), e);
index ef8790b337f1425ffcac77b18930020b2635aced..7a5d7a96ff63d17a841b212375c8dc15d24cf0f9 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.batch.languages;
\ No newline at end of file
+package org.sonar.batch.languages;
index 3fc0f17db9a20a031a857d3643c762e05fb84b2e..32b26389862dc28450b4df0f7757f00f1d9e2408 100644 (file)
@@ -32,7 +32,6 @@ import org.sonar.core.graph.graphson.GraphsonMode;
 import org.sonar.core.graph.graphson.GraphsonWriter;
 import org.sonar.core.graph.jdbc.GraphDto;
 import org.sonar.core.graph.jdbc.GraphDtoMapper;
-import org.sonar.core.persistence.BatchSession;
 import org.sonar.core.persistence.DbSession;
 import org.sonar.core.persistence.MyBatis;
 
index 819e886c070d2fb04996ff6bde030e2a1513c4da..1522091f4ebad8cceb6d939d1bf0f14e4fd74415 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.batch.rules;
\ No newline at end of file
+package org.sonar.batch.rules;
index 7205c1c483f047fd8b5515b199e85ade50a45c1c..0e534a334f9b0021ac605eebac759a7c27e3a131 100644 (file)
@@ -70,17 +70,20 @@ public class DefaultModuleFileSystem extends DefaultFileSystem implements Module
   /**
    * Used by scan2 
    */
-  public DefaultModuleFileSystem(ModuleInputFileCache moduleInputFileCache, ProjectDefinition def, Settings settings, FileIndexer indexer, ModuleFileSystemInitializer initializer) {
+  public DefaultModuleFileSystem(ModuleInputFileCache moduleInputFileCache, ProjectDefinition def, Settings settings,
+                                 FileIndexer indexer, ModuleFileSystemInitializer initializer) {
     this(moduleInputFileCache, def.getKey(), settings, indexer, initializer, null);
   }
 
-  public DefaultModuleFileSystem(ModuleInputFileCache moduleInputFileCache, ProjectDefinition def, Project project, Settings settings, FileIndexer indexer,
+  public DefaultModuleFileSystem(ModuleInputFileCache moduleInputFileCache, ProjectDefinition def, Project project,
+                                 Settings settings, FileIndexer indexer,
     ModuleFileSystemInitializer initializer,
     ComponentIndexer componentIndexer) {
     this(moduleInputFileCache, project.getKey(), settings, indexer, initializer, componentIndexer);
   }
 
-  private DefaultModuleFileSystem(ModuleInputFileCache moduleInputFileCache, String moduleKey, Settings settings, FileIndexer indexer, ModuleFileSystemInitializer initializer,
+  private DefaultModuleFileSystem(ModuleInputFileCache moduleInputFileCache, String moduleKey, Settings settings,
+                                  FileIndexer indexer, ModuleFileSystemInitializer initializer,
     @Nullable ComponentIndexer componentIndexer) {
     super(moduleInputFileCache);
     this.componentIndexer = componentIndexer;
index d5d167bf737c0028bab2ef1868c7a875e2c7d819..62ae62716bb0f0394a9b3f80849e44a1204c7698 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.batch.scan2;
\ No newline at end of file
+package org.sonar.batch.scan2;
index 6d2df16d3ab1a004aaa2e5bb847dc2f56e0accf2..7e52b5b6d3e052f6fcdc32e04573c4000201015f 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.batch.settings;
\ No newline at end of file
+package org.sonar.batch.settings;
index f529267ab8082385706f839e9004ad6640f400de..fae97eca964353a4b291ac085db6065cba35caa0 100644 (file)
@@ -77,12 +77,12 @@ class GraphsonUtil {
    * A GraphSONUtility that includes the specified properties.
    */
   GraphsonUtil(GraphsonMode mode, ElementFactory factory,
-               Set<String> vertexPropertyKeys, Set<String> edgePropertyKeys) {
+    Set<String> vertexPropertyKeys, Set<String> edgePropertyKeys) {
     this(mode, factory, ElementPropertyConfig.includeProperties(vertexPropertyKeys, edgePropertyKeys));
   }
 
   GraphsonUtil(GraphsonMode mode, ElementFactory factory,
-               ElementPropertyConfig config) {
+    ElementPropertyConfig config) {
     this.vertexPropertyKeys = config.getVertexPropertyKeys();
     this.edgePropertyKeys = config.getEdgePropertyKeys();
     this.vertexPropertiesRule = config.getVertexPropertiesRule();
@@ -110,7 +110,7 @@ class GraphsonUtil {
    */
   static JSONObject jsonFromElement(Element element, @Nullable Set<String> propertyKeys, GraphsonMode mode) {
     GraphsonUtil graphson = element instanceof Edge ? new GraphsonUtil(mode, null, null, propertyKeys)
-        : new GraphsonUtil(mode, null, propertyKeys, null);
+      : new GraphsonUtil(mode, null, propertyKeys, null);
     return graphson.jsonFromElement(element);
   }
 
@@ -123,7 +123,7 @@ class GraphsonUtil {
    * @param propertyKeys a list of keys to include on reading of element properties
    */
   static Vertex vertexFromJson(JSONObject json, ElementFactory factory, GraphsonMode mode,
-                               Set<String> propertyKeys) throws IOException {
+    Set<String> propertyKeys) throws IOException {
     GraphsonUtil graphson = new GraphsonUtil(mode, factory, propertyKeys, null);
     return graphson.vertexFromJson(json);
   }
@@ -137,7 +137,7 @@ class GraphsonUtil {
    * @param propertyKeys a list of keys to include on reading of element properties
    */
   static Vertex vertexFromJson(String json, ElementFactory factory, GraphsonMode mode,
-                               Set<String> propertyKeys) throws ParseException {
+    Set<String> propertyKeys) throws ParseException {
     GraphsonUtil graphson = new GraphsonUtil(mode, factory, propertyKeys, null);
     return graphson.vertexFromJson(json);
   }
@@ -151,39 +151,35 @@ class GraphsonUtil {
    * @param propertyKeys a list of keys to include on reading of element properties
    */
   static Vertex vertexFromJson(InputStream json, ElementFactory factory, GraphsonMode mode,
-                               Set<String> propertyKeys) throws IOException, ParseException {
+    Set<String> propertyKeys) throws IOException, ParseException {
     GraphsonUtil graphson = new GraphsonUtil(mode, factory, propertyKeys, null);
     return graphson.vertexFromJson(json);
   }
 
   private static boolean includeReservedKey(GraphsonMode mode, String key,
-                                            Set<String> propertyKeys,
-                                            ElementPropertiesRule rule) {
-    // the key is always included in modes other than compact.  if it is compact, then validate that the
+    Set<String> propertyKeys,
+    ElementPropertiesRule rule) {
+    // the key is always included in modes other than compact. if it is compact, then validate that the
     // key is in the property key list
     return mode != GraphsonMode.COMPACT || includeKey(key, propertyKeys, rule);
   }
 
   private static boolean includeKey(String key, Set<String> propertyKeys,
-                                    ElementPropertiesRule rule) {
+    ElementPropertiesRule rule) {
     if (propertyKeys == null) {
       // when null always include the key and shortcut this piece
       return true;
     }
 
-    // default the key situation.  if it's included then it should be explicitly defined in the
+    // default the key situation. if it's included then it should be explicitly defined in the
     // property keys list to be included or the reverse otherwise
     boolean keySituation = rule == ElementPropertiesRule.INCLUDE;
 
-    switch (rule) {
-      case INCLUDE:
-        keySituation = propertyKeys.contains(key);
-        break;
-      case EXCLUDE:
-        keySituation = !propertyKeys.contains(key);
-        break;
+    if (rule == ElementPropertiesRule.INCLUDE) {
+      keySituation = propertyKeys.contains(key);
+    } else if (rule == ElementPropertiesRule.EXCLUDE) {
+      keySituation = !propertyKeys.contains(key);
     }
-
     return keySituation;
   }
 
@@ -196,8 +192,8 @@ class GraphsonUtil {
    * @param propertyKeys a list of keys to include when reading of element properties
    */
   static Edge edgeFromJson(String json, Vertex out, Vertex in,
-                           ElementFactory factory, GraphsonMode mode,
-                           Set<String> propertyKeys) throws IOException, ParseException {
+    ElementFactory factory, GraphsonMode mode,
+    Set<String> propertyKeys) throws IOException, ParseException {
     GraphsonUtil graphson = new GraphsonUtil(mode, factory, null, propertyKeys);
     return graphson.edgeFromJson(json, out, in);
   }
@@ -211,8 +207,8 @@ class GraphsonUtil {
    * @param propertyKeys a list of keys to include when reading of element properties
    */
   static Edge edgeFromJson(InputStream json, Vertex out, Vertex in,
-                           ElementFactory factory, GraphsonMode mode,
-                           Set<String> propertyKeys) throws IOException, ParseException {
+    ElementFactory factory, GraphsonMode mode,
+    Set<String> propertyKeys) throws IOException, ParseException {
     GraphsonUtil graphson = new GraphsonUtil(mode, factory, null, propertyKeys);
     return graphson.edgeFromJson(json, out, in);
   }
@@ -226,8 +222,8 @@ class GraphsonUtil {
    * @param propertyKeys a list of keys to include when reading of element properties
    */
   static Edge edgeFromJson(JSONObject json, Vertex out, Vertex in,
-                           ElementFactory factory, GraphsonMode mode,
-                           Set<String> propertyKeys) throws IOException {
+    ElementFactory factory, GraphsonMode mode,
+    Set<String> propertyKeys) throws IOException {
     GraphsonUtil graphson = new GraphsonUtil(mode, factory, null, propertyKeys);
     return graphson.edgeFromJson(json, out, in);
   }
@@ -249,7 +245,7 @@ class GraphsonUtil {
 
   private static boolean isReservedKey(String key) {
     return key.equals(GraphsonTokens._ID) || key.equals(GraphsonTokens._TYPE) || key.equals(GraphsonTokens._LABEL)
-        || key.equals(GraphsonTokens._OUT_V) || key.equals(GraphsonTokens._IN_V);
+      || key.equals(GraphsonTokens._OUT_V) || key.equals(GraphsonTokens._IN_V);
   }
 
   private static JSONArray createJSONList(List list, Set<String> propertyKeys, boolean showTypes) {
@@ -285,7 +281,7 @@ class GraphsonUtil {
           value = createJSONMap((Map) value, propertyKeys, showTypes);
         } else if (value instanceof Element) {
           value = jsonFromElement((Element) value, propertyKeys,
-              showTypes ? GraphsonMode.EXTENDED : GraphsonMode.NORMAL);
+            showTypes ? GraphsonMode.EXTENDED : GraphsonMode.NORMAL);
         } else if (value.getClass().isArray()) {
           value = createJSONList(convertArrayToList(value), propertyKeys, showTypes);
         }
@@ -466,7 +462,7 @@ class GraphsonUtil {
 
       } else if (type.equals(GraphsonTokens.TYPE_MAP)) {
 
-        // maps are converted to a ObjectNode.  called recursively to traverse
+        // maps are converted to a ObjectNode. called recursively to traverse
         // the entire object graph within the map.
         JSONObject convertedMap = new JSONObject();
         JSONObject jsonObject = (JSONObject) value;
@@ -482,7 +478,7 @@ class GraphsonUtil {
 
       } else {
 
-        // this must be a primitive value or a complex object.  if a complex
+        // this must be a primitive value or a complex object. if a complex
         // object it will be handled by a call to toString and stored as a
         // string value
         putObject(valueAndType, GraphsonTokens.VALUE, value);
@@ -637,7 +633,7 @@ class GraphsonUtil {
       putObject(jsonElement, GraphsonTokens._ID, element.getId());
     }
 
-    // it's important to keep the order of these straight.  check Edge first and then Vertex because there
+    // it's important to keep the order of these straight. check Edge first and then Vertex because there
     // are graph implementations that have Edge extend from Vertex
     if (element instanceof Edge) {
       Edge edge = (Edge) element;
index a12942e13a97d6edfd67d1d680a2e04c813e8916..d7f3d8255bd69ff626f7b04a6287b741d07d5759 100644 (file)
@@ -29,7 +29,6 @@ import org.apache.ibatis.session.RowBounds;
 import org.apache.ibatis.session.SqlSession;
 import org.sonar.core.cluster.WorkQueue;
 
-import java.sql.Connection;
 import java.util.List;
 import java.util.Map;
 
@@ -40,7 +39,6 @@ public class BatchSession extends DbSession {
   private final int batchSize;
   private int count = 0;
 
-
   BatchSession(WorkQueue queue, SqlSession session) {
     this(queue, session, MAX_BATCH_SIZE);
   }
@@ -50,67 +48,80 @@ public class BatchSession extends DbSession {
     this.batchSize = batchSize;
   }
 
+  @Override
   public void select(String statement, Object parameter, ResultHandler handler) {
     reset();
     super.select(statement, parameter, handler);
   }
 
+  @Override
   public void select(String statement, ResultHandler handler) {
     reset();
     super.select(statement, handler);
   }
 
+  @Override
   public <T> T selectOne(String statement) {
     reset();
     return (T) super.selectOne(statement);
   }
 
+  @Override
   public <T> T selectOne(String statement, Object parameter) {
     reset();
     return (T) super.selectOne(statement, parameter);
   }
 
+  @Override
   public <E> List<E> selectList(String statement) {
     reset();
     return super.selectList(statement);
   }
 
+  @Override
   public <E> List<E> selectList(String statement, Object parameter) {
     reset();
     return super.selectList(statement, parameter);
   }
 
+  @Override
   public <E> List<E> selectList(String statement, Object parameter, RowBounds rowBounds) {
     reset();
     return super.selectList(statement, parameter, rowBounds);
   }
 
+  @Override
   public <K, V> Map<K, V> selectMap(String statement, String mapKey) {
     reset();
     return super.selectMap(statement, mapKey);
   }
 
+  @Override
   public <K, V> Map<K, V> selectMap(String statement, Object parameter, String mapKey) {
     reset();
     return super.selectMap(statement, parameter, mapKey);
   }
 
+  @Override
   public <K, V> Map<K, V> selectMap(String statement, Object parameter, String mapKey, RowBounds rowBounds) {
     reset();
     return super.selectMap(statement, parameter, mapKey, rowBounds);
   }
 
+  @Override
   public void select(String statement, Object parameter, RowBounds rowBounds, ResultHandler handler) {
     reset();
     super.select(statement, parameter, rowBounds, handler);
   }
 
+  @Override
   public int insert(String statement) {
     makeSureGeneratedKeysAreNotUsedInBatchInserts(statement);
     increment();
     return super.insert(statement);
   }
 
+  @Override
   public int insert(String statement, Object parameter) {
     makeSureGeneratedKeysAreNotUsedInBatchInserts(statement);
     increment();
@@ -130,72 +141,66 @@ public class BatchSession extends DbSession {
     }
   }
 
+  @Override
   public int update(String statement) {
     increment();
     return super.update(statement);
   }
 
+  @Override
   public int update(String statement, Object parameter) {
     increment();
     return super.update(statement, parameter);
   }
 
+  @Override
   public int delete(String statement) {
     increment();
     return super.delete(statement);
   }
 
+  @Override
   public int delete(String statement, Object parameter) {
     increment();
     return super.delete(statement, parameter);
   }
 
+  @Override
   public void commit() {
     super.commit();
     reset();
   }
 
+  @Override
   public void commit(boolean force) {
     super.commit(force);
     reset();
   }
 
+  @Override
   public void rollback() {
     super.rollback();
     reset();
   }
 
+  @Override
   public void rollback(boolean force) {
     super.rollback(force);
     reset();
   }
 
+  @Override
   public List<BatchResult> flushStatements() {
     List<BatchResult> batchResults = super.flushStatements();
     reset();
     return batchResults;
   }
 
-  public void close() {
-    super.close();
-  }
-
-  public void clearCache() {
-    super.clearCache();
-  }
-
-  public Configuration getConfiguration() {
-    return super.getConfiguration();
-  }
-
+  @Override
   public <T> T getMapper(Class<T> type) {
     return getConfiguration().getMapper(type, this);
   }
 
-  public Connection getConnection() {
-    return super.getConnection();
-  }
-
   private BatchSession increment() {
     count += 1;
     if (count >= batchSize) {
index faacc1f77a6fd40283e5593228c4db11b1b34603..b61bc2267b80134170eaf45b02faf2c22cc0d36c 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.api.batch.analyzer.internal;
\ No newline at end of file
+package org.sonar.api.batch.analyzer.internal;
index 61cb3773bcfc3f5ceb6a16063e20c46d86edaef0..4034ce7fdfa6f91abec305a0650d030124e1972b 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.api.batch.analyzer.issue.internal;
\ No newline at end of file
+package org.sonar.api.batch.analyzer.issue.internal;
index 1f3b2671fab6fea2dc217f3956f994c4532fcdb9..7ccdcaf5e21e813b5ff6e5bb84a9c2f37c755745 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.api.batch.analyzer.issue;
\ No newline at end of file
+package org.sonar.api.batch.analyzer.issue;
index d948c6e9894effbab31255f96c23fb59187dad6b..a508e1eacb0692916dbcd53582103d120607dd3f 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.api.batch.analyzer.measure.internal;
\ No newline at end of file
+package org.sonar.api.batch.analyzer.measure.internal;
index 426294473207e5df91ad27180fd15f97385e7b30..a72c5a175f1d89b977c71e3a00b73f1ea620f20c 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.api.batch.analyzer.measure;
\ No newline at end of file
+package org.sonar.api.batch.analyzer.measure;
index 02b956c197644bea473a5cdb472e248de6708955..4ae26db9bf10687ba2c9cb7844418f92c3f1df2a 100644 (file)
@@ -18,4 +18,4 @@
  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
  */
 @javax.annotation.ParametersAreNonnullByDefault
-package org.sonar.api.batch.analyzer;
\ No newline at end of file
+package org.sonar.api.batch.analyzer;
index ac2abd69414e5519dfc0e7d51af799cfb4a79b6f..2002c6feb1f61732d96318c5ecd1f380b199091a 100644 (file)
@@ -204,7 +204,7 @@ public class DefaultFileSystem implements FileSystem {
     // nothing to do by default
   }
 
-  public static abstract class Cache {
+  public abstract static class Cache {
     protected abstract Iterable<InputFile> inputFiles();
 
     @CheckForNull
index 6758799a3d437c8525ed64a8536b7fd16bf8e749..96c2e6e7979867bad249e30a36e3905491d4bf36 100644 (file)
@@ -48,14 +48,14 @@ public class ActivityNormalizer extends BaseNormalizer<ActivityDto, String> {
 
   public static final class LogFields extends Indexable {
 
-    public final static IndexField KEY = add(IndexField.Type.STRING, "key");
-    public final static IndexField TYPE = addSortable(IndexField.Type.STRING, "type");
-    public final static IndexField ACTION = addSortable(IndexField.Type.STRING, "action");
-    public final static IndexField CREATED_AT = addSortable(IndexField.Type.DATE, "createdAt");
-    public final static IndexField UPDATED_AT = addSortable(IndexField.Type.DATE, BaseNormalizer.UPDATED_AT_FIELD);
-    public final static IndexField LOGIN = addSearchable(IndexField.Type.STRING, "login");
-    public final static IndexField DETAILS = addSearchable(IndexField.Type.OBJECT, "details");
-    public final static IndexField MESSAGE = addSearchable(IndexField.Type.STRING, "message");
+    public static final IndexField KEY = add(IndexField.Type.STRING, "key");
+    public static final IndexField TYPE = addSortable(IndexField.Type.STRING, "type");
+    public static final IndexField ACTION = addSortable(IndexField.Type.STRING, "action");
+    public static final IndexField CREATED_AT = addSortable(IndexField.Type.DATE, "createdAt");
+    public static final IndexField UPDATED_AT = addSortable(IndexField.Type.DATE, BaseNormalizer.UPDATED_AT_FIELD);
+    public static final IndexField LOGIN = addSearchable(IndexField.Type.STRING, "login");
+    public static final IndexField DETAILS = addSearchable(IndexField.Type.OBJECT, "details");
+    public static final IndexField MESSAGE = addSearchable(IndexField.Type.STRING, "message");
 
     public static Set<IndexField> ALL_FIELDS = getAllFields();
 
index a06d94615fd32b04e5e645464ed5c86a37bdf9c0..8563f27f283ac397ae906f8c586621eac561674f 100644 (file)
@@ -20,7 +20,6 @@
 
 package org.sonar.server.db.migrations.v43;
 
-import com.google.common.annotations.VisibleForTesting;
 import org.sonar.api.issue.Issue;
 import org.sonar.api.utils.System2;
 import org.sonar.core.persistence.Database;
@@ -81,7 +80,7 @@ public class NotResolvedIssuesOnRemovedComponentsMigration implements DatabaseMi
           return true;
         }
       }
-    );
+      );
   }
 
   private static class Row {
index 79b4fa8d66a1d84cbfdae2aad5cbe6aa981d58fc..3ecbb8bb0dc384c8316f1dd530bf2add13aa7e7b 100644 (file)
@@ -384,7 +384,7 @@ class ServerComponents {
    * when its schema is up-to-date.
    */
   Collection level3Components() {
-    List components = Lists.newArrayList(
+    return Lists.newArrayList(
       PersistentSettings.class,
       DefaultDatabaseConnector.class,
       ThreadLocalDatabaseSessionFactory.class,
@@ -394,7 +394,6 @@ class ServerComponents {
       UriReader.class,
       ServerIdGenerator.class
     );
-    return components;
   }
 
 
index 13b799f18703731addb63e7497993b8c7ed47471..0ff56b58ae1deb505d83f44db4c9f0afb64af5a3 100644 (file)
@@ -45,7 +45,7 @@ import java.util.Map;
  */
 public class RuleDoc extends BaseDoc implements Rule {
 
-  public final static String MANUAL_REPOSITORY = "manual";
+  public static final String MANUAL_REPOSITORY = "manual";
 
   RuleDoc(@Nullable Map<String, Object> fields) {
     super(fields);
index c36d5016a6155e6c8695aef572a33c9ec351228a..0d7e44d56d87f6a699787a79999ceab9815c45a3 100644 (file)
@@ -28,6 +28,7 @@ import org.sonar.server.search.IndexUtils;
 import org.sonar.server.search.QueryOptions;
 
 import javax.annotation.Nullable;
+
 import java.util.Arrays;
 import java.util.List;
 import java.util.Set;
@@ -107,7 +108,7 @@ public abstract class BaseMapping implements ServerComponent {
     void write(JsonWriter json, D doc);
   }
 
-  public static abstract class IndexField<D> implements Field<D> {
+  public abstract static class IndexField<D> implements Field<D> {
     protected final String[] indexFields;
 
     protected IndexField(String... indexFields) {
@@ -138,7 +139,8 @@ public abstract class BaseMapping implements ServerComponent {
     @Override
     public void write(JsonWriter json, BaseDoc doc) {
       Object val = doc.getNullableField(indexFields[0]);
-      if (val == null && indexFields.length == 2) { // There is an alternative value
+      if (val == null && indexFields.length == 2) {
+        // There is an alternative value
         val = doc.getNullableField(indexFields[1]);
       }
       json.prop(key, val != null ? val.toString() : null);
index 966d8b8c83738b9c115dbfe99343b63373b8533e..ae72b1324a9e2efba005edf46ca931ab6b84c65d 100644 (file)
@@ -31,10 +31,9 @@ public class ServerSetupUnmarshaller implements Unmarshaller<ServerSetup> {
   public ServerSetup toModel(String json) {
     WSUtils utils = WSUtils.getINSTANCE();
     Object map = utils.parse(json);
-    ServerSetup server = new ServerSetup()
-        .setStatus(utils.getString(map, "status"))
-        .setMessage(utils.getString(map, "msg"));
-    return server;
+    return new ServerSetup()
+      .setStatus(utils.getString(map, "status"))
+      .setMessage(utils.getString(map, "msg"));
   }
 
   public List<ServerSetup> toModels(String json) {