Welcome to mirror list, hosted at ThFree Co, Russian Federation.

github.com/gohugoio/hugo.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>2022-01-28 11:45:11 +0300
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>2022-01-28 19:35:49 +0300
commitef7d14a2418e62e6a2fdde41c636f7c1b9300a02 (patch)
tree6a7a8a83a616e0a6d296888fb32ca489ef385142
parentc05c99f0c552553914689cb6a29edb5ac8b4f88d (diff)
Fix erroneous warning with .Page.RenderString on a page without a backing file
Fixes #9433
-rw-r--r--hugolib/content_render_hooks_test.go13
-rw-r--r--hugolib/page__meta.go6
2 files changed, 18 insertions, 1 deletions
diff --git a/hugolib/content_render_hooks_test.go b/hugolib/content_render_hooks_test.go
index f1c27d511..edfeaa82a 100644
--- a/hugolib/content_render_hooks_test.go
+++ b/hugolib/content_render_hooks_test.go
@@ -18,6 +18,7 @@ import (
"testing"
qt "github.com/frankban/quicktest"
+ "github.com/gohugoio/hugo/common/loggers"
)
func TestRenderHookEditNestedPartial(t *testing.T) {
@@ -484,3 +485,15 @@ func TestRenderStringOnListPage(t *testing.T) {
b.AssertFileContent("public/"+filename, `<strong>Hello</strong>`)
}
}
+
+// Issue 9433
+func TestRenderStringOnPageNotBackedByAFile(t *testing.T) {
+ t.Parallel()
+ logger := loggers.NewWarningLogger()
+ b := newTestSitesBuilder(t).WithLogger(logger).WithConfigFile("toml", `
+disableKinds = ["page", "section", "taxonomy", "term"]
+`)
+ b.WithTemplates("index.html", `{{ .RenderString "**Hello**" }}`).WithContent("p1.md", "")
+ b.BuildE(BuildCfg{})
+ b.Assert(int(logger.LogCounters().WarnCounter.Count()), qt.Equals, 0)
+}
diff --git a/hugolib/page__meta.go b/hugolib/page__meta.go
index 5581b6ebd..dcbf1a606 100644
--- a/hugolib/page__meta.go
+++ b/hugolib/page__meta.go
@@ -768,16 +768,20 @@ func (p *pageMeta) newContentConverter(ps *pageState, markup string, renderingCo
var id string
var filename string
+ var path string
if !p.f.IsZero() {
id = p.f.UniqueID()
filename = p.f.Filename()
+ path = p.f.Path()
+ } else {
+ path = p.Pathc()
}
cpp, err := cp.New(
converter.DocumentContext{
Document: newPageForRenderHook(ps),
DocumentID: id,
- DocumentName: p.File().Path(),
+ DocumentName: path,
Filename: filename,
ConfigOverrides: renderingConfigOverrides,
},