- Issue created by @guiu.rocafort.ferrer
- Status changed to Needs review
over 1 year ago 1:37pm 16 February 2024 - 🇪🇸Spain guiu.rocafort.ferrer Barcelona
Changing issue priority to Major as per issue priorities →
- Merge request !135Resolve #3421922 "Remove domainentityload hook" → (Merged) created by guiu.rocafort.ferrer
- 🇪🇸Spain guiu.rocafort.ferrer Barcelona
Steps to reproduce:
- Install Country Path 8.x-1.x ( dev branch ).
- Enable Domain and Country Path.
- Go to "/admin/config/domain" and create multiple domains.
- Navigate to the different domains, validate they load properly.
- Apply this patch to the Country path module ( https://www.drupal.org/project/country_path/issues/2961459 🐛 Country path not managed in domain navigation block Needs work ).
- Navigate to any page in any domain. There is an infinite loop and the pages fail to load.Then, apply the merge request diff ( https://git.drupalcode.org/project/domain/-/merge_requests/135.diff ) to domain module and clear caches. The domains should load properly now.
- First commit to issue fork.
- First commit to issue fork.
-
webflo →
committed a10b5da0 on 2.0.x authored by
guiu.rocafort.ferrer →
Issue #3421922 by guiu.rocafort.ferrer, webflo, idebr: Remove...
-
webflo →
committed a10b5da0 on 2.0.x authored by
guiu.rocafort.ferrer →
- 🇩🇪Germany webflo
Thanks, I looked at the current test coverage of this feature. I think its fine to remove the hook. The getter methods work the same as before.
Automatically closed - issue fixed for 2 weeks with no activity.