- Issue created by @a.raguckas
- Status changed to Needs review
about 1 year 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
- Status changed to RTBC
2 months ago 10:17am 31 January 2025 - First commit to issue fork.
-
frederikvho →
committed 9e3eb11b on 8.x-1.x
Issue #3423475: Applied patch from comment 2.
-
frederikvho →
committed 9e3eb11b on 8.x-1.x
- 🇧🇪Belgium Frederikvho Belgium
Thank you everyone, this is in the latest release 8.x-1.0-rc3
Automatically closed - issue fixed for 2 weeks with no activity.