You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sh...@apache.org on 2014/03/25 06:05:03 UTC
svn commit: r1581234 - in /lucene/dev/trunk/solr: CHANGES.txt
core/src/java/org/apache/solr/search/CollapsingQParserPlugin.java
core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
Author: shalin
Date: Tue Mar 25 05:05:02 2014
New Revision: 1581234
URL: http://svn.apache.org/r1581234
Log:
SOLR-5905: CollapsingQParserPlugin throws a NPE if required 'field' param is missing
Modified:
lucene/dev/trunk/solr/CHANGES.txt
lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/CollapsingQParserPlugin.java
lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
Modified: lucene/dev/trunk/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/CHANGES.txt?rev=1581234&r1=1581233&r2=1581234&view=diff
==============================================================================
--- lucene/dev/trunk/solr/CHANGES.txt (original)
+++ lucene/dev/trunk/solr/CHANGES.txt Tue Mar 25 05:05:02 2014
@@ -330,6 +330,9 @@ Bug Fixes
* SOLR-5899: CloudSolrServer's RouteResponse and RouteException should be
publicly accessible. (Mark Miller, shalin)
+* SOLR-5905: CollapsingQParserPlugin throws a NPE if required 'field' param is missing.
+ (Spyros Kapnissis via shalin)
+
Other Changes
---------------------
Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/CollapsingQParserPlugin.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/CollapsingQParserPlugin.java?rev=1581234&r1=1581233&r2=1581234&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/CollapsingQParserPlugin.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/CollapsingQParserPlugin.java Tue Mar 25 05:05:02 2014
@@ -211,6 +211,9 @@ public class CollapsingQParserPlugin ext
public CollapsingPostFilter(SolrParams localParams, SolrParams params, SolrQueryRequest request) throws IOException {
this.field = localParams.get("field");
+ if (this.field == null) {
+ throw new IllegalStateException("Required 'field' param is missing.");
+ }
this.max = localParams.get("max");
this.min = localParams.get("min");
if(this.min != null || this.max != null) {
Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java?rev=1581234&r1=1581233&r2=1581234&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestCollapseQParserPlugin.java Tue Mar 25 05:05:02 2014
@@ -18,6 +18,7 @@
package org.apache.solr.search;
import org.apache.solr.SolrTestCaseJ4;
+import org.apache.solr.common.SolrException;
import org.apache.solr.common.params.ModifiableSolrParams;
import org.junit.Before;
import org.junit.BeforeClass;
@@ -296,4 +297,14 @@ public class TestCollapseQParserPlugin e
assertQ(req(params), "*[count(//doc)=2]");
}
+
+ @Test
+ public void testMissingFieldParam() throws Exception {
+ ModifiableSolrParams params = new ModifiableSolrParams();
+ params.add("q", "*:*");
+ params.add("fq", "{!collapse}");
+ assertQEx("It should respond with a bad request when the 'field' param is missing", req(params),
+ SolrException.ErrorCode.BAD_REQUEST);
+ }
+
}