Download & Extend
Search 404 » Issues

Use Search Lucene Content when available

Posted by wimh on October 13, 2009 at 10:04pm
Project:Search 404
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:zyxware
Status:closed (fixed)

Issue Summary

Just like it already does with Apache Solr, this module should use Search Lucene API/Content to search for nodes when available.

Login or register to post comments

Comments

#1

Posted by wimh on October 13, 2009 at 10:05pm
Status:active» needs review

Patch attached.

AttachmentSize
search404.module.patch 878 bytes

#2

Posted by mfer on November 25, 2009 at 3:52pm

any movement on this?

#3

Posted by wimh on November 30, 2009 at 3:13pm

I didn't hear any response from the module maintainers yet... But my patch is working for some time now on my production site, so go ahead and try it out!

#4

Posted by Ela on January 25, 2010 at 10:26pm

Works great!!! Thank you for the patch!!! :)

#5

Posted by bunker on February 19, 2010 at 11:53pm

this module sounds great, I'm using lucene api for search

when I try to patch I get the following error:

$ sudo patch < search404.module_4.patch
(Stripping trailing CRs from patch.)
patching file search404.module
Hunk #1 FAILED at 102.
1 out of 1 hunk FAILED -- saving rejects to file search404.module.rej

what should I do?

#6

Posted by wimh on February 22, 2010 at 3:27pm

That patch is for the 6.x-1.8 version. Are you trying to apply it to the Drupal 5 version of the module by any chance? In any case, you can open the patch file with a text editor, have a look at what replacements it's supposed to do, and make the same changes by hand.

#7

Posted by J0nathan on March 15, 2010 at 11:45pm

Hello and thank you for the module.
Is there any updates about the integration of the patch into the module?

#8

Posted by WorldFallz on March 19, 2010 at 7:00pm

Here's an updated patch against the current 6.x dev version.

AttachmentSize
search404_luceneapi.patch 1.04 KB

#9

Posted by fragger_fox on March 22, 2010 at 8:30am

Thank you for the patch I will apply this in the latest dev version of 6.x

Regards
Santhosh N.Raju

#10

Posted by zyxware on March 22, 2010 at 9:37am
Assigned to:Anonymous» zyxware

I have updated the dev version of Search404 with this patch

Let me know if things are fine.

Regards
Santhosh Raju

#11

Posted by J0nathan on March 22, 2010 at 6:53pm

Hi,
Version 6.x-1.x-dev (2010-Mar-22 ) gives me results with Lucene.
Thank you very much.

#12

Posted by zyxware on March 23, 2010 at 6:16am
Status:needs review» fixed

Thanks for the update J0nathan

I am marking this issue as fixed

Regards
Santhosh Raju

#13

Posted by WorldFallz on March 23, 2010 at 1:51pm

Yep I just updated and it's working fine. thanks for committing this!

#14

Posted by System Message on April 6, 2010 at 2:00pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

#15

Posted by mrgavindb on May 18, 2010 at 1:37am

Great fix! Works for me.

EDIT: I just noticed that version 1.9 offers this feature without making any code changes.

#16

Posted by nicoz on January 11, 2011 at 12:46am

have I missed something? I have both installed and I do not see the option for using Lucene with this module. Or does it automatically use it?

#17

Posted by wimh on January 11, 2011 at 10:32am

have I missed something? I have both installed and I do not see the option for using Lucene with this module. Or does it automatically use it?

There is no configuration setting, it should just use Lucene automatically. (Unless you also have Apache SOLR installed, then that one takes precedence).

#18

Posted by nicoz on January 11, 2011 at 8:14pm

hmm. I don't have Solr installed, but the search through a 404 doesn't reveal the same results as a general search (using the same terms)

nobody click here