From a3fd7c1ab67f8aad7f7a266de24132914eb3a992 Mon Sep 17 00:00:00 2001
From: Jean-Baptiste Lievremont <jean-baptiste.lievremont@sonarsource.com>
Date: Fri, 24 Oct 2014 16:56:43 +0200
Subject: [PATCH] Fix quality flaws

---
 .../src/main/java/org/sonar/server/search/BaseIndex.java     | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/server/sonar-server/src/main/java/org/sonar/server/search/BaseIndex.java b/server/sonar-server/src/main/java/org/sonar/server/search/BaseIndex.java
index 3fba52fdeb5..1d957cd9612 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/search/BaseIndex.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/search/BaseIndex.java
@@ -613,7 +613,7 @@ public abstract class BaseIndex<DOMAIN, DTO extends Dto<KEY>, KEY extends Serial
   }
 
   protected FilterAggregationBuilder buildTopFacetAggregation(String fieldName, String facetName, BoolFilterBuilder facetFilter) {
-    FilterAggregationBuilder facetTopAggregation = AggregationBuilders
+    return AggregationBuilders
       .filter(facetName + "_filter")
       .filter(facetFilter)
       .subAggregation(
@@ -622,12 +622,11 @@ public abstract class BaseIndex<DOMAIN, DTO extends Dto<KEY>, KEY extends Serial
           .order(Terms.Order.count(false))
           .size(10)
           .minDocCount(1));
-    return facetTopAggregation;
   }
 
   protected FilterAggregationBuilder addSelectedItemsToFacet(String fieldName, String facetName, FilterAggregationBuilder facetTopAggregation, String... selected) {
     if (selected.length > 0) {
-      facetTopAggregation = facetTopAggregation.subAggregation(
+      facetTopAggregation.subAggregation(
         AggregationBuilders.terms(facetName + "_selected")
           .field(fieldName)
           .include(Joiner.on('|').join(selected)));
-- 
2.39.5