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) {