Bug in Acquia Search module
Admin page url: /admin/config/search/search-api/server/acquia_search_server

Error log: Error: Call to undefined function theme_search_api_server() in acquia_search_theme_search_api_server() (line 663 of

Using:
Drupal 8.8.6
Acquia connector: 8.x-1.21
Search API: 8.x-1.17

CommentFileSizeAuthor
#7 acquia_connector-fix-search-api-theming-changes-3145134-7.patch2.03 KBwells
#6 acquia_connector-fix-search-api-theming-changes-3145134-6.patch2.1 KBwells
#5 acquia_connector-fix-search-api-theming-changes-3145134-5.patch1.68 KBShawnRong
#3 acquia_connector-fix-search-api-theming-changes-3145134-3.patch4.82 KBjanusman
Screenshot 2020-06-03 at 12.33.55 PM.png16.47 KBgaurav_manerkar

Comments

gauravmanerkar created an issue. See original summary.

gaurav_manerkar’s picture

gaurav_manerkar commented

Search API 1.17 module has removed all theme function.
So please update module accordingly.
Thanks

janusman’s picture

janusman commented
StatusFileSize
new acquia_connector-fix-search-api-theming-changes-3145134-3.patch4.82 KB

Patch for review.

janusman’s picture

janusman commented
Status: Active » Needs review
ShawnRong’s picture

ShawnRong commented
StatusFileSize
new acquia_connector-fix-search-api-theming-changes-3145134-5.patch1.68 KB

According to the Search API Theme hook update.
https://www.drupal.org/node/3138575
Replace "theme_search_api_index" with "template_preprocess_search_api_index" , "theme_search_api_server" with "template_preprocess_search_api_server"

wells’s picture

wells
he/him/his
Primary language English
Location Seattle, WA
commented
StatusFileSize
new acquia_connector-fix-search-api-theming-changes-3145134-6.patch2.1 KB

Patch in #5 overwrites the output from Search API so the actual server and index info is missing.

It looks like Acquia is already working on this (see Upgrading Search API causes "Call to undefined function theme_search_api_server()" error) but I am attaching a temporary fix patch that will work for now. This likely won't be the best way to address this change.

wells’s picture

wells
he/him/his
Primary language English
Location Seattle, WA
commented
StatusFileSize
new acquia_connector-fix-search-api-theming-changes-3145134-7.patch2.03 KB

Scratch #6 patch, PHPStorm mangled my line endings (:

Updated patch attached.

ankit agrawal’s picture

ankit agrawal commented

Reverting the Search API module to 1.16 as per https://support.acquia.com/hc/en-us/articles/360048786374-Upgrading-Sear... fixed the issue.
Acquia search going to have the new release which should be compatible with Search API 1.17.

gaurav_manerkar’s picture

gaurav_manerkar commented

Thanks everyone.
I am going to close this issue as new version of Acquia Connector module is already released by Acquia which is compatible with Search API v1.17

gaurav_manerkar’s picture

gaurav_manerkar commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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