diff options
author | Bjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com> | 2016-10-24 13:45:30 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-10-24 13:45:30 +0200 |
commit | a10b2cd372798c4e4b862f0ec03010d2aea2ff1e (patch) | |
tree | f768c420aac0008e4d118709e13fda278a7588c5 | |
parent | dffd7da07c3fb198acfa6c4664b53132c4cabe55 (diff) |
Avoid reading from Viper for path and URL funcs
The gain, given the "real sites benchmark" below, is obvious:
```
benchmark old ns/op new ns/op delta
BenchmarkHugo-4 14497594101 13084156335 -9.75%
benchmark old allocs new allocs delta
BenchmarkHugo-4 57404335 48282002 -15.89%
benchmark old bytes new bytes delta
BenchmarkHugo-4 9933505624 9721984424 -2.13%
```
Fixes #2495
-rw-r--r-- | helpers/configProvider.go | 39 | ||||
-rw-r--r-- | helpers/language.go | 21 | ||||
-rw-r--r-- | helpers/language_test.go | 32 | ||||
-rw-r--r-- | helpers/path.go | 16 | ||||
-rw-r--r-- | helpers/path_test.go | 18 | ||||
-rw-r--r-- | helpers/pathspec.go | 54 | ||||
-rw-r--r-- | helpers/pathspec_test.go | 45 | ||||
-rw-r--r-- | helpers/url.go | 28 | ||||
-rw-r--r-- | helpers/url_test.go | 12 | ||||
-rw-r--r-- | hugolib/hugo_sites_test.go | 1 | ||||
-rw-r--r-- | hugolib/node.go | 2 | ||||
-rw-r--r-- | hugolib/page.go | 9 | ||||
-rw-r--r-- | hugolib/pageSort_test.go | 5 | ||||
-rw-r--r-- | hugolib/page_permalink_test.go | 6 | ||||
-rw-r--r-- | hugolib/page_test.go | 3 | ||||
-rw-r--r-- | hugolib/pagination.go | 6 | ||||
-rw-r--r-- | hugolib/pagination_test.go | 4 | ||||
-rw-r--r-- | hugolib/permalinks.go | 8 | ||||
-rw-r--r-- | hugolib/permalinks_test.go | 4 | ||||
-rw-r--r-- | hugolib/site.go | 83 | ||||
-rw-r--r-- | hugolib/taxonomy.go | 2 | ||||
-rw-r--r-- | tpl/template.go | 4 | ||||
-rw-r--r-- | tpl/template_funcs.go | 16 | ||||
-rw-r--r-- | tpl/template_funcs_test.go | 8 | ||||
-rw-r--r-- | tpl/template_i18n.go | 8 | ||||
-rw-r--r-- | tpl/template_i18n_test.go | 4 |
26 files changed, 348 insertions, 90 deletions
diff --git a/helpers/configProvider.go b/helpers/configProvider.go index 35500f6de..a631ea63d 100644 --- a/helpers/configProvider.go +++ b/helpers/configProvider.go @@ -21,16 +21,55 @@ import ( "github.com/spf13/viper" ) +// A cached version of the current ConfigProvider (language) and relatives. These globals +// are unfortunate, but we still have some places that needs this that does +// not have access to the site configuration. +// These values will be set on initialization when rendering a new language. +// +// TODO(bep) Get rid of these. +var ( + currentConfigProvider ConfigProvider + currentPathSpec *PathSpec +) + // ConfigProvider provides the configuration settings for Hugo. type ConfigProvider interface { GetString(key string) string GetInt(key string) int + GetBool(key string) bool GetStringMap(key string) map[string]interface{} GetStringMapString(key string) map[string]string + Get(key string) interface{} } // Config returns the currently active Hugo config. This will be set // per site (language) rendered. func Config() ConfigProvider { + if currentConfigProvider != nil { + return currentConfigProvider + } + // Some tests rely on this. We will fix that, eventually. return viper.Get("CurrentContentLanguage").(ConfigProvider) } + +// CurrentPathSpec returns the current PathSpec. +// If it is not set, a new will be created based in the currently active Hugo config. +func CurrentPathSpec() *PathSpec { + if currentPathSpec != nil { + return currentPathSpec + } + // Some tests rely on this. We will fix that, eventually. + return NewPathSpecFromConfig(Config()) +} + +// InitConfigProviderForCurrentContentLanguage does what it says. +func InitConfigProviderForCurrentContentLanguage() { + currentConfigProvider = viper.Get("CurrentContentLanguage").(ConfigProvider) + currentPathSpec = NewPathSpecFromConfig(currentConfigProvider) +} + +// ResetConfigProvider is used in tests. +func ResetConfigProvider() { + currentConfigProvider = nil + currentPathSpec = nil +} diff --git a/helpers/language.go b/helpers/language.go index 9ebec5a65..0a1affd18 100644 --- a/helpers/language.go +++ b/helpers/language.go @@ -23,6 +23,19 @@ import ( "github.com/spf13/viper" ) +// These are the settings that should only be looked up in the global Viper +// config and not per language. +// This list may not be complete, but contains only settings that we know +// will be looked up in both. +// This isn't perfect, but it is ultimately the user who shoots him/herself in +// the foot. +// See the pathSpec. +var globalOnlySettings = map[string]bool{ + strings.ToLower("defaultContentLanguageInSubdir"): true, + strings.ToLower("defaultContentLanguage"): true, + strings.ToLower("multilingual"): true, +} + type Language struct { Lang string LanguageName string @@ -81,7 +94,7 @@ func (l *Language) Params() map[string]interface{} { } func (l *Language) SetParam(k string, v interface{}) { - l.params[k] = v + l.params[strings.ToLower(k)] = v } func (l *Language) GetBool(key string) bool { return cast.ToBool(l.Get(key)) } @@ -101,8 +114,10 @@ func (l *Language) Get(key string) interface{} { panic("language not set") } key = strings.ToLower(key) - if v, ok := l.params[key]; ok { - return v + if !globalOnlySettings[key] { + if v, ok := l.params[key]; ok { + return v + } } return viper.Get(key) } diff --git a/helpers/language_test.go b/helpers/language_test.go new file mode 100644 index 000000000..f812d26a2 --- /dev/null +++ b/helpers/language_test.go @@ -0,0 +1,32 @@ +// Copyright 2016-present The Hugo Authors. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package helpers + +import ( + "testing" + + "github.com/spf13/viper" + "github.com/stretchr/testify/require" +) + +func TestGetGlobalOnlySetting(t *testing.T) { + lang := NewDefaultLanguage() + lang.SetParam("defaultContentLanguageInSubdir", false) + lang.SetParam("paginatePath", "side") + viper.Set("defaultContentLanguageInSubdir", true) + viper.Set("paginatePath", "page") + + require.True(t, lang.GetBool("defaultContentLanguageInSubdir")) + require.Equal(t, "side", lang.GetString("paginatePath")) +} diff --git a/helpers/path.go b/helpers/path.go index b8f642470..cf49858ee 100644 --- a/helpers/path.go +++ b/helpers/path.go @@ -75,16 +75,16 @@ var fpb filepathBridge // It does so by creating a Unicode-sanitized string, with the spaces replaced, // whilst preserving the original casing of the string. // E.g. Social Media -> Social-Media -func MakePath(s string) string { - return UnicodeSanitize(strings.Replace(strings.TrimSpace(s), " ", "-", -1)) +func (p *PathSpec) MakePath(s string) string { + return p.UnicodeSanitize(strings.Replace(strings.TrimSpace(s), " ", "-", -1)) } // MakePathSanitized creates a Unicode-sanitized string, with the spaces replaced -func MakePathSanitized(s string) string { - if viper.GetBool("DisablePathToLower") { - return MakePath(s) +func (p *PathSpec) MakePathSanitized(s string) string { + if p.disablePathToLower { + return p.MakePath(s) } - return strings.ToLower(MakePath(s)) + return strings.ToLower(p.MakePath(s)) } // MakeTitle converts the path given to a suitable title, trimming whitespace @@ -110,7 +110,7 @@ func ishex(c rune) bool { // a predefined set of special Unicode characters. // If RemovePathAccents configuration flag is enabled, Uniccode accents // are also removed. -func UnicodeSanitize(s string) string { +func (p *PathSpec) UnicodeSanitize(s string) string { source := []rune(s) target := make([]rune, 0, len(source)) @@ -124,7 +124,7 @@ func UnicodeSanitize(s string) string { var result string - if viper.GetBool("RemovePathAccents") { + if p.removePathAccents { // remove accents - see https://blog.golang.org/normalization t := transform.Chain(norm.NFD, transform.RemoveFunc(isMn), norm.NFC) result, _, _ = transform.String(t, string(target)) diff --git a/helpers/path_test.go b/helpers/path_test.go index cbdcd8da2..ef0c16505 100644 --- a/helpers/path_test.go +++ b/helpers/path_test.go @@ -33,9 +33,14 @@ import ( "github.com/spf13/viper" ) +func initCommonTestConfig() { + viper.Set("CurrentContentLanguage", NewLanguage("en")) +} + func TestMakePath(t *testing.T) { viper.Reset() defer viper.Reset() + initCommonTestConfig() tests := []struct { input string @@ -57,7 +62,8 @@ func TestMakePath(t *testing.T) { for _, test := range tests { viper.Set("RemovePathAccents", test.removeAccents) - output := MakePath(test.input) + p := NewPathSpecFromConfig(viper.GetViper()) + output := p.MakePath(test.input) if output != test.expected { t.Errorf("Expected %#v, got %#v\n", test.expected, output) } @@ -67,6 +73,9 @@ func TestMakePath(t *testing.T) { func TestMakePathSanitized(t *testing.T) { viper.Reset() defer viper.Reset() + initCommonTestConfig() + + p := NewPathSpecFromConfig(viper.GetViper()) tests := []struct { input string @@ -81,7 +90,7 @@ func TestMakePathSanitized(t *testing.T) { } for _, test := range tests { - output := MakePathSanitized(test.input) + output := p.MakePathSanitized(test.input) if output != test.expected { t.Errorf("Expected %#v, got %#v\n", test.expected, output) } @@ -91,7 +100,10 @@ func TestMakePathSanitized(t *testing.T) { func TestMakePathSanitizedDisablePathToLower(t *testing.T) { viper.Reset() defer viper.Reset() + + initCommonTestConfig() viper.Set("DisablePathToLower", true) + p := NewPathSpecFromConfig(viper.GetViper()) tests := []struct { input string @@ -106,7 +118,7 @@ func TestMakePathSanitizedDisablePathToLower(t *testing.T) { } for _, test := range tests { - output := MakePathSanitized(test.input) + output := p.MakePathSanitized(test.input) if output != test.expected { t.Errorf("Expected %#v, got %#v\n", test.expected, output) } diff --git a/helpers/pathspec.go b/helpers/pathspec.go new file mode 100644 index 000000000..4c578cdf7 --- /dev/null +++ b/helpers/pathspec.go @@ -0,0 +1,54 @@ +// Copyright 2016-present The Hugo Authors. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package helpers + +// PathSpec holds methods that decides how paths in URLs and files in Hugo should look like. +type PathSpec struct { + disablePathToLower bool + removePathAccents bool + uglyURLs bool + canonifyURLs bool + + currentContentLanguage *Language + + // pagination path handling + paginatePath string + + // The PathSpec looks up its config settings in both the current language + // and then in the global Viper config. + // Some settings, the settings listed below, does not make sense to be set + // on per-language-basis. We have no good way of protecting against this + // other than a "white-list". See language.go. + defaultContentLanguageInSubdir bool + defaultContentLanguage string + multilingual bool +} + +func NewPathSpecFromConfig(config ConfigProvider) *PathSpec { + return &PathSpec{ + disablePathToLower: config.GetBool("disablePathToLower"), + removePathAccents: config.GetBool("removePathAccents"), + uglyURLs: config.GetBool("uglyURLs"), + canonifyURLs: config.GetBool("canonifyURLs"), + multilingual: config.GetBool("multilingual"), + defaultContentLanguageInSubdir: config.GetBool("defaultContentLanguageInSubdir"), + defaultContentLanguage: config.GetString("defaultContentLanguage"), + currentContentLanguage: config.Get("currentContentLanguage").(*Language), + paginatePath: config.GetString("paginatePath"), + } +} + +func (p *PathSpec) PaginatePath() string { + return p.paginatePath +} diff --git a/helpers/pathspec_test.go b/helpers/pathspec_test.go new file mode 100644 index 000000000..9cd0af80e --- /dev/null +++ b/helpers/pathspec_test.go @@ -0,0 +1,45 @@ +// Copyright 2016-present The Hugo Authors. All rights reserved. +// +// Licensed under the Apache License, Version 2.0 (the "License"); +// you may not use this file except in compliance with the License. +// You may obtain a copy of the License at +// http://www.apache.org/licenses/LICENSE-2.0 +// +// Unless required by applicable law or agreed to in writing, software +// distributed under the License is distributed on an "AS IS" BASIS, +// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +// See the License for the specific language governing permissions and +// limitations under the License. + +package helpers + +import ( + "testing" + + "github.com/spf13/viper" + "github.com/stretchr/testify/require" +) + +func TestNewPathSpecFromConfig(t *testing.T) { + viper.Set("disablePathToLower", true) + viper.Set("removePathAccents", true) + viper.Set("uglyURLs", true) + viper.Set("multilingual", true) + viper.Set("defaultContentLanguageInSubdir", true) + viper.Set("defaultContentLanguage", "no") + viper.Set("currentContentLanguage", NewLanguage("no")) + viper.Set("canonifyURLs", true) + viper.Set("paginatePath", "side") + + pathSpec := NewPathSpecFromConfig(viper.GetViper()) + + require.True(t, pathSpec.canonifyURLs) + require.True(t, pathSpec.defaultContentLanguageInSubdir) + require.True(t, pathSpec.disablePathToLower) + require.True(t, pathSpec.multilingual) + require.True(t, pathSpec.removePathAccents) + require.True(t, pathSpec.uglyURLs) + require.Equal(t, "no", pathSpec.defaultContentLanguage) + require.Equal(t, "no", pathSpec.currentContentLanguage.Lang) + require.Equal(t, "side", pathSpec.paginatePath) +} diff --git a/helpers/url.go b/helpers/url.go index 49f670da2..ffb0ba234 100644 --- a/helpers/url.go +++ b/helpers/url.go @@ -101,8 +101,8 @@ func SanitizeURLKeepTrailingSlash(in string) string { // Example: // uri: Vim (text editor) // urlize: vim-text-editor -func URLize(uri string) string { - sanitized := MakePathSanitized(uri) +func (p *PathSpec) URLize(uri string) string { + sanitized := p.MakePathSanitized(uri) // escape unicode letters parsedURI, err := url.Parse(sanitized) @@ -147,7 +147,7 @@ func MakePermalink(host, plink string) *url.URL { } // AbsURL creates a absolute URL from the relative path given and the BaseURL set in config. -func AbsURL(in string, addLanguage bool) string { +func (p *PathSpec) AbsURL(in string, addLanguage bool) string { url, err := url.Parse(in) if err != nil { return in @@ -168,7 +168,7 @@ func AbsURL(in string, addLanguage bool) string { } if addLanguage { - prefix := getLanguagePrefix() + prefix := p.getLanguagePrefix() if prefix != "" { hasPrefix := false // avoid adding language prefix if already present @@ -191,15 +191,15 @@ func AbsURL(in string, addLanguage bool) string { return MakePermalink(baseURL, in).String() } -func getLanguagePrefix() string { - if !viper.GetBool("Multilingual") { +func (p *PathSpec) getLanguagePrefix() string { + if !p.multilingual { return "" } - defaultLang := viper.GetString("DefaultContentLanguage") - defaultInSubDir := viper.GetBool("DefaultContentLanguageInSubdir") + defaultLang := p.defaultContentLanguage + defaultInSubDir := p.defaultContentLanguageInSubdir - currentLang := viper.Get("CurrentContentLanguage").(*Language).Lang + currentLang := p.currentContentLanguage.Lang if currentLang == "" || (currentLang == defaultLang && !defaultInSubDir) { return "" } @@ -218,9 +218,9 @@ func IsAbsURL(path string) bool { // RelURL creates a URL relative to the BaseURL root. // Note: The result URL will not include the context root if canonifyURLs is enabled. -func RelURL(in string, addLanguage bool) string { +func (p *PathSpec) RelURL(in string, addLanguage bool) string { baseURL := viper.GetString("BaseURL") - canonifyURLs := viper.GetBool("canonifyURLs") + canonifyURLs := p.canonifyURLs if (!strings.HasPrefix(in, baseURL) && strings.HasPrefix(in, "http")) || strings.HasPrefix(in, "//") { return in } @@ -232,7 +232,7 @@ func RelURL(in string, addLanguage bool) string { } if addLanguage { - prefix := getLanguagePrefix() + prefix := p.getLanguagePrefix() if prefix != "" { hasPrefix := false // avoid adding language prefix if already present @@ -288,8 +288,8 @@ func AddContextRoot(baseURL, relativePath string) string { return newPath } -func URLizeAndPrep(in string) string { - return URLPrep(viper.GetBool("UglyURLs"), URLize(in)) +func (p *PathSpec) URLizeAndPrep(in string) string { + return URLPrep(p.uglyURLs, p.URLize(in)) } func URLPrep(ugly bool, in string) string { diff --git a/helpers/url_test.go b/helpers/url_test.go index 2cf93b859..5394e9573 100644 --- a/helpers/url_test.go +++ b/helpers/url_test.go @@ -24,6 +24,10 @@ import ( ) func TestURLize(t *testing.T) { + initCommonTestConfig() + + p := NewPathSpecFromConfig(viper.GetViper()) + tests := []struct { input string expected string @@ -37,7 +41,7 @@ func TestURLize(t *testing.T) { } for _, test := range tests { - output := URLize(test.input) + output := p.URLize(test.input) if output != test.expected { t.Errorf("Expected %#v, got %#v\n", test.expected, output) } @@ -83,7 +87,8 @@ func doTestAbsURL(t *testing.T, defaultInSubDir, addLanguage, multilingual bool, for _, test := range tests { viper.Set("BaseURL", test.baseURL) - output := AbsURL(test.input, addLanguage) + p := NewPathSpecFromConfig(viper.GetViper()) + output := p.AbsURL(test.input, addLanguage) expected := test.expected if multilingual && addLanguage { if !defaultInSubDir && lang == "en" { @@ -159,8 +164,9 @@ func doTestRelURL(t *testing.T, defaultInSubDir, addLanguage, multilingual bool, for i, test := range tests { viper.Set("BaseURL", test.baseURL) viper.Set("canonifyURLs", test.canonify) + p := NewPathSpecFromConfig(viper.GetViper()) - output := RelURL(test.input, addLanguage) + output := p.RelURL(test.input, addLanguage) expected := test.expected if multilingual && addLanguage { diff --git a/hugolib/hugo_sites_test.go b/hugolib/hugo_sites_test.go index b7a52b1e4..be2ba9803 100644 --- a/hugolib/hugo_sites_test.go +++ b/hugolib/hugo_sites_test.go @@ -35,6 +35,7 @@ func testCommonResetState() { hugofs.InitMemFs() viper.Reset() viper.SetFs(hugofs.Source()) + helpers.ResetConfigProvider() loadDefaultSettings() // Default is false, but true is easier to use as default in tests diff --git a/hugolib/node.go b/hugolib/node.go index eefb78bec..16c83e2e8 100644 --- a/hugolib/node.go +++ b/hugolib/node.go @@ -178,7 +178,7 @@ func (n *Node) URL() string { } func (n *Node) Permalink() string { - return permalink(n.URL()) + return n.Site.permalink(n.URL()) } // Scratch returns the writable context associated with this Node. diff --git a/hugolib/page.go b/hugolib/page.go index fee7c334a..484425697 100644 --- a/hugolib/page.go +++ b/hugolib/page.go @@ -569,9 +569,9 @@ func (p *Page) analyzePage() { func (p *Page) permalink() (*url.URL, error) { baseURL := string(p.Site.BaseURL) - dir := strings.TrimSpace(helpers.MakePath(filepath.ToSlash(strings.ToLower(p.Source.Dir())))) - pSlug := strings.TrimSpace(helpers.URLize(p.Slug)) - pURL := strings.TrimSpace(helpers.URLize(p.URLPath.URL)) + dir := strings.TrimSpace(p.Site.pathSpec.MakePath(filepath.ToSlash(strings.ToLower(p.Source.Dir())))) + pSlug := strings.TrimSpace(p.Site.pathSpec.URLize(p.Slug)) + pURL := strings.TrimSpace(p.Site.pathSpec.URLize(p.URLPath.URL)) var permalink string var err error @@ -1171,5 +1171,6 @@ func (p *Page) TargetPath() (outfile string) { outfile = helpers.ReplaceExtension(p.Source.TranslationBaseName(), p.Extension()) } - return p.addLangFilepathPrefix(filepath.Join(strings.ToLower(helpers.MakePath(p.Source.Dir())), strings.TrimSpace(outfile))) + return p.addLangFilepathPrefix(filepath.Join(strings.ToLower( + p.Site.pathSpec.MakePath(p.Source.Dir())), strings.TrimSpace(outfile))) } diff --git a/hugolib/pageSort_test.go b/hugolib/pageSort_test.go index 23a3fd07c..738fcc80e 100644 --- a/hugolib/pageSort_test.go +++ b/hugolib/pageSort_test.go @@ -20,6 +20,7 @@ import ( "testing" "time" + "github.com/spf13/hugo/helpers" "github.com/spf13/hugo/source" "github.com/stretchr/testify/assert" ) @@ -134,6 +135,8 @@ func setSortVals(dates [3]time.Time, titles [3]string, weights [3]int, pages Pag func createSortTestPages(num int) Pages { pages := make(Pages, num) + info := newSiteInfo(siteBuilderCfg{baseURL: "http://base", language: helpers.NewDefaultLanguage()}) + for i := 0; i < num; i++ { pages[i] = &Page{ Node: Node{ @@ -141,7 +144,7 @@ func createSortTestPages(num int) Pages { Section: "z", URL: fmt.Sprintf("http://base/x/y/p%d.html", i), }, - Site: newSiteInfoDefaultLanguage("http://base/"), + Site: &info, }, Source: Source{File: *source.NewFile(filepath.FromSlash(fmt.Sprintf("/x/y/p%d.md", i)))}, } diff --git a/hugolib/page_permalink_test.go b/hugolib/page_permalink_test.go index d185bebab..eb3a6c878 100644 --- a/hugolib/page_permalink_test.go +++ b/hugolib/page_permalink_test.go @@ -18,6 +18,7 @@ import ( "path/filepath" "testing" + "github.com/spf13/hugo/helpers" "github.com/spf13/hugo/source" "github.com/spf13/viper" ) @@ -59,17 +60,18 @@ func TestPermalink(t *testing.T) { } viper.Set("DefaultExtension", "html") - for i, test := range tests { viper.Set("uglyurls", test.uglyURLs) viper.Set("canonifyurls", test.canonifyURLs) + info := newSiteInfo(siteBuilderCfg{baseURL: string(test.base), language: helpers.NewDefaultLanguage()}) + p := &Page{ Node: Node{ URLPath: URLPath{ Section: "z", URL: test.url, }, - Site: newSiteInfoDefaultLanguage(string(test.base)), + Site: &info, }, Source: Source{File: *source.NewFile(filepath.FromSlash(test.file))}, } diff --git a/hugolib/page_test.go b/hugolib/page_test.go index 0bb924f01..342ff3bd5 100644 --- a/hugolib/page_test.go +++ b/hugolib/page_test.go @@ -1122,7 +1122,8 @@ func TestPagePaths(t *testing.T) { for _, test := range tests { p, _ := NewPageFrom(strings.NewReader(test.content), filepath.FromSlash(test.path)) - p.Node.Site = newSiteInfoDefaultLanguage("") + info := newSiteInfo(siteBuilderCfg{language: helpers.NewDefaultLanguage()}) + p.Node.Site = &info if test.hasPermalink { p.Node.Site.Permalinks = siteParmalinksSetting diff --git a/hugolib/pagination.go b/hugolib/pagination.go index 5bba5d89f..81f3e0cda 100644 --- a/hugolib/pagination.go +++ b/hugolib/pagination.go @@ -508,16 +508,16 @@ func newPaginator(elements []paginatedElement, total, size int, urlFactory pagin } func newPaginationURLFactory(pathElements ...string) paginationURLFactory { - paginatePath := helpers.Config().GetString("paginatePath") + pathSpec := helpers.CurrentPathSpec() return func(page int) string { var rel string if page == 1 { rel = fmt.Sprintf("/%s/", path.Join(pathElements...)) } else { - rel = fmt.Sprintf("/%s/%s/%d/", path.Join(pathElements...), paginatePath, page) + rel = fmt.Sprintf("/%s/%s/%d/", path.Join(pathElements...), pathSpec.PaginatePath(), page) } - return helpers.URLizeAndPrep(rel) + return pathSpec.URLizeAndPrep(rel) } } diff --git a/hugolib/pagination_test.go b/hugolib/pagination_test.go index 786650469..7bbc38058 100644 --- a/hugolib/pagination_test.go +++ b/hugolib/pagination_test.go @@ -19,6 +19,7 @@ import ( "path/filepath" "testing" + "github.com/spf13/hugo/helpers" "github.com/spf13/hugo/source" "github.com/spf13/viper" "github.com/stretchr/testify/assert" |