You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by ry...@apache.org on 2007/11/29 16:26:06 UTC

svn commit: r599487 - /lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/SolrQuery.java

Author: ryan
Date: Thu Nov 29 07:26:04 2007
New Revision: 599487

URL: http://svn.apache.org/viewvc?rev=599487&view=rev
Log:
removing the setFacetMinCount() from addFacetField().  see:
http://www.nabble.com/small-bug-in-solrj--resultquery.addFacetField-overwrites-resultquery.setFacetMinCount-to-1.-tf4895967.html

Also fixing the setFields() issue

Modified:
    lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/SolrQuery.java

Modified: lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/SolrQuery.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/SolrQuery.java?rev=599487&r1=599486&r2=599487&view=diff
==============================================================================
--- lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/SolrQuery.java (original)
+++ lucene/solr/trunk/client/java/solrj/src/org/apache/solr/client/solrj/SolrQuery.java Thu Nov 29 07:26:04 2007
@@ -56,10 +56,11 @@
    * 
    * @param f the field name from the IndexSchema
    */
-  public void addFacetField(String f) {
-    this.add(FacetParams.FACET_FIELD, f);
+  public void addFacetField(String ... fields) {
+    for( String f : fields ) {
+      this.add(FacetParams.FACET_FIELD, f);
+    }
     this.set(FacetParams.FACET, true);
-    this.setFacetMinCount(1);
   }
 
   /** get the facet fields
@@ -325,7 +326,16 @@
   }
 
   public void setFields(String ... fields) {
-    this.set(CommonParams.FL, fields);
+    if( fields == null || fields.length == 0 ) {
+      this.remove( CommonParams.FL );
+    }
+    StringBuilder sb = new StringBuilder();
+    sb.append( fields[0] );
+    for( int i=1; i<fields.length; i++ ) {
+      sb.append( ',' );
+      sb.append( fields[i] );
+    }
+    this.set(CommonParams.FL, sb.toString() );
   }
     
   public void addField(String field) {