The search API pages integration does not work for me after updating to alpha 3. I am able to configure it but the search pages block does not contain the auto complete functionality like it did in the alpha 1 and alpha 2 version. I have checked and confirmed that all of the settings are correct.

The search view integration does work as expected.

CommentFileSizeAuthor
#6 2903830-pages-integration-broken.patch1013 bytesdavy-r
#3 2903830-3--fix_pages_integration.patch1.18 KBdrunken monkey
#2 search_api_autocomplete-fix_support_for_pages-2903830-2.patch719 byteschrissnyder

Comments

chrissnyder created an issue. See original summary.

chrissnyder’s picture

chrissnyder
he/him
commented
Status: Active » Needs review
StatusFileSize
new search_api_autocomplete-fix_support_for_pages-2903830-2.patch719 bytes

Found the issue can be solved by accounting for the new ID format in the block form alter.

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Component: Code » General code
StatusFileSize
new 2903830-3--fix_pages_integration.patch1.18 KB

Thanks for reporting this problem!
Your suggested solution might work in most cases, but it's not completely correct (since that ID is not guaranteed). Instead, we should do the same as for the Views integration – see the attached patch (and please test!).
Sorry for not thinking about this during the change.

davy-r’s picture

davy-r commented

The latest patch fails with: "Error: Call to undefined method Drupal\search_api_autocomplete\Entity\SearchStorage::loadByType() in search_api_autocomplete_form_search_api_page_block_form_alter() (line 157 of modules/contrib/search_api_autocomplete/search_api_autocomplete.module)."

davy-r’s picture

davy-r commented
Status: Needs review » Needs work
davy-r’s picture

davy-r commented
Status: Needs work » Needs review
StatusFileSize
new 2903830-pages-integration-broken.patch1013 bytes
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Needs review » Fixed

Ah, right, that changed again …
Thanks a lot, looks great. Committed.
Thanks again, everyone!

  • drunken monkey committed 8803c7e on 8.x-1.x 
    Issue #2903830 by drunken monkey, chrissnyder, davy-r: Fixed integration...

Status: Fixed » Closed (fixed)

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