- Issue created by @larowlan
- First commit to issue fork.
- Merge request !6638Convert EntityReferenceSelection plugin discovery to attributes → (Closed) created by quietone
- Status changed to Needs work
10 months ago 8:32am 16 February 2024 - 🇳🇿New Zealand quietone
This will fail with phpstan errors. One is about the deliberately broken plugin and another is for the missing '$deriver' property.
- First commit to issue fork.
- Status changed to Needs review
10 months ago 6:39am 1 March 2024 - Status changed to Needs work
10 months ago 7:49am 1 March 2024 - Status changed to Needs review
10 months ago 10:49am 1 March 2024 - Status changed to Needs work
10 months ago 3:17pm 1 March 2024 - Status changed to Needs review
10 months ago 12:06pm 6 March 2024 - Status changed to RTBC
10 months ago 6:18pm 6 March 2024 - 🇺🇸United States smustgrave
Appears all instances and feedback has been addressed.
- Status changed to Needs work
10 months ago 10:33am 8 March 2024 - Status changed to Needs review
10 months ago 10:59am 8 March 2024 - Status changed to RTBC
10 months ago 1:28pm 8 March 2024 - Status changed to Needs review
10 months ago 2:34pm 8 March 2024 - 🇬🇧United Kingdom alexpott 🇪🇺🌍
I had to make some changes to the MR in order to benefit from attributes. Weight and group are now required. Annotations was quite loose so we couldn't enforce this properly there but now we can. I don't think that the broken handler should cause to make them optional - given how the broken handler works - see \Drupal\Core\Entity\EntityReferenceSelection\SelectionPluginManager. We can provide some defaults in that plugin that are going to be ignored.
- Status changed to RTBC
10 months ago 4:23pm 8 March 2024 - 🇺🇸United States smustgrave
Update using weight and group didn't break anything.
-
larowlan →
committed 2061b8fb on 10.3.x
Issue #3420982 by sorlov, alexpott: Convert EntityReferenceSelection...
-
larowlan →
committed 2061b8fb on 10.3.x
-
larowlan →
committed b4a9a825 on 11.x
Issue #3420982 by sorlov, alexpott: Convert EntityReferenceSelection...
-
larowlan →
committed b4a9a825 on 11.x
- Status changed to Fixed
10 months ago 8:03am 10 March 2024 - 🇦🇺Australia larowlan 🇦🇺🏝.au GMT+10
Committed to 11.x and backported to 10.3.x
Automatically closed - issue fixed for 2 weeks with no activity.