- Issue created by @RandalV
- 🇧🇪Belgium RandalV
A little correction from my end, all the functions that call the "renderSiteSettings"-method in the extension class do seem to render with the correct language context, but the other two functions (`site_setting_field` && `site_setting_entity_by_name`) specifically check for a langcode and load that translation, so those should be altered I think.
I'll add a MR for it.
- Merge request !49Issue #3507349 by randalv: Twig functions aren't aware of the currently active language → (Merged) created by RandalV
- 🇧🇪Belgium RandalV
Merge request added!
I also took the liberty to remove what I assume was an orphaned `var_dump` 😇
- 🇬🇧United Kingdom scott_euser
Ha good spot on the var_dump, thank you!
Let me know if you're able to add automated test coverage to this? I believe the sample data module would need some translations added in order to make it testable. If you aren't able I'll add to my list to eventually do.
Thanks!
- First commit to issue fork.
- 🇧🇪Belgium svendecabooter Gent
I have added test coverage for this.
Not sure if this belongs in LoaderLanguageTest specifically, but it had all the setup logic already available, so seemed like the easiest place to add this test. - 🇬🇧United Kingdom scott_euser
Cheers thank you! Added a post update hook to clear the cache for the service args change just in case per https://www.drupal.org/project/drupal/issues/2743313 →
Merging, thanks!
-
scott_euser →
committed 3e45f52e on 2.0.x authored by
randalv →
Issue #3507349 by svendecabooter, randalv: Some twig functions aren't...
-
scott_euser →
committed 3e45f52e on 2.0.x authored by
randalv →
Automatically closed - issue fixed for 2 weeks with no activity.