Add sitemap index for multilingual sites

See #2309
This commit is contained in:
Bjørn Erik Pedersen 2016-08-04 22:12:19 +02:00
parent d572071784
commit b6fcb3d0bb
8 changed files with 82 additions and 29 deletions

View file

@ -61,6 +61,14 @@ func (h HugoSites) Reset() {
}
}
func (h HugoSites) siteInfos() []*SiteInfo {
infos := make([]*SiteInfo, len(h.Sites))
for i, s := range h.Sites {
infos[i] = &s.Info
}
return infos
}
type BuildCfg struct {
// Whether we are in watch (server) mode
Watching bool
@ -118,9 +126,9 @@ func (h HugoSites) Build(config BuildCfg) error {
return err
}
if !config.skipRender {
for _, s := range h.Sites {
if !config.skipRender {
if err := s.Render(); err != nil {
return err
}
@ -129,7 +137,10 @@ func (h HugoSites) Build(config BuildCfg) error {
s.Stats()
}
}
// TODO(bep) ml lang in site.Info?
if err := h.render(); err != nil {
return err
}
}
if config.PrintStats {
@ -180,6 +191,10 @@ func (h HugoSites) Rebuild(config BuildCfg, events ...fsnotify.Event) error {
s.Stats()
}
}
if err := h.render(); err != nil {
return err
}
}
if config.PrintStats {
@ -190,6 +205,28 @@ func (h HugoSites) Rebuild(config BuildCfg, events ...fsnotify.Event) error {
}
// Render the cross-site artifacts.
func (h *HugoSites) render() error {
if !h.Multilingual.enabled() {
return nil
}
// TODO(bep) DRY
sitemapDefault := parseSitemap(viper.GetStringMap("Sitemap"))
s := h.Sites[0]
smLayouts := []string{"sitemapindex.xml", "_default/sitemapindex.xml", "_internal/_default/sitemapindex.xml"}
if err := s.renderAndWriteXML("sitemapindex", sitemapDefault.Filename,
h.siteInfos(), s.appendThemeTemplates(smLayouts)...); err != nil {
return err
}
return nil
}
func (s *HugoSites) setupTranslations(master *Site) {
for _, p := range master.rawAllPages {

View file

@ -126,11 +126,18 @@ func TestMultiSites(t *testing.T) {
assert.Equal(t, "fr", frenchPage.Lang())
}
// Check redirect to main language, French
languageRedirect := readDestination(t, "public/index.html")
// French is the main content language
require.True(t, strings.Contains(languageRedirect, "0; url=http://example.com/blog/fr"), languageRedirect)
// Check sitemap(s)
sitemapIndex := readDestination(t, "public/sitemap.xml")
require.True(t, strings.Contains(sitemapIndex, "<loc>http:/example.com/blog/en/sitemap.xml</loc>"), sitemapIndex)
require.True(t, strings.Contains(sitemapIndex, "<loc>http:/example.com/blog/fr/sitemap.xml</loc>"), sitemapIndex)
sitemapEn := readDestination(t, "public/en/sitemap.xml")
sitemapFr := readDestination(t, "public/fr/sitemap.xml")
require.True(t, strings.Contains(sitemapEn, "http://example.com/blog/en/sect/doc2/"), sitemapEn)
require.True(t, strings.Contains(sitemapFr, "http://example.com/blog/fr/sect/doc1/"), sitemapFr)
}
func TestMultiSitesRebuild(t *testing.T) {

View file

@ -1,11 +0,0 @@
<?xml version="1.0" encoding="utf-8" standalone="yes" ?>
<rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom">
<channel>
<title></title>
<link>/rss/</link>
<description>Recent content on </description>
<generator>Hugo -- gohugo.io</generator>
<atom:link href="/rss/" rel="self" type="application/rss+xml" />
</channel>
</rss>

View file

@ -1,8 +0,0 @@
<?xml version="1.0" encoding="utf-8" standalone="yes" ?>
<urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
<url>
<loc>/</loc>
</url>
</urlset>

View file

@ -171,7 +171,7 @@ type SiteInfo struct {
Data *map[string]interface{}
Multilingual bool
CurrentLanguage string
Language *Language
LanguagePrefix string
Languages Languages
}
@ -831,6 +831,21 @@ func (s *Site) initialize() (err error) {
return
}
// HomeURL is a convenience method giving the absolute URL to the home page.
func (s *SiteInfo) HomeAbsURL() string {
base := "/"
if s.Multilingual {
base = s.Language.Lang
}
return helpers.AbsURL(base)
}
// SitemapAbsURL is a convenience method giving the absolute URL to the sitemap.
func (s *SiteInfo) SitemapAbsURL() string {
sitemapDefault := parseSitemap(viper.GetStringMap("Sitemap"))
return path.Join(s.HomeAbsURL(), sitemapDefault.Filename)
}
func (s *Site) initializeSiteInfo() {
var (
@ -864,7 +879,7 @@ func (s *Site) initializeSiteInfo() {
DisqusShortname: lang.GetString("DisqusShortname"),
// TODO(bep) multilang, consolidate the below (make into methods etc.)
Multilingual: s.multilingualEnabled(),
CurrentLanguage: lang.Lang,
Language: lang,
LanguagePrefix: languagePrefix,
Languages: languages,
GoogleAnalytics: lang.GetString("GoogleAnalytics"),

View file

@ -100,6 +100,19 @@ func (t *GoHTMLTemplate) EmbedTemplates() {
{{ end }}
</urlset>`)
// For multilanguage sites
t.AddInternalTemplate("_default", "sitemapindex.xml", `<sitemapindex xmlns="http://www.sitemaps.org/schemas/sitemap/0.9">
{{ range . }}
<sitemap>
<loc>{{ .SitemapAbsURL }}</loc>
{{ if not .LastChange.IsZero }}
<lastmod>{{ .LastChange.Format "2006-01-02T15:04:05-07:00" | safeHTML }}</lastmod>
{{ end }}
</sitemap>
{{ end }}
</sitemapindex>
`)
t.AddInternalTemplate("", "pagination.html", `{{ $pag := $.Paginator }}
{{ if gt $pag.TotalPages 1 }}
<ul class="pagination">