- Issue created by @ΓΈkse
- Merge request !10Issue #3344938: Get the current translation within the wrapped field output β (Merged) created by ΓΈkse
- Status changed to Needs review
about 2 years ago 8:18am 31 March 2023 - π§πͺBelgium tim-diels Belgium π§πͺ
Setting to NR as there is a patch.
- πͺπΈSpain akalam
I can confirm the issue, the referenced entities are displayed in the original content of the parent entity, instead of using the current language selected for the page. The MR is fixing the issue.
- Status changed to RTBC
almost 2 years ago 5:38am 6 June 2023 - last update
almost 2 years ago 10 pass - πΊπ¦Ukraine nginex
Checked code and functionally tested on real project, let's move on +1
Besides, I'm uploading a static copy of the MR 10.
- Status changed to Needs review
over 1 year ago 11:00am 5 September 2023 - last update
over 1 year ago 10 pass - π§πͺBelgium Kvermeir
Using patch form #5 in combination with the solution from π Infinite loop when first available formatter extends FieldWrapperBase Fixed , still gives use infinite loop for entity reference fields.
The function getTranslationFromContext triggers the loop, therefore we used the default functions hasTranslation() & getTranslation(). - Status changed to Needs work
over 1 year ago 2:58pm 16 October 2023 - π©πͺGermany Anybody Porta Westfalica
Thanks all! Let's get this finished. Could someone please rebase this, incorporate the new branch changes and also discuss #6?
Generally the code from #6
+ if ($entity->hasTranslation($langcode)) { + $entity = $entity->getTranslation($langcode); + }
is a typical pattern, but we should please ensure it's needed here and not already inherited or handled somewhere down or up the trace.
- π§πͺBelgium elangoens
Updated the patch from #5 as it doesn't apply on rc3.
- Status changed to Needs review
12 months ago 12:45pm 26 April 2024 - last update
12 months ago 10 pass - π§πͺBelgium elangoens
Updated the patch from #5 as it didn't apply to rc3.
https://www.drupal.org/files/issues/2024-04-26/3344938-8.patch β - π©πͺGermany Anybody Porta Westfalica
Mhm I got a ping from this issue but can't see any activity. Would anyone like to finish this perhaps?
- Status changed to Needs work
11 months ago 5:12am 7 May 2024 - First commit to issue fork.
- last update
10 months ago 10 pass - π«π·France duaelfr Montpellier, France
Patch from #8 is working and looking good.
It's better than what's suggested in #6 becausegetTranslationFromContext
allows fallback languages to be selected which can be very useful for some projects.MR rebased!
- Status changed to RTBC
10 months ago 7:29am 18 June 2024 - π©πͺGermany Anybody Porta Westfalica
Perfect, thank you @DuaelFr!
Let's merge this.
- last update
10 months ago 10 pass - Status changed to Fixed
10 months ago 7:30am 18 June 2024 -
Anybody β
committed 562587c4 on 3.x authored by
ΓΈkse β
Issue #3344938 by elangoens, DuaelFr, nginex, Kvermeir, Anybody: Get the...
-
Anybody β
committed 562587c4 on 3.x authored by
ΓΈkse β
Automatically closed - issue fixed for 2 weeks with no activity.