Author: ryan
Date: Mon Jan  7 18:57:08 2008
New Revision: 609854

URL: http://svn.apache.org/viewvc?rev=609854&view=rev
Log:
let SolrQuery take null start/rows to remove them  (rather then throw NPE)

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=609854&r1=609853&r2=609854&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
 Mon Jan  7 18:57:08 2008
@@ -372,7 +372,12 @@
   }
 
   public void setRows(Integer rows) {
-    this.set(CommonParams.ROWS, rows);
+    if( rows == null ) {
+      this.remove( CommonParams.ROWS );
+    }
+    else {
+      this.set(CommonParams.ROWS, rows);
+    }
   }
 
   public Integer getRows()
@@ -390,7 +395,12 @@
 //  }
 
   public void setStart(Integer start) {
-    this.set(CommonParams.START, start);
+    if( start == null ) {
+      this.remove( CommonParams.START );
+    }
+    else {
+      this.set(CommonParams.START, start);
+    }
   }
   
   public Integer getStart()


Reply via email to