You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by sh...@apache.org on 2009/04/28 14:16:23 UTC

svn commit: r769357 - in /lucene/solr/trunk/src: java/org/apache/solr/handler/component/SpellCheckComponent.java test/org/apache/solr/handler/component/SpellCheckComponentTest.java

Author: shalin
Date: Tue Apr 28 12:16:22 2009
New Revision: 769357

URL: http://svn.apache.org/viewvc?rev=769357&view=rev
Log:
SOLR-1094 followup -- Fix for single correctly spelled word

Modified:
    lucene/solr/trunk/src/java/org/apache/solr/handler/component/SpellCheckComponent.java
    lucene/solr/trunk/src/test/org/apache/solr/handler/component/SpellCheckComponentTest.java

Modified: lucene/solr/trunk/src/java/org/apache/solr/handler/component/SpellCheckComponent.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/handler/component/SpellCheckComponent.java?rev=769357&r1=769356&r2=769357&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/handler/component/SpellCheckComponent.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/handler/component/SpellCheckComponent.java Tue Apr 28 12:16:22 2009
@@ -227,6 +227,8 @@
     }
     if (hasFreqInfo) {
       result.add("correctlySpelled", isCorrectlySpelled);
+    } else if(extendedResults && suggestions.size() == 0) { // if the word is misspelled, its added to suggestions with freqinfo
+      result.add("correctlySpelled", true);
     }
     if (collate == true){
       StringBuilder collation = new StringBuilder(origQuery);

Modified: lucene/solr/trunk/src/test/org/apache/solr/handler/component/SpellCheckComponentTest.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/test/org/apache/solr/handler/component/SpellCheckComponentTest.java?rev=769357&r1=769356&r2=769357&view=diff
==============================================================================
--- lucene/solr/trunk/src/test/org/apache/solr/handler/component/SpellCheckComponentTest.java (original)
+++ lucene/solr/trunk/src/test/org/apache/solr/handler/component/SpellCheckComponentTest.java Tue Apr 28 12:16:22 2009
@@ -271,6 +271,19 @@
     
     assertQ("Make sure correct spellings are signalled in the response", req, 
         "//*[@numFound='0']", "//*/lst[@name='suggestions']", "//*/bool[@name='correctlySpelled'][.='false']");
+    
+    
+    args = new HashMap<String, String>();
+
+    args.put(CommonParams.Q, "lowerfilt:lazy");
+    args.put(CommonParams.QT, "spellCheckCompRH");
+    args.put(SpellCheckComponent.SPELLCHECK_EXTENDED_RESULTS, "true");
+    args.put(SpellCheckComponent.COMPONENT_NAME, "true");
+    req = new LocalSolrQueryRequest(core, new MapSolrParams(
+            args));
+    
+    assertQ("Make sure correct spellings are signalled in the response", req, 
+        "//*[@numFound='1']", "//*/lst[@name='suggestions']", "//*/bool[@name='correctlySpelled'][.='true']");
   }
 
   public void testInit() throws Exception {