Fix quality flaws

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/FacetValue.java b/server/sonar-server/src/main/java/org/sonar/server/search/FacetValue.java
index ab447a9db5cf95c4e0853d684140d81caedfcd3b..887e13847beb8eace9eede23f53ecebab8680fb0 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/search/FacetValue.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/FacetValue.java
@@ -75,23 +75,27 @@ public class FacetValue implements Comparable<FacetValue> {
   public int compareTo(FacetValue other) {
     if (this.sort.equals(Sort.BY_KEY)) {
       return this.getKey().compareTo(other.getKey());
-    } else {
-      return this.getValue().compareTo(other.getValue());
     }
+    return this.getValue().compareTo(other.getValue());
   }
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-    if (!(o instanceof FacetValue)) return false;
+    if (this == o) {
+      return true;
+    }
+    if (!(o instanceof FacetValue)) {
+      return false;
+    }
 
     FacetValue that = (FacetValue) o;
-
-    if (!key.equals(that.key)) return false;
-    if (subFacets != null ? !subFacets.equals(that.subFacets) : that.subFacets != null) return false;
-    if (!value.equals(that.value)) return false;
-
-    return true;
+    if (!key.equals(that.key)) {
+      return false;
+    }
+    if (subFacets != null ? !subFacets.equals(that.subFacets) : that.subFacets != null) {
+      return false;
+    }
+    return value.equals(that.value);
   }
 
   @Override

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/Hit.java b/server/sonar-server/src/main/java/org/sonar/server/search/Hit.java
index ef04a7ddd6b7c791a3cddbb7e00f0d82cf1014ad..d4e0e4c8647bb83f9f12b391f1ecda7ff4924e3d 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/search/Hit.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/Hit.java
@@ -66,25 +66,29 @@ public class Hit implements Comparable<Hit> {
 
   @Override
   public int compareTo(Hit hit) {
-    if (this.getRank() != null) {
-      return this.getRank().compareTo(hit.getRank());
-    } else {
-      return this.getScore().compareTo(hit.getScore());
+    if (getRank() != null) {
+      return getRank().compareTo(hit.getRank());
     }
+    return getScore().compareTo(hit.getScore());
   }
 
   @Override
   public boolean equals(Object o) {
-    if (this == o) return true;
-    if (!(o instanceof Hit)) return false;
+    if (this == o) {
+      return true;
+    }
+    if (!(o instanceof Hit)) {
+      return false;
+    }
 
     Hit hit = (Hit) o;
-
-    if (fields != null ? !fields.equals(hit.fields) : hit.fields != null) return false;
-    if (rank != null ? !rank.equals(hit.rank) : hit.rank != null) return false;
-    if (score != null ? !score.equals(hit.score) : hit.score != null) return false;
-
-    return true;
+    if (fields != null ? !fields.equals(hit.fields) : hit.fields != null) {
+      return false;
+    }
+    if (rank != null ? !rank.equals(hit.rank) : hit.rank != null) {
+      return false;
+    }
+    return !(score != null ? !score.equals(hit.score) : hit.score != null);
   }
 
   @Override

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteDto.java b/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteDto.java
index 8b801a45b7247d424175304349a551b5f6ae609f..73941663882aba68c46431bd7a32464a359df062 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteDto.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteDto.java
@@ -42,7 +42,7 @@ public class DeleteDto<DTO extends Dto> extends IndexActionRequest {
   }
 
   @Override
-  public Class<?> getPayloadClass() {
+  public Class getPayloadClass() {
     return dto.getClass();
   }
 

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteKey.java b/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteKey.java
index adc15a9a9a5ce54ec0adb2111e3b96d66dcee55c..772f34f90418a3be6d68b3686333a63f7a4737ab 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteKey.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteKey.java
@@ -42,7 +42,7 @@ public class DeleteKey<K extends Serializable> extends IndexActionRequest {
   }
 
   @Override
-  public Class<?> getPayloadClass() {
+  public Class getPayloadClass() {
     throw new IllegalStateException("Deletion by key does not have an object payload!");
   }
 

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteNestedItem.java b/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteNestedItem.java
index 15555a4481b27fe7d4f51fa5abdfd878fa09d1ac..e85dff4279a4d6c6e6f3042c250fee7a87a9dbaa 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteNestedItem.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/action/DeleteNestedItem.java
@@ -45,7 +45,7 @@ public class DeleteNestedItem<K extends Serializable> extends IndexActionRequest
   }
 
   @Override
-  public Class<?> getPayloadClass() {
+  public Class getPayloadClass() {
     return item.getClass();
   }
 

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/action/IndexActionRequest.java b/server/sonar-server/src/main/java/org/sonar/server/search/action/IndexActionRequest.java
index 2d36fa0d5dc68459659c7b2c0570665bd0de134e..8eb540df4a62331eb3ef60e95aa2cc639bf5184a 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/search/action/IndexActionRequest.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/action/IndexActionRequest.java
@@ -46,7 +46,7 @@ public abstract class IndexActionRequest implements ClusterAction<List<ActionReq
 
   public abstract String getKey();
 
-  public abstract Class<?> getPayloadClass();
+  public abstract Class getPayloadClass();
 
   public String getIndexType() {
     return indexType;

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/action/UpsertDto.java b/server/sonar-server/src/main/java/org/sonar/server/search/action/UpsertDto.java
index 5ad7af6c3f53598c9a49aa4a1297c9a184f8664a..c2c6b5c0e65edc2cdd2a3ad133d6f2f942e354d9 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/search/action/UpsertDto.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/action/UpsertDto.java
@@ -44,7 +44,7 @@ public class UpsertDto<DTO extends Dto> extends IndexActionRequest {
   }
 
   @Override
-  public Class<?> getPayloadClass() {
+  public Class getPayloadClass() {
     return dto.getClass();
   }
 

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/action/UpsertNestedItem.java b/server/sonar-server/src/main/java/org/sonar/server/search/action/UpsertNestedItem.java
index e87fe604e86bb1726966b73285d4fdc9a55a794d..a16ba90bfa66199f7a593e625fb7fb8c7deae7ff 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/search/action/UpsertNestedItem.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/action/UpsertNestedItem.java
@@ -46,7 +46,7 @@ public class UpsertNestedItem<K extends Serializable> extends IndexActionRequest
   }
 
   @Override
-  public Class<?> getPayloadClass() {
+  public Class getPayloadClass() {
     return item.getClass();
   }