tpl: Use AllTranslations in sitemap template

bep · bep · commit 6475d308ec8c · 2026-05-21T19:44:23.000+02:00
Closes #14912 Closes #14917
diff --git a/hugolib/sitemap_test.go b/hugolib/sitemap_test.go
@@ -80,8 +80,8 @@ Doc2
 	b := Test(t, files)
 
 	b.AssertFileContent("public/sitemap.xml", "<loc>https://example.com/en/sitemap.xml</loc>", "<loc>https://example.com/nn/sitemap.xml</loc>")
-	b.AssertFileContent("public/en/sitemap.xml", " <loc>https://example.com/sect/doc1/</loc>", "doc2")
-	b.AssertFileContent("public/nn/sitemap.xml", " <loc>https://example.com/nn/sect/doc2/</loc>")
+	b.AssertFileContent("public/en/sitemap.xml", " <loc>https://example.com/sect/doc1/</loc>", "doc2", `hreflang="en"`, `hreflang="nn"`)
+	b.AssertFileContent("public/nn/sitemap.xml", " <loc>https://example.com/nn/sect/doc2/</loc>", `hreflang="en"`, `hreflang="nn"`)
 }
 
 // https://github.com/gohugoio/hugo/issues/5910
diff --git a/tpl/tplimpl/embedded/templates/sitemap.xml b/tpl/tplimpl/embedded/templates/sitemap.xml
@@ -8,17 +8,12 @@
     <loc>{{ .Permalink }}</loc>{{ if not .Lastmod.IsZero }}
     <lastmod>{{ safeHTML ( .Lastmod.Format "2006-01-02T15:04:05-07:00" ) }}</lastmod>{{ end }}{{ with .Sitemap.ChangeFreq }}
     <changefreq>{{ . }}</changefreq>{{ end }}{{ if ge .Sitemap.Priority 0.0 }}
-    <priority>{{ .Sitemap.Priority }}</priority>{{ end }}{{ if .IsTranslated }}{{ range .Translations }}
+    <priority>{{ .Sitemap.Priority }}</priority>{{ end }}{{ $allTranslations := .AllTranslations }}{{ if gt ($allTranslations | len) 1  }}{{ range $allTranslations }}
     <xhtml:link
                 rel="alternate"
                 hreflang="{{ .Language.Locale }}"
                 href="{{ .Permalink }}"
-                />{{ end }}
-    <xhtml:link
-                rel="alternate"
-                hreflang="{{ .Language.Locale }}"
-                href="{{ .Permalink }}"
-                />{{ end }}
+                />{{ end }}{{ end }}
   </url>
     {{- end -}}
   {{ end }}

Original file line number	Diff line number	Diff line change
`@@ -80,8 +80,8 @@ Doc2`
`80`	`80`	`b := Test(t, files)`
`81`	`81`
`82`	`82`	`b.AssertFileContent("public/sitemap.xml", "<loc>https://example.com/en/sitemap.xml</loc>", "<loc>https://example.com/nn/sitemap.xml</loc>")`
`83`		`- b.AssertFileContent("public/en/sitemap.xml", " <loc>https://example.com/sect/doc1/</loc>", "doc2")`
`84`		`- b.AssertFileContent("public/nn/sitemap.xml", " <loc>https://example.com/nn/sect/doc2/</loc>")`
	`83`	+ b.AssertFileContent("public/en/sitemap.xml", " <loc>https://example.com/sect/doc1/</loc>", "doc2", `hreflang="en"`, `hreflang="nn"`)
	`84`	+ b.AssertFileContent("public/nn/sitemap.xml", " <loc>https://example.com/nn/sect/doc2/</loc>", `hreflang="en"`, `hreflang="nn"`)
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`// https://github.com/gohugoio/hugo/issues/5910`