chore(TranslationAware): Try to get translation resolved from LocaleS…

…ubscriber before
roadiz · Feb 13, 2024 · 18edbeb · 18edbeb
1 parent 9e70e04
commit 18edbeb
Show file tree

Hide file tree

Showing 2 changed files with 34 additions and 19 deletions.
diff --git a/lib/RoadizCoreBundle/src/Serializer/Normalizer/TranslationAwareNormalizer.php b/lib/RoadizCoreBundle/src/Serializer/Normalizer/TranslationAwareNormalizer.php
@@ -82,6 +82,9 @@ private function getTranslationFromRequest(): ?TranslationInterface
                 ->findDefault();
         }
 
+        /*
+         * Try to get translation resolved from LocaleSubscriber before
+         */
         $requestTranslation = $request->attributes->get('_translation');
         if ($requestTranslation instanceof TranslationInterface) {
             return $requestTranslation;

diff --git a/lib/RoadizCoreBundle/src/Serializer/TranslationAwareContextBuilder.php b/lib/RoadizCoreBundle/src/Serializer/TranslationAwareContextBuilder.php
@@ -33,26 +33,38 @@ public function createFromRequest(Request $request, bool $normalization, array $
     {
         $context = $this->decorated->createFromRequest($request, $normalization, $extractedAttributes);
 
-        if (!isset($context['translation']) || !($context['translation'] instanceof TranslationInterface)) {
-            /** @var TranslationRepository $repository */
-            $repository = $this->managerRegistry
-                ->getRepository(TranslationInterface::class);
-            $locale = $request->query->get('_locale', $request->getLocale());
-
-            if (!\is_string($locale)) {
-                return $context;
-            }
-
-            if ($this->previewResolver->isPreview()) {
-                $translation = $repository->findOneByLocaleOrOverrideLocale($locale);
-            } else {
-                $translation = $repository->findOneAvailableByLocaleOrOverrideLocale($locale);
-            }
-
-            if ($translation instanceof TranslationInterface) {
-                $context['translation'] = $translation;
-            }
+        if (isset($context['translation']) && $context['translation'] instanceof TranslationInterface) {
+            return $context;
         }
+
+        /*
+         * Try to get translation resolved from LocaleSubscriber before
+         */
+        $requestTranslation = $request->attributes->get('_translation');
+        if ($requestTranslation instanceof TranslationInterface) {
+            $context['translation'] = $requestTranslation;
+            return $context;
+        }
+
+        /** @var TranslationRepository $repository */
+        $repository = $this->managerRegistry
+            ->getRepository(TranslationInterface::class);
+        $locale = $request->query->get('_locale', $request->getLocale());
+
+        if (!\is_string($locale)) {
+            return $context;
+        }
+
+        if ($this->previewResolver->isPreview()) {
+            $translation = $repository->findOneByLocaleOrOverrideLocale($locale);
+        } else {
+            $translation = $repository->findOneAvailableByLocaleOrOverrideLocale($locale);
+        }
+
+        if ($translation instanceof TranslationInterface) {
+            $context['translation'] = $translation;
+        }
+
         return $context;
     }
 }