ACE-3290: Added date facet fields in their own bean, so it can be overridden, rather than the full date display handler bean.

git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@89637 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261

config/alfresco/solr-facets-context.xml

@@ -107,13 +107,19 @@
       </constructor-arg>
    </bean>
 
-   <bean id="facet.dateBucketsDisplayHandler" class="org.alfresco.repo.search.impl.solr.facet.handler.DateBucketsDisplayHandler" parent="baseFacetLabelDisplayHandler" >
+   <bean id="facet.dateFacetFields" class="org.alfresco.repo.search.impl.solr.facet.FacetFieldBean">
-      <constructor-arg index="0">
+      <constructor-arg>
          <set>
             <value>@{http://www.alfresco.org/model/content/1.0}created</value>
             <value>@{http://www.alfresco.org/model/content/1.0}modified</value>
          </set>
       </constructor-arg>
+   </bean>
+
+   <bean id="facet.dateBucketsDisplayHandler" class="org.alfresco.repo.search.impl.solr.facet.handler.DateBucketsDisplayHandler" parent="baseFacetLabelDisplayHandler" >
+      <constructor-arg index="0">
+         <ref bean="facet.dateFacetFields" />
+      </constructor-arg>
       <constructor-arg index="1">
          <bean class="org.springframework.beans.factory.config.MapFactoryBean">
             <property name="targetMapClass">

source/java/org/alfresco/repo/search/impl/solr/facet/FacetFieldBean.java

@@ -0,0 +1,48 @@
+/*
+ * Copyright (C) 2005-2014 Alfresco Software Limited.
+ *
+ * This file is part of Alfresco
+ *
+ * Alfresco is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Lesser General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Alfresco is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public License
+ * along with Alfresco. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+package org.alfresco.repo.search.impl.solr.facet;
+
+import java.util.Collections;
+import java.util.HashSet;
+import java.util.Set;
+
+import org.springframework.extensions.surf.util.ParameterCheck;
+
+/**
+ * A simple bean class to provide facet fields.
+ * 
+ * @author Jamal Kaabi-Mofrad
+ * @since 5.0
+ */
+public class FacetFieldBean
+{
+    private final Set<String> fields;
+
+    public FacetFieldBean(Set<String> fields)
+    {
+        ParameterCheck.mandatory("fields", fields);
+        this.fields = new HashSet<>(fields);
+    }
+
+    public Set<String> getFields()
+    {
+        return Collections.unmodifiableSet(this.fields);
+    }
+}

source/java/org/alfresco/repo/search/impl/solr/facet/SolrFacetHelper.java

@@ -51,7 +51,7 @@ public class SolrFacetHelper
     /**
      * Constructor
      * 
-     * @param serviceRegistry
+     * @param facetQueryProviders
      */
     public SolrFacetHelper(List<FacetQueryProvider> facetQueryProviders)
     {

source/java/org/alfresco/repo/search/impl/solr/facet/handler/AbstractFacetLabelDisplayHandler.java

@@ -80,7 +80,7 @@ public abstract class AbstractFacetLabelDisplayHandler implements FacetLabelDisp
     /**
      * Set the registry to register with
      * 
-     * @param registry a metadata extracter registry
+     * @param registry a facet label display handler registry
      */
     public void setRegistry(FacetLabelDisplayHandlerRegistry registry)
     {

source/java/org/alfresco/repo/search/impl/solr/facet/handler/DateBucketsDisplayHandler.java

@@ -30,6 +30,7 @@ import java.util.Map.Entry;
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
+import org.alfresco.repo.search.impl.solr.facet.FacetFieldBean;
 import org.alfresco.repo.search.impl.solr.facet.FacetQueryProvider;
 import org.alfresco.repo.search.impl.solr.facet.SolrFacetConfigException;
 import org.springframework.extensions.surf.util.ParameterCheck;
@@ -52,7 +53,7 @@ public class DateBucketsDisplayHandler extends AbstractFacetLabelDisplayHandler
         ParameterCheck.mandatory("facetQueryFields", facetQueryFields);
         ParameterCheck.mandatory("dateBucketsMap", dateBucketsMap);
 
-        this.supportedFieldFacets = Collections.unmodifiableSet(facetQueryFields);
+        this.supportedFieldFacets = facetQueryFields;
 
         facetLabelMap = new HashMap<>(dateBucketsMap.size());
         Map<String, List<String>> facetQueries = new LinkedHashMap<>(facetQueryFields.size());
@@ -84,6 +85,11 @@ public class DateBucketsDisplayHandler extends AbstractFacetLabelDisplayHandler
         this.facetQueriesMap = Collections.unmodifiableMap(facetQueries);
     }
 
+    public DateBucketsDisplayHandler(FacetFieldBean dateFacetField, LinkedHashMap<String, String> dateBucketsMap)
+    {
+        this(dateFacetField.getFields(), dateBucketsMap);
+    }
+
     @Override
     public FacetLabel getDisplayLabel(String value)
     {
@@ -96,5 +102,4 @@ public class DateBucketsDisplayHandler extends AbstractFacetLabelDisplayHandler
     {
         return this.facetQueriesMap;
     }
-
 }