Does this module support Search API-based searches in D7?

If not, will it be in the future?

CommentFileSizeAuthor
#19 0001-Issue-1076190-by-Ga-lG-Search-API-integration.patch10.35 KBGaëlG
#17 0001-Use-Google-geocoding-components-filter-for-better-re.patch2.61 KBGaëlG
#16 search_api404.tar_.gz810 bytesezeedub
#8 custom-path-404.1076190.patch2.1 KBBlackice2999
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

zyxware’s picture

zyxware CreditAttribution: zyxware commented

Can you please clarify? What exactly are you looking for?

Shadlington’s picture

Shadlington CreditAttribution: Shadlington commented

I see a bunch of search modules listed as being supported, but not the Search API module.
That's pretty much it really - I love Search API and would like to use it for Search 404 rather than any of the alternatives.

pixelsweatshop’s picture

pixelsweatshop CreditAttribution: pixelsweatshop commented

sub +1 for this feature.

pixelsweatshop’s picture

pixelsweatshop CreditAttribution: pixelsweatshop commented

Search_api maintainers response http://drupal.org/node/1176820

nigelw’s picture

nigelw CreditAttribution: nigelw commented

Looking forward to this feature as well! :) Keep up the good work!

yareckon’s picture

yareckon CreditAttribution: yareckon commented

subscribe

zyxware’s picture

zyxware CreditAttribution: zyxware commented

If somebody can come up with a patch for doing this I can integrate it to the module.

Blackice2999’s picture

Blackice2999 CreditAttribution: Blackice2999 commented
FileSize
custom-path-404.1076190.patch2.1 KB

Hi @all,

as a small and fast solution i have added a custom path url option. You can enter a path like "search/@keys" and we got a redirect to this path. It works fine with search_api and views page because its not depending on this modules. Sure there is no more logic behind that but it can be a fast and small solution.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

I successfully use the patch from #8 with Search API Pages. Doesn't necessarily require a view. Thanks!

pixelsweatshop’s picture

pixelsweatshop CreditAttribution: pixelsweatshop commented
Title: Search API integration » Search API integration (patch)
Status: Active » Reviewed & tested by the community

Patch works for me too. @zyxware maybe we can get this committed, unless you see an issue with this approach?

zyxware’s picture

zyxware CreditAttribution: zyxware commented
Status: Reviewed & tested by the community » Fixed

@Blackice2999 - Thanks for the patch. I have applied this on the 7.x-1.x branch. Please check and revert if there are issues.

zyxware’s picture

zyxware CreditAttribution: zyxware commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Fixed » Active
Issue tags: +needs backport to 6.x
zyxware’s picture

zyxware CreditAttribution: zyxware commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Active » Fixed

Applied the changes in 6.x-1.x as well.

Status: Fixed » Closed (fixed)

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

ezeedub’s picture

ezeedub CreditAttribution: ezeedub commented
Category: support » feature
Status: Closed (fixed) » Active

I'd like this integration to work without the dependency on core search, which I prefer to disable when using search_api.

Is that possible?

ezeedub’s picture

ezeedub CreditAttribution: ezeedub commented
Status: Active » Closed (fixed)
FileSize
search_api404.tar_.gz810 bytes

On second thought, that may be more work than it's worth.

I just created a pared down version of search404 for use with search_api that meets my current needs. I post it here in case it's of use to someone else.

GaëlG’s picture

GaëlG
he/him
Primary language French
Location Lille, France
CreditAttribution: GaëlG commented
Status: Closed (fixed) » Needs review
FileSize
0001-Use-Google-geocoding-components-filter-for-better-re.patch2.61 KB

Here's a patch.

jtbayly’s picture

jtbayly CreditAttribution: jtbayly commented
Status: Needs review » Closed (fixed)

I'm thinking the patch above was put on the wrong issue somehow. Changing status back.
-Joseph

GaëlG’s picture

GaëlG
he/him
Primary language French
Location Lille, France
CreditAttribution: GaëlG commented
Status: Closed (fixed) » Needs review
FileSize
0001-Issue-1076190-by-Ga-lG-Search-API-integration.patch10.35 KB

Sorry, actually it was the right issue but the wrong patch!

zyxware’s picture

zyxware CreditAttribution: zyxware commented
Status: Needs review » Postponed (maintainer needs more info)

@GaëlG - Thanks for the patch. Can you please clarify as to what this patch is for? The searchAPI integration patch has already gone into the code and is available in the dev version of the module. You should be able to get that in the upcoming release.

GaëlG’s picture

GaëlG
he/him
Primary language French
Location Lille, France
CreditAttribution: GaëlG commented
Status: Postponed (maintainer needs more info) » Needs review

@zyxware: I can't remember what I did exactly, but I think I patched against last dev. Anyway, there's currently a problem in the last dev because search404_page() checks that search module is enabled, which most of the time is false when Search API is used.
If I'm right, what the patch does is simply change the checks order to behave correctly when search module is disabled.

zyxware’s picture

zyxware CreditAttribution: zyxware commented

@GaëlG - Thanks for the reply. If it is about removing search module dependency there is some work going on in this ticket - #1733762: Remove search.module dependency. I will be taking out the dependency on the search module with the next release. If that was the only issue please close this issue and continue discussions at the above mentioned issue.

GaëlG’s picture

GaëlG
he/him
Primary language French
Location Lille, France
CreditAttribution: GaëlG commented
Status: Needs review » Closed (duplicate)

Allright.