You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ry...@apache.org on 2010/12/29 10:38:13 UTC

svn commit: r1053583 - in /lucene/dev/branches/branch_3x/solr: ./ src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java

Author: ryan
Date: Wed Dec 29 09:38:12 2010
New Revision: 1053583

URL: http://svn.apache.org/viewvc?rev=1053583&view=rev
Log:
SOLR-2302 -- only show an error if neither a field name nor a field type are sent (merged from trunk)

Modified:
    lucene/dev/branches/branch_3x/solr/   (props changed)
    lucene/dev/branches/branch_3x/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java

Modified: lucene/dev/branches/branch_3x/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java?rev=1053583&r1=1053582&r2=1053583&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java (original)
+++ lucene/dev/branches/branch_3x/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java Wed Dec 29 09:38:12 2010
@@ -96,8 +96,8 @@ public class FieldAnalysisRequest extend
    */
   @Override
   public FieldAnalysisResponse process(SolrServer server) throws SolrServerException, IOException {
-    if (fieldTypes == null || fieldNames == null) {
-      throw new IllegalStateException("A list one field type or field name need to be specified");
+    if (fieldTypes == null && fieldNames == null) {
+      throw new IllegalStateException("At least one field type or field name need to be specified");
     }
     if (fieldValue == null) {
       throw new IllegalStateException("The field value must be set");