Terms query is broken, closes #80.

Author: kimchy

modules/elasticsearch/src/main/java/org/elasticsearch/action/terms/TermsRequest.java

@@ -180,7 +180,11 @@ public String from() {
      * first.
      */
     public TermsRequest from(Object from) {
-        this.from = from.toString();
+        if (from == null) {
+            this.from = null;
+        } else {
+            this.from = from.toString();
+        }
         return this;
     }
 
@@ -202,17 +206,21 @@ public TermsRequest fromInclusive(boolean fromInclusive) {
     }
 
     /**
-     * The upper bound (lex) term to which the iteration will end. Defaults to unbound (<tt>null</tt>).
+     * The upper bound term to which the iteration will end. Defaults to unbound (<tt>null</tt>).
      */
     public String to() {
         return to;
     }
 
     /**
-     * The upper bound (lex) term to which the iteration will end. Defaults to unbound (<tt>null</tt>).
+     * The upper bound term to which the iteration will end. Defaults to unbound (<tt>null</tt>).
      */
     public TermsRequest to(Object to) {
-        this.to = to.toString();
+        if (to == null) {
+            this.to = null;
+        } else {
+            this.to = to.toString();
+        }
         return this;
     }
 

modules/elasticsearch/src/main/java/org/elasticsearch/rest/action/terms/RestTermsAction.java

@@ -72,13 +72,13 @@ public class RestTermsAction extends BaseRestHandler {
             termsRequest.operationThreading(operationThreading);
 
             List<String> fields = request.params("field");
-            if (fields == null) {
-                fields = new ArrayList<String>();
-            }
             String sField = request.param("fields");
             if (sField != null) {
                 String[] sFields = fieldsPattern.split(sField);
                 if (sFields != null) {
+                    if (fields == null) {
+                        fields = new ArrayList<String>();
+                    }
                     for (String field : sFields) {
                         fields.add(field);
                     }