You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ds...@apache.org on 2012/03/24 06:06:48 UTC
svn commit: r1304704 - in /lucene/dev/trunk/solr/core/src:
java/org/apache/solr/search/ExtendedDismaxQParserPlugin.java
test/org/apache/solr/search/TestExtendedDismaxParser.java
Author: dsmiley
Date: Sat Mar 24 05:06:48 2012
New Revision: 1304704
URL: http://svn.apache.org/viewvc?rev=1304704&view=rev
Log:
SOLR-3270 edismax blank q bug
Modified:
lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/ExtendedDismaxQParserPlugin.java
lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestExtendedDismaxParser.java
Modified: lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/ExtendedDismaxQParserPlugin.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/ExtendedDismaxQParserPlugin.java?rev=1304704&r1=1304703&r2=1304704&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/ExtendedDismaxQParserPlugin.java (original)
+++ lucene/dev/trunk/solr/core/src/java/org/apache/solr/search/ExtendedDismaxQParserPlugin.java Sat Mar 24 05:06:48 2012
@@ -163,7 +163,7 @@ class ExtendedDismaxQParser extends QPar
parsedUserQuery = null;
String userQuery = getString();
altUserQuery = null;
- if( userQuery == null || userQuery.length() < 1 ) {
+ if( userQuery == null || userQuery.trim().length() == 0 ) {
// If no query is specified, we may have an alternate
String altQ = solrParams.get( DisMaxParams.ALTQ );
if (altQ != null) {
Modified: lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestExtendedDismaxParser.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestExtendedDismaxParser.java?rev=1304704&r1=1304703&r2=1304704&view=diff
==============================================================================
--- lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestExtendedDismaxParser.java (original)
+++ lucene/dev/trunk/solr/core/src/test/org/apache/solr/search/TestExtendedDismaxParser.java Sat Mar 24 05:06:48 2012
@@ -71,8 +71,14 @@ public class TestExtendedDismaxParser ex
String oner = "*[count(//doc)=1]";
String twor = "*[count(//doc)=2]";
String nor = "*[count(//doc)=0]";
-
- assertQ("expected doc is missing (using un-escaped edismax w/qf)",
+
+ assertQ("blank q",
+ req("q"," ",
+ "q.alt",allq,
+ "defType","edismax")
+ ,allr);
+
+ assertQ("expected doc is missing (using un-escaped edismax w/qf)",
req("q", "literal:colon",
"qf", "t_special",
"defType", "edismax"),