- Issue created by @duaelfr
- Status changed to Needs review
6 months ago 8:54am 18 June 2024 - last update
6 months ago 31 pass - Status changed to Needs work
6 months ago 10:20am 18 June 2024 - π©πͺGermany mkalkbrenner π©πͺ
I'll leave that issue open for others to find it. But the the fix is wrong!
The correct fix is to not include the DependencySerializationTrait anymore as this one is already included by PluginBase.
This way the implementation in Search API works and we already committed that change to Search API Solr. - π©πͺGermany chr.fritsch π©πͺπͺπΊπ
Hi Markus,
could you please create a new release then? - π©πͺGermany mkalkbrenner π©πͺ
I think I would be faster if people stop asking over different channels and if I need to answer less questions ;-)
- π¬π§United Kingdom 2dareis2do
I tried commenting put DependencySerializationTrait as per #4 but then I still receive
Fatal error: Declaration of Drupal\search_api_solr\Plugin\search_api\backend\SearchApiSolrBackend::__sleep() must be compatible with Drupal\search_api\Backend\BackendPluginBase::__sleep(): array in /var/www/html/web/modules/contrib/search_api_solr/src/Plugin/search_api/backend/SearchApiSolrBackend.php on line 5077
- π¬π§United Kingdom 2dareis2do
Rolling back to from search api from 1.35 to 1.34 seems to resolve the problem for me (for now).
- π©πͺGermany mkalkbrenner π©πͺ
tried commenting put DependencySerializationTrait as per #4 but then I still receive
Fatal error: Declaration of Drupal\search_api_solr\Plugin\search_api\backend\SearchApiSolrBackend::__sleep() must be compatible with Drupal\search_api\Backend\BackendPluginBase::__sleep(): array in /var/www/html/web/modules/contrib/search_api_solr/src/Plugin/search_api/backend/SearchApiSolrBackend.php on line 5077
You mix two different issues here. Removing DependencySerializationTrait fixes __wakeup(), not __sleep()!
- Status changed to Closed: works as designed
6 months ago 4:31pm 18 June 2024 - π©πͺGermany mkalkbrenner π©πͺ