- Issue created by @mkalkbrenner
- Status changed to Needs review
2 months ago 3:26pm 5 April 2024 - π©πͺGermany mkalkbrenner π©πͺ
The issue is caused by π Not all fields that have a target_type setting are entity references Fixed .
That patch tries to avoid aDrupal\Component\Plugin\Exception\PluginNotFoundException
.But the new
$definition->getClass()
that has been added before the critical section could lead to aDrupal\Component\Plugin\Exception\PluginNotFoundException
itself! - π¦πΉAustria drunken monkey Vienna, Austria
drunken monkey β made their first commit to this issueβs fork.
- Merge request !128Fix PluginNotFoundException while adding Views handlers β (Merged) created by drunken monkey
- Status changed to Active
2 months ago 9:00am 6 April 2024 - π¦πΉAustria drunken monkey Vienna, Austria
Created an MR with your patch and some minor style changes. (Testing of patches is not working anymore, we now need to use MRs β a bit funny, I know.) Would be great to get some quick feedback on whether this still worked for you, then Iβll merge and create release 1.34 β hopefully without any critical bugs this time.
And then weβll either keep this open or create a follow-up to develop a proper fix.
-
drunken monkey β
committed cfde7b0a on 8.x-1.x
Issue #3438940 by mkalkbrenner, drunken monkey: Fixed...
-
drunken monkey β
committed cfde7b0a on 8.x-1.x
- Status changed to Fixed
2 months ago 10:45am 6 April 2024 - π¦πΉAustria drunken monkey Vienna, Austria
As discussed on Slack this seems to not be a bug in this module after all, but one in the Solr module that was simply uncovered by the recent change in this one. I created a follow-up issue π Invalid type "text" specified in SolrFieldDefinition data defintions Active in the Solr module but will still create a new release of this module with the workaround in the MR to keep more people from being affected.
Automatically closed - issue fixed for 2 weeks with no activity.