- Issue created by @a.raguckas
- Status changed to Needs review
10 months ago 1:30pm 23 February 2024 - 🇮🇳India abhishek_virasat
fixed the issue and created patch please review it.
- Merge request !143423475 replace hook_field_widget_form_alter() because it is deprecated to... → (Open) created by a.raguckas
- 🇱🇹Lithuania a.raguckas Kaunas
Hi. I just send first pull request please review.
- 🇪🇸Spain jlbellido
Hello,
I've checked the patch from #2 but unfortunately it is not complete and didn't work for me.
I've also tested the MR from #4 and it did work. Now the current domain is applied as default value for taxonomies in my case.Finally, I'm adding some additional context information to the ticket description.
- 🇪🇸Spain jlbellido
Ok, I had a second look and I was mistaken.
#2 Is correct, because it is using hook_field_widget_single_element_form_alter which is the recommended replacement for hook_field_widget_complete_form_alter().
#4 Uses hook_field_widget_complete_form_alter() instead but it is not needed for this context.
In any case, I think it would be good to have a MR with the #2 approach. In that way it is easier for maintainers to merge it.
Thanks both!
- 🇺🇦Ukraine Anna D
Tested #2 works for me.
From #5 comment:
Now the current domain is applied as default value for taxonomies in my case.
Related issue is here: https://www.drupal.org/project/domain_entity/issues/3365165 🐛 Default domain values not set and overridden by current domain Needs review