- Issue created by @hktang
- 🇸🇮Slovenia DeaOm
I tried recreating this issue on the clean Drupal install but can't. Not sure if this is actually a meili search issue or a search api issue. Did you try with a different server or on a clean install? Next option would also be to try it with dev version of module. Which version of Drupal are you using?
- 🇸🇮Slovenia bcizej
At first I could not reproduce this as well but then this line hit me
In the Javascript console, the following message shows up:
and I was able to reproduce the issue. I had for a while suspected that the issue might be related with our dependencies that are injected into the plugin and that it might be connected with
__sleep
and__wakeup
methods that de/serialize the plugin however we are testing on clean drupal instances and we only have Meilisearch plugin installed and as so the form is automatically set to select "Meilisearch" as backend plugin. Installing more modules that provide backend plugins makes this error reproducible.I have updated the steps to reproduce. Thanks for the report!
- 🇸🇮Slovenia bcizej
Also 🐛 LogicException: The database connection is not serializable. This probably means you are serializing an object that has an indirect reference to the database connection. Fixed is most likely the same issue that we had problems reproducing.
- last update
about 1 year ago Unable to generate test groups - @bcizej opened merge request.
- Status changed to Needs work
about 1 year ago 6:44pm 5 September 2023 - last update
about 1 year ago 102 pass, 1 fail - 🇯🇵Japan hktang
Thank you for picking this up. Yes, after I deleted other search API modules (db, solr, etc), the Meilisearch config is visible by default and saves correctly.
- last update
about 1 year ago 106 pass - Status changed to Needs review
about 1 year ago 6:30am 6 September 2023 - 🇸🇮Slovenia bcizej
Rebased the branch and added a fix. The api service was not being removed from the plugin before serialization which caused the error.
- last update
about 1 year ago 106 pass - last update
about 1 year ago 106 pass - Status changed to RTBC
about 1 year ago 8:30am 6 September 2023 - 🇸🇮Slovenia DeaOm
With the updated steps to reproduce the issue, I was able to reproduce it. The correction from @bcizej is working as expected and it seems DB plugin has something similar implemented. I did correct 2 small CS issues (not from this issue) as did not want to open another ticket just for that. The tests are also passing so marking it as RTBC.
- last update
about 1 year ago 107 pass - Status changed to Fixed
about 1 year ago 2:40pm 6 September 2023 - 🇯🇵Japan hktang
Thank you for your swift response! I confirm it works perfectly on my instance.
Automatically closed - issue fixed for 2 weeks with no activity.