- Status changed to Needs review
5 months ago 1:46pm 1 August 2024 - 🇧🇪Belgium waropd
Extended the first patch to catch more errors
- Check for multiple values
- If value is not numeric, extend loadbyproperties with the vid
I may be using the filter wrong but through my tests I fell on two separate issues ;
From what I see the adminSummary() method tries to load a term based on its id, where it should now try to load it through its machine name properties.
The whole logic behind the autocomplete field seems to be broken as an autocomplete field will always return a tid (based on what the core returns). I'm not sure this type of field should be kept at all to select the terms.
Decide what to do with the autocomplete selection mode.
Once the proposed patch is applied there is still an error when trying to save the view. To fix this this patch is required:
https://www.drupal.org/project/drupal/issues/2925890
🐛
Invalid config structures can result in exceptions when saving a config entity
Needs work
None
None
None
Needs review
1.0
Taxonomy
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Extended the first patch to catch more errors
- Check for multiple values
- If value is not numeric, extend loadbyproperties with the vid