Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.After #2044393: Changes in search query functionality search api solr integration is completely broken
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 2636270-6--head_broken-interdiff.txt | 632 bytes | Berdir |
| #6 | 2636270-6--head_broken.patch | 12.96 KB | Berdir |
| #5 | 2636270-5--head_broken.patch | 12.34 KB | drunken monkey |
Comments
Comment #2
swentel CreditAttribution: swentel commentedMerely did a search and replace, search at least starts working again
Comment #3
BerdirTried this, doesn't work for me yet, maybe that's a fault with my custom filters but they don't seem to do much.
I get this error:
Recoverable fatal error: Argument 1 passed to Drupal\search_api_solr\Plugin\search_api\backend\SearchApiSolrBackend::createFilterQueries() must implement interface Drupal\search_api\Query\ConditionGroupInterface, instance of Drupal\search_api\Query\Condition given, called in ...modules/contrib/search_api_solr/src/Plugin/search_api/backend/SearchApiSolrBackend.php on line 1326 and defined in Drupal\search_api_solr\Plugin\search_api\backend\SearchApiSolrBackend->createFilterQueries() (line 1313 ...
It seems to be called repeatedly.. the first time with a condition group and the second time with a Condition..
Comment #4
drunken monkeyComment #5
drunken monkeyThis one works for me to bring this module up-to-date regarding changes both in the Search API and in Core.
I also created #2645222: Clean up code with some unrelated clean-up.
Comment #6
BerdirAwesome.
I've tested this with the updated test and I got a few errors related to facets and specific regression tests (e.g. case stuff) but commented out a few that failed and then got a passing tests, so the basics seem to be working again!
Will commit this with the attached additional test fix.
Comment #7
Berdir