- Issue created by @joachim
- Assigned to prabha1997
- Merge request !8384Issue #3446403 by edutrul, jnicola, mradcliffe: [name]: This field is missing for example recipe → (Open) created by prabha1997
- Issue was unassigned.
- Status changed to Needs review
10 months ago 8:10am 12 June 2024 - Status changed to Needs work
10 months ago 8:37am 12 June 2024 - 🇬🇧United Kingdom joachim
There's something wrong with your MR -- it's saying 1000+ changes!
Are you sure your branches are correct?
- First commit to issue fork.
- 🇵🇹Portugal dxvargas
I'm extremely confused with the MR that I see related with this issue. So many commits... why?
As I understand we simply need to add something like this to
lib/Drupal/Core/Entity/entity.api.php:/** * Modify the list of available entity reference plugins. * * This hook may be used to modify plugin properties after they have been * specified by other modules. * * @param array $plugins * An array of all the existing plugin definitions, passed by reference. * * @see \Drupal\Core\Entity\EntityReferenceSelection\SelectionPluginManager */ function hook_entity_reference_selection_alter(array &$plugins) { // Remove a plugin. unset($plugins['broken']); }What am I missing?
- First commit to issue fork.
- Merge request !9634Document the hookhook_entity_reference_selection_alter. → (Open) created by Unnamed author
- 🇬🇧United Kingdom joachim
> I'm extremely confused with the MR that I see related with this issue. So many commits... why?
That can happen when the target branch set on the MR and the actual parent branch of the MR branch are not the same.
- 🇧🇪Belgium xdequinze Brussels
I was working on this issue with nhojivar7 at DrupalCon Barcelona contribution's day.
- Status changed to Needs review
6 days ago 1:47pm 5 April 2025 - 🇧🇷Brazil isa.bel Balneário Camboriú
The documentation seems complete and correct now, I will set this to Needs Review
- 🇺🇸United States smustgrave
Can 1 MR be closed or hidden.
Also may want to check usage as I don't see anything in core besides a test using this hook. Maybe it's not needed?
- 🇬🇧United Kingdom joachim
> Also may want to check usage as I don't see anything in core besides a test using this hook. Maybe it's not needed?
It's standard for all plugin managers to have an alter hook for the discovered plugin info.
The Needs Review Queue Bot → tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.