- Issue created by @pdureau
- π©πͺGermany Christian.wiedemann
Christian.wiedemann β made their first commit to this issueβs fork.
- π©πͺGermany Christian.wiedemann
I see folowing possible solutions:
- In SourcePluginManager::getDefinitionsForPropType: if ($prop instanceOf UnknownPropType) {return [];}
- Add prop types to EntityFieldSource via alter hook and make prop_type required.
- Add a is isApplicable method to the source. (We had something like this before. :))
- Assigned to just_like_good_vibes
- π©πͺGermany Christian.wiedemann
@just_like_good_vibes: I need your 5 cents here :)
- π«π·France just_like_good_vibes PARIS
my question, why do we need unknownPropType ? sorry i forgot.
if it is an hardcoded edge case, where no sources should be shown,
my favorite solution between the three solutions proposed would be : "In SourcePluginManager::getDefinitionsForPropType: if ($prop instanceOf UnknownPropType) {return [];}" - Assigned to Christian.wiedemann
- Assigned to just_like_good_vibes
- Status changed to Needs review
3 months ago 5:05pm 4 August 2024 - Assigned to pdureau
- π«π·France just_like_good_vibes PARIS
Pierre, would you review please :)?
-
pdureau β
committed 66cd3dfe on 2.0.x authored by
Christian.wiedemann β
Issue #3465524 by Christian.wiedemann, pdureau, just_like_good_vibes:...
-
pdureau β
committed 66cd3dfe on 2.0.x authored by
Christian.wiedemann β
- Issue was unassigned.
- Status changed to Fixed
3 months ago 6:19pm 4 August 2024 - Status changed to Fixed
3 months ago 2:47pm 12 August 2024