- Issue created by @dipakmdhrm
- ๐ฎ๐ณIndia dipakmdhrm
Patch for people not comfortable with MR workflow.
- Status changed to Needs review
about 2 years ago 3:51pm 1 June 2023 - ๐บ๐ธUnited States devkinetic
I took a look and this seems to make sense to me. I'll do some testing, and would welcome anyone else's feedback as well.
- Status changed to RTBC
about 2 years ago 7:10pm 12 July 2023 - ๐ง๐ทBrazil andre.bonon
I did a test setting the "Allowed number of values" to unlimited in the field storage settings and set the cardinality limited to 4.
In the entity form, Drupal was displaying 2 fields and the 'Add another item' button before this patch was applied.
Once this patch is applied, I can see the 4 fields displayed.
- Status changed to Needs work
4 months ago 2:32pm 14 April 2025 - First commit to issue fork.
- ๐บ๐ธUnited States smustgrave
MR needs to be updated for 4.0.x and hopefully those pipeline errors go away
- ๐ฐ๐ฌKyrgyzstan elaman
elaman โ changed the visibility of the branch 4.0.x to hidden.
- ๐ฐ๐ฌKyrgyzstan elaman
elaman โ changed the visibility of the branch 3.x to hidden.
- ๐ฐ๐ฌKyrgyzstan elaman
elaman โ changed the visibility of the branch 3.x to hidden.
- ๐ง๐ทBrazil andre.bonon
I just tested the patch for 4.x, and it still works as expected with the โAutocompleteโ widget. I tested it using both the Gin and Claro admin themes.
- ๐บ๐ธUnited States smustgrave
Almost the entire issue queue seems to be bugs and probably because it lacks test coverage. I'm working on adding some basic coverage so we can wait on those are start new ones too
- ๐บ๐ธUnited States smustgrave
Kinda seems like a feature request to me that should be configurable. Someone may not want the limit showing all at once.
- ๐ง๐ทBrazil andre.bonon
Iโve added a new configuration option to the Entity Reference field settings that allows site builders to enable the behavior of displaying all field instances. This functionality is currently scoped to Entity Reference fields only, as extending it to other field types would require a more significant refactor across multiple widget implementations.
Iโve also added test coverage for this new setting, following the structure of the existing tests.
- ๐บ๐ธUnited States smustgrave
Will need a post-update hook also. Should use ConfigImporter to do it batches