Merge pull request #3 from oliverschroeder/main

[BUGFIX] Language is taken into account when generating the source path
georgringer · Jul 31, 2023 · c4dfd28 · c4dfd28
2 parents bf8b152 + 598b99a
commit c4dfd28
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/Classes/Service/SlugService.php b/Classes/Service/SlugService.php
@@ -91,7 +91,7 @@ protected function createRedirect(string $originalSlug, int $recordId, int $lang
             'parameters' => '_language=' . $languageId . '&tx_news_pi1[controller]=News&tx_news_pi1[action]=detail&tx_news_pi1[news]=' . $recordId,
         ]);
 
-        $sourcePath = $this->generateUrl($this->site, $recordId, $this->targetPageId);
+        $sourcePath = $this->generateUrl($this->site, $recordId, $this->targetPageId, $languageId);
         $sourcePath = '/' . ltrim(str_replace(['http://', 'https://', $siteLanguage->getBase()->getHost()], '', $sourcePath), '/');
 
         $record = [
@@ -126,14 +126,15 @@ protected function createRedirect(string $originalSlug, int $recordId, int $lang
     }
 
 
-    protected function generateUrl(SiteInterface $site, int $recordId, int $detailPageId): string
+    protected function generateUrl(SiteInterface $site, int $recordId, int $detailPageId, int $languageId): string
     {
         $additionalQueryParams = [
             'tx_news_pi1' => [
                 'action' => 'detail',
                 'controller' => 'News',
                 'news' => $recordId
-            ]
+            ],
+            '_language' => $languageId
         ];
         return (string)$site->getRouter()->generateUri(
             (string)$detailPageId,