Closed (fixed)
Project:
Search API Algolia
Version:
8.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
7 Jan 2020 at 06:01 UTC
Updated:
30 Jan 2020 at 16:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
miteshmapComment #4
nikunjkotechaComment #5
jrockowitz commentedI am reopening this issue because I am getting the below error which is related to this change. The website that is throwing this error is multilingual with the default language being English. I think the issue is that my existing Algolia indexes are NOT multiple languages. The error seems to be triggered because {MY_INDEX}_en does not exist and {MY_INDEX} does exist.
I am marking this Critical because it is throwing a fatal error. The immediate solution is to downgrade to 1.0-alpha2.
Comment #6
nikunjkotechaHi Jacob,
Thanks for the details.
Question: How does your search / indexation work if you are not using multi-lingual approach like described in readme?
The approach implemented is after discussion with Algolia team and would be great if you can help with bit more info around the approach you use to generalise the code in module and implement hooks / events for custom work.
Thanks,
Nikunj
Comment #7
jrockowitz commentedWe currently have no multilingual search indexes set up. Algolia and Search API are just defaulting to English.
I am not seeing any multilingual approach being document in the README.txt.
When I review the recent changes there is an assumption that if Drupal is multilingual then all of related the Algolia indexes are multilingual. Without diving deeper into the code, it seems like there needs to be a method that checks if Algolia has corresponding multilingual indexes.
We are upgrading from 8.x-1.0-alpha2 below are all the changes since alpha2. I think #3053205: Support viewing results via views, support facets and autocomplete is causing this regression.
Changes since 8.x-1.0-alpha2:
Comment #8
jrockowitz commentedBTW, now that I have investigated this issue further. This is probably the wrong ticket and a new ticket needs to be created for this regression. Do you want me to create a new ticket?
Comment #9
nikunjkotechaYes, I suspected that and that's why I asked for more details
And my bad, I added that to INSTALL
https://git.drupalcode.org/project/search_api_algolia/blob/8.x-1.x/INSTA...
I think yes, we need a new ticket to make language suffix a configuration instead of simply assuming that's the best approach.
Comment #10
jrockowitz commented@see #3106901: Make language suffix a configuration