- Merge request !15Issue #3177109: The invoice print builder should switch the active language to... → (Merged) created by jsacksick
- 🇮🇱Israel jsacksick
Opened an MR with the patch from #13, let's see what the tests say.
- 🇮🇱Israel jsacksick
I'm wondering if we aren't missing code similar to what we're doing in the PromotionOrderProcessor:
if ($promotion->hasTranslation($content_langcode)) { $promotion = $promotion->getTranslation($content_langcode); }
And rely on the content langcode rather than the current language... But I'm going to trust that this has been tested... Fixed the tests, so will probably go ahead and commit this... Or just wait for the next release.
- 🇮🇱Israel jsacksick
Or maybe just:
$current_langcode = $this->languageManager->getCurrentLanguage()->getId();
to
$current_langcode = $this->languageManager->getCurrentLanguage(LanguageInterface::TYPE_CONTENT)->getId();
-
jsacksick →
committed 67d29156 on 8.x-2.x
Issue #3177109 by jsacksick, idebr, lisastreeter, kristiaanvandeneynde:...
-
jsacksick →
committed 67d29156 on 8.x-2.x
Automatically closed - issue fixed for 2 weeks with no activity.