- Issue created by @dieterholvoet
- First commit to issue fork.
- Merge request !87Issue #3444425 by gxleano: Add validation for field name on Tagify element → (Merged) created by gxleano
- last update
9 months ago 7 pass - Status changed to Needs review
9 months ago 2:56pm 1 May 2024 - 🇪🇸Spain gxleano Cáceres
I guess this makes sense, because in that context the widget isn't working with a field. I'm not sure why the #field_name element property was added in the first place, so no idea how to fix this.
The problem is coming from the element, the field name when it's used as views filter is coming from
$element['#name']
instead of$element['#field_name']
, so when the element is used in a different context, the identifier would be thename
attribute. - last update
9 months ago 7 pass - Status changed to RTBC
9 months ago 8:46am 2 May 2024 - 🇧🇪Belgium dieterholvoet Brussels
Just tested and the warning doesn't appear anymore. Why did you delete
Drupal.behaviors.nodeDetailsSummaries
from the main widget though? - 🇪🇸Spain gxleano Cáceres
Thanks Dieter for review.
Why did you delete
Drupal.behaviors.nodeDetailsSummaries
from the main widget though?Because this is already defined in Tagify User List module, and when I was testing the bug that you reported about the filter, I was getting lot of errors related with it, when Tagify is used out of the administration theme. So, it was defined twice and it should be only used in Tagify User List module.
- Status changed to Fixed
9 months ago 9:15am 2 May 2024 - Status changed to Fixed
8 months ago 3:49pm 7 May 2024 - last update
8 months ago 7 pass