hi

is it possible to use http://drupal.org/project/fuzzysearch instead of normal search?

thanks

CommentFileSizeAuthor
#12 add-fuzzysearch-838054.patch2.97 KBawolfey

Comments

awolfey’s picture

I can provide a patch for this if you're interested. I currently have a 404 takeover in the dev version of fuzzysearch, but I think this is a better place for it.

Because fuzzy doesn't use the core search hooks, most of the patch woule lie out side of your main if statement in search404_page().

Aaron

akaserer’s picture

wow, that sounds great.

i think its a great help if fuzzy search would take over 404.
it would be usefull to be able to add an admin-setting if fuzzy search should take over 404.

thanks

awolfey’s picture

Is the maintainer interested? If zyxware want's it I can do it quickly.

riversidekid’s picture

Subscribing!

(Already using (and loving) fuzzy search, just not ready to install a .dev module.)

Sepero’s picture

Priority: Normal » Major

Also subscribing

Fuzzy search is a million times better than the worthless default search. I'm subscribing in the hopes that these two modules can work together. Searching with default core search is pathetic.

akaserer’s picture

@awolfey :

i think it would be great if you can provide a patch.
seems like many people would need this function.

thanks

zyxware’s picture

@awolfey - If you could put the patch across I could set 404 to use fuzzysearch

awolfey’s picture

Assigned: Unassigned » awolfey

zyxware, sounds good. I'll put it on my list.

zyxware’s picture

Project: Search 404 » Fuzzy Search
Version: 6.x-1.9 » 6.x-1.x-dev
Component: User interface » Code

@awolfey - Are you still interested in doing this?

zyxware’s picture

@awolfey - Please assign this issue back to search404 with your answer.

awolfey’s picture

Project: Fuzzy Search » Search 404

My apologies, I have become very busy with work. I do still have this on my list to do. I'm also willing to help anyone else who wants to do it.

awolfey’s picture

Status: Active » Needs review
StatusFileSize
new2.97 KB

Here's a patch. A couple of issues to deal with eventually:

1. search404 requires core search module, but people using fuzzysearch probably aren't using core search. Maybe there needs to be some kind of check that ensures at least one search module is enabled, without dependencies in the .info file.

2. Because of the above, there is some small amount of duplicated code in the patch.

3. I did this the easy way :), so jump to first or only result only works with core search.

WickedJ’s picture

Subscribing

zyxware’s picture

Status: Needs review » Fixed

Committed to latest dev version.

@awolfey - Points noted.
Is there a way to invoke

$results = module_invoke($type_search, 'search', 'search', $keys);

for fuzzy search to get results? If so we could implement item 3 as well.

awolfey’s picture

There isn't now, although I've planned something like that when I get time. I'll update here, but don't hold your breath.

Status: Fixed » Closed (fixed)

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