- ๐ง๐ทBrazil carolpettirossi Campinas - SP
This would be really really useful. It's very common to have Content referencing other content of the same type to create blocks like "Related Articles", "Recommended products" and editors being responsible to curate these blocks by choosing the specific nodes in the entity reference field.
I tested the patch from #79 and it did not work as I show in the video attached:
Screenshare - 2023-02-07 12_05_04 PM.mp4 (3.96 MB) - Status changed to Needs review
almost 2 years ago 4:09pm 7 February 2023 - ๐ฎ๐ณIndia Nitin shrivastava
For, d10 #79 patch not applied cleanly.
Fix for d10 - Status changed to Needs work
almost 2 years ago 8:59pm 18 February 2023 - ๐บ๐ธUnited States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request โ as a guide.
Issue summary should include the proposed solution to the problem.
Also since we are adding a new setting this seems like it could affect existing sites. Think we would need an upgrade path.
- ๐จ๐ญSwitzerland berdir Switzerland
This needs to be redone on top of ๐ Entity autocomplete widget does not pass along entity to AJAX request Fixed . That issue already added a test implementation for this, we just need to convert that to a setting like this issue is doing. We can probably remove the test implementation there again.
- ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
Liam Morland โ made their first commit to this issueโs fork.
- Merge request !7159Resolve #2280479 "Add setting to EntityReferenceSelection plugins to prevent references to referencing entity" โ (Open) created by Liam Morland
- ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
I have created a merge request with the patch in #88 ported to 11.x.
Upgrade path: Perhaps the variable should be
prevent_self_reference
instead ofallow_self_reference
. That way, if it is not set, it has the same behaviour as before. Nothing changes until someone configures it differently. - Status changed to Needs review
8 months ago 3:32pm 22 March 2024 - ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
The upgrade path already works. I installed this on my 10.2.4 site. In my test field, allow_self_reference was already checked and self-reference was allowed, as before. I unchecked it and self-reference was prevented.
- ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
Patch from merge request ported to Drupal 10.2.4.
- Status changed to Needs work
8 months ago 4:47pm 22 March 2024 The Needs Review Queue Bot โ tested this issue. It no longer applies to Drupal core. 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.
- Status changed to Needs review
8 months ago 4:49pm 22 March 2024 - ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
The bot is referring to the patch in 97. The current version is the merge request.
- Status changed to Needs work
8 months ago 5:27pm 22 March 2024 The Needs Review Queue Bot โ tested this issue. It no longer applies to Drupal core. 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.
- Status changed to Needs review
8 months ago 5:40pm 22 March 2024 - ๐จ๐ฆCanada Liam Morland Ontario, CA ๐จ๐ฆ
Patch from merge request, just to stop the bot from resetting the status.
- Status changed to Needs work
8 months ago 7:24pm 28 March 2024 - ๐บ๐ธUnited States smustgrave
If going to add a new UI setting will probably need usability sign off.
Also with schema change going to an upgrade path.