phoenix/hugo
1
0
Fork 0
mirror of https://github.com/gohugoio/hugo.git synced 2024-07-07 09:36:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix URLs for bundle resources in multihost mode

Browse source 
Fixes #4217
This commit is contained in:
Bjørn Erik Pedersen 2018-01-05 12:01:13 +01:00
parent f25d8a9e17
commit ab82a27d05
6 changed files with 42 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
helpers/pathspec.go
View file

@ -35,8 +35,8 @@ type PathSpec struct {
uglyURLs bool uglyURLs bool
canonifyURLs bool canonifyURLs bool
language *Language Language *Language
languages Languages Languages Languages
// pagination path handling // pagination path handling
paginatePath string paginatePath string
@ -70,7 +70,7 @@ type PathSpec struct {
} }
func (p PathSpec) String() string { func (p PathSpec) String() string {
return fmt.Sprintf("PathSpec, language %q, prefix %q, multilingual: %T", p.language.Lang, p.getLanguagePrefix(), p.multilingual) return fmt.Sprintf("PathSpec, language %q, prefix %q, multilingual: %T", p.Language.Lang, p.getLanguagePrefix(), p.multilingual)
} }
// NewPathSpec creats a new PathSpec from the given filesystems and Language. // NewPathSpec creats a new PathSpec from the given filesystems and Language.
@ -113,8 +113,8 @@ func NewPathSpec(fs *hugofs.Fs, cfg config.Provider) (*PathSpec, error) {
uglyURLs: cfg.GetBool("uglyURLs"), uglyURLs: cfg.GetBool("uglyURLs"),
canonifyURLs: cfg.GetBool("canonifyURLs"), canonifyURLs: cfg.GetBool("canonifyURLs"),
multilingual: cfg.GetBool("multilingual"), multilingual: cfg.GetBool("multilingual"),
language: language, Language: language,
languages: languages, Languages: languages,
defaultContentLanguageInSubdir: cfg.GetBool("defaultContentLanguageInSubdir"), defaultContentLanguageInSubdir: cfg.GetBool("defaultContentLanguageInSubdir"),
defaultContentLanguage: cfg.GetString("defaultContentLanguage"), defaultContentLanguage: cfg.GetString("defaultContentLanguage"),
paginatePath: cfg.GetString("paginatePath"), paginatePath: cfg.GetString("paginatePath"),

2
helpers/pathspec_test.go
View file

@ -50,7 +50,7 @@ func TestNewPathSpecFromConfig(t *testing.T) {
require.True(t, p.removePathAccents) require.True(t, p.removePathAccents)
require.True(t, p.uglyURLs) require.True(t, p.uglyURLs)
require.Equal(t, "no", p.defaultContentLanguage) require.Equal(t, "no", p.defaultContentLanguage)
require.Equal(t, "no", p.language.Lang) require.Equal(t, "no", p.Language.Lang)
require.Equal(t, "side", p.paginatePath) require.Equal(t, "side", p.paginatePath)
require.Equal(t, "http://base.com", p.BaseURL.String()) require.Equal(t, "http://base.com", p.BaseURL.String())

4
helpers/url.go
View file

@ -208,7 +208,7 @@ func (p *PathSpec) getLanguagePrefix() string {
defaultLang := p.defaultContentLanguage defaultLang := p.defaultContentLanguage
defaultInSubDir := p.defaultContentLanguageInSubdir defaultInSubDir := p.defaultContentLanguageInSubdir
currentLang := p.language.Lang currentLang := p.Language.Lang
if currentLang == "" || (currentLang == defaultLang && !defaultInSubDir) { if currentLang == "" || (currentLang == defaultLang && !defaultInSubDir) {
return "" return ""
} }
@ -221,7 +221,7 @@ func (p *PathSpec) GetLangSubDir(lang string) string {
return "" return ""
} }
if p.languages.IsMultihost() { if p.Languages.IsMultihost() {
return "" return ""
} }

2
hugolib/hugo_sites_build_test.go
View file

@ -394,7 +394,7 @@ func doTestMultiSitesBuild(t *testing.T, configTemplate, configSuffix string) {
} }
// Check bundles // Check bundles
bundleFr := enSite.getPage(KindPage, "bundles/b1/index.md") bundleFr := frSite.getPage(KindPage, "bundles/b1/index.md")
require.NotNil(t, bundleFr) require.NotNil(t, bundleFr)
require.Equal(t, "/blog/fr/bundles/b1/", bundleFr.RelPermalink()) require.Equal(t, "/blog/fr/bundles/b1/", bundleFr.RelPermalink())
require.Equal(t, 1, len(bundleFr.Resources)) require.Equal(t, 1, len(bundleFr.Resources))

35
hugolib/hugo_sites_multihost_test.go
View file

@ -1,6 +1,7 @@
package hugolib package hugolib
import ( import (
"path/filepath"
"testing" "testing"
"github.com/spf13/afero" "github.com/spf13/afero"
@ -26,7 +27,7 @@ tag = "tags"
[Languages] [Languages]
[Languages.en] [Languages.en]
staticDir2 = ["ens1", "ens2"] staticDir2 = ["ens1", "ens2"]
baseURL = "https://example.com" baseURL = "https://example.com/docs"
weight = 10 weight = 10
title = "In English" title = "In English"
languageName = "English" languageName = "English"
@ -65,7 +66,7 @@ languageName = "Nynorsk"
s1h := s1.getPage(KindHome) s1h := s1.getPage(KindHome)
assert.True(s1h.IsTranslated()) assert.True(s1h.IsTranslated())
assert.Len(s1h.Translations(), 2) assert.Len(s1h.Translations(), 2)
assert.Equal("https://example.com/", s1h.Permalink()) assert.Equal("https://example.com/docs/", s1h.Permalink())
// For “regular multilingual” we kept the aliases pages with url in front matter // For “regular multilingual” we kept the aliases pages with url in front matter
// as a literal value that we use as is. // as a literal value that we use as is.
@ -76,12 +77,12 @@ languageName = "Nynorsk"
pageWithURLInFrontMatter := s1.getPage(KindPage, "sect/doc3.en.md") pageWithURLInFrontMatter := s1.getPage(KindPage, "sect/doc3.en.md")
assert.NotNil(pageWithURLInFrontMatter) assert.NotNil(pageWithURLInFrontMatter)
assert.Equal("/superbob", pageWithURLInFrontMatter.URL()) assert.Equal("/superbob", pageWithURLInFrontMatter.URL())
assert.Equal("/superbob/", pageWithURLInFrontMatter.RelPermalink()) assert.Equal("/docs/superbob/", pageWithURLInFrontMatter.RelPermalink())
th.assertFileContent("public/en/superbob/index.html", "doc3|Hello|en") th.assertFileContent("public/en/superbob/index.html", "doc3|Hello|en")
// check alias: // check alias:
th.assertFileContent("public/en/al/alias1/index.html", `content="0; url=https://example.com/superbob/"`) th.assertFileContent("public/en/al/alias1/index.html", `content="0; url=https://example.com/docs/superbob/"`)
th.assertFileContent("public/en/al/alias2/index.html", `content="0; url=https://example.com/superbob/"`) th.assertFileContent("public/en/al/alias2/index.html", `content="0; url=https://example.com/docs/superbob/"`)
s2 := sites.Sites[1] s2 := sites.Sites[1]
assert.Equal([]string{"s1", "s2", "frs1", "frs2"}, s2.StaticDirs()) assert.Equal([]string{"s1", "s2", "frs1", "frs2"}, s2.StaticDirs())
@ -93,9 +94,29 @@ languageName = "Nynorsk"
th.assertFileContentStraight("public/en/index.html", "Default Home Page") th.assertFileContentStraight("public/en/index.html", "Default Home Page")
// Check paginators // Check paginators
th.assertFileContent("public/en/page/1/index.html", `refresh" content="0; url=https://example.com/"`) th.assertFileContent("public/en/page/1/index.html", `refresh" content="0; url=https://example.com/docs/"`)
th.assertFileContent("public/nn/page/1/index.html", `refresh" content="0; url=https://example.no/"`) th.assertFileContent("public/nn/page/1/index.html", `refresh" content="0; url=https://example.no/"`)
th.assertFileContent("public/en/sect/page/2/index.html", "List Page 2", "Hello", "https://example.com/sect/", "\"/sect/page/3/") th.assertFileContent("public/en/sect/page/2/index.html", "List Page 2", "Hello", "https://example.com/docs/sect/", "\"/docs/sect/page/3/")
th.assertFileContent("public/fr/sect/page/2/index.html", "List Page 2", "Bonjour", "https://example.fr/sect/") th.assertFileContent("public/fr/sect/page/2/index.html", "List Page 2", "Bonjour", "https://example.fr/sect/")
// Check bundles
bundleEn := s1.getPage(KindPage, "bundles/b1/index.en.md")
require.NotNil(t, bundleEn)
require.Equal(t, "/docs/bundles/b1/", bundleEn.RelPermalink())
require.Equal(t, 1, len(bundleEn.Resources))
logoEn := bundleEn.Resources.GetByPrefix("logo")
require.NotNil(t, logoEn)
require.Equal(t, "/docs/bundles/b1/logo.png", logoEn.RelPermalink())
require.Contains(t, readFileFromFs(t, fs.Destination, filepath.FromSlash("public/en/bundles/b1/logo.png")), "PNG Data")
bundleFr := s2.getPage(KindPage, "bundles/b1/index.md")
require.NotNil(t, bundleFr)
require.Equal(t, "/bundles/b1/", bundleFr.RelPermalink())
require.Equal(t, 1, len(bundleFr.Resources))
logoFr := bundleFr.Resources.GetByPrefix("logo")
require.NotNil(t, logoFr)
require.Equal(t, "/bundles/b1/logo.png", logoFr.RelPermalink())
require.Contains(t, readFileFromFs(t, fs.Destination, filepath.FromSlash("public/fr/bundles/b1/logo.png")), "PNG Data")
} }

6
resource/resource.go
View file

@ -273,7 +273,11 @@ func (l *genericResource) Publish() error {
} }
func (l *genericResource) target() string { func (l *genericResource) target() string {
return l.relPermalinkForRel(l.rel, false) target := l.relPermalinkForRel(l.rel, false)
if l.spec.PathSpec.Languages.IsMultihost() {
target = path.Join(l.spec.PathSpec.Language.Lang, target)
}
return target
} }
func (r *Spec) newGenericResource( func (r *Spec) newGenericResource(