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:28:31 UTC
svn commit: r1053578 -
/lucene/dev/trunk/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java
Author: ryan
Date: Wed Dec 29 09:28:31 2010
New Revision: 1053578
URL: http://svn.apache.org/viewvc?rev=1053578&view=rev
Log:
SOLR-2302 -- only show an error if neither a field name nor a field type are sent
Modified:
lucene/dev/trunk/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java
Modified: lucene/dev/trunk/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java?rev=1053578&r1=1053577&r2=1053578&view=diff
==============================================================================
--- lucene/dev/trunk/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java (original)
+++ lucene/dev/trunk/solr/src/solrj/org/apache/solr/client/solrj/request/FieldAnalysisRequest.java Wed Dec 29 09:28:31 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");