Currently, when a term reference field is not set to auto create new terms, its not possible to add terms without overwriting existing ones. I think this behavior should only be dependent on cardinality. Also it seems that the implemented autocomplete field does not even support auto creating terms (as documented here https://www.drupal.org/node/2418529).
If cardinality is 1 overwrite existing term, otherwise merge with existing terms.
Add support for term auto creation.
None
None
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | allow_adding_with_autocreate_disabled-3248763-4.patch | 3.39 KB | fjgarlin |
| #3 | Screenshot 2021-11-11 at 20.55.21.png | 17 KB | fjgarlin |
| #2 | allow_adding_with_autocreate_disabled-3248763-2.patch | 3.04 KB | criz |
Comments
Comment #2
crizHere is a patch.
Please note that when the maximum of allowed values (cardinality) is already populated for a node, an existing value will be overwritten, no matter if cardinality is not 1. This was also the case before this patch. Not sure how to handle this, as the description of the form field is not always true. Maybe at least a note in the readme would be good.
Comment #3
fjgarlin commentedThis line will add the new terms to the first available vocabulary, not the one selected:
'bundle' => array_values($target_bundles)[0],Selected one:
So in this example, new tags will be added to "Tags", but should be added to "Test"
Comment #4
fjgarlin commentedI altered slightly your patch to deal with the situation described above. Please test it out and let me know if it still does everything as you would expect. I tried myself and it was working but it'd be great a second pair of eyes.
Comment #5
crizThanks! Yes, I missed this setting. Tested and works.
Comment #7
fjgarlin commentedThanks for this addition. I've pushed the change so I'll close the issue.
Comment #8
fjgarlin commented