Add default locale itself as hreflang too when using alternate option (…

…#271)
prestaconcept · Feb 25, 2021 · fb83304 · fb83304
1 parent 63f54b8
commit fb83304
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/EventListener/StaticRoutesAlternateEventListener.php b/EventListener/StaticRoutesAlternateEventListener.php
@@ -66,10 +66,6 @@ public function addAlternate(SitemapAddUrlEvent $event): void
             )
         );
         foreach ($this->options['locales'] as $alternate) {
-            if ($alternate === $locale) {
-                continue; // avoid re-adding default route
-            }
-
             $url->addLink($this->generateTranslatedRouteUrl($translatedName, $alternate), $alternate);
         }
 

diff --git a/Tests/Unit/EventListener/StaticRoutesAlternateEventListenerTest.php b/Tests/Unit/EventListener/StaticRoutesAlternateEventListenerTest.php
@@ -47,7 +47,7 @@ public function testTranslatedUrls(
     public function translated(): \Generator
     {
         $options = ['lastmod' => null, 'changefreq' => null, 'priority' => null];
-        $xml = '<url><loc>https://acme.org/about</loc><xhtml:link rel="alternate" hreflang="fr" href="https://acme.org/a-propos" /></url>';
+        $xml = '<url><loc>https://acme.org/about</loc><xhtml:link rel="alternate" hreflang="en" href="https://acme.org/about" /><xhtml:link rel="alternate" hreflang="fr" href="https://acme.org/a-propos" /></url>';
         yield [
             self::SYMFONY_OPTIONS,
             'about.en',