projects / brevno-suite / hugo / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: c1eb625)
Fix crashes for 404 in IsAncestor etc.
authorBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Sat, 22 Feb 2020 09:57:43 +0000 (10:57 +0100)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Sat, 22 Feb 2020 12:36:04 +0000 (13:36 +0100)
Fixes #6931

docs/content/en/readfiles/sectionvars.md patch | blob | history
hugolib/404_test.go patch | blob | history
hugolib/content_map_page.go patch | blob | history
hugolib/page__tree.go patch | blob | history

diff --git a/docs/content/en/readfiles/sectionvars.md b/docs/content/en/readfiles/sectionvars.md
index 698955c023bef0f301d4486f50e3e6e79ff5ccd4..45aaff1f3f425544e247ac3852118aeeb41960d2 100644 (file)
--- a/docs/content/en/readfiles/sectionvars.md
+++ b/docs/content/en/readfiles/sectionvars.md
@@ -5,13 +5,13 @@
 : The page's first section below root, e.g. `/docs`, `/blog` etc.
 
 .InSection $anotherPage
-: Whether the given page is in the current section. Note that this will always return false for pages that are not either regular, home or section pages.
+: Whether the given page is in the current section.
 
 .IsAncestor $anotherPage
-: Whether the current page is an ancestor of the given page. Note that this method is not relevant for taxonomy lists and taxonomy terms pages.
+: Whether the current page is an ancestor of the given page.
 
 .IsDescendant $anotherPage
-: Whether the current page is a descendant of the given page. Note that this method is not relevant for taxonomy lists and taxonomy terms pages.
+: Whether the current page is a descendant of the given page.
 
 .Parent
 : A section's parent section or a page's section.
diff --git a/hugolib/404_test.go b/hugolib/404_test.go
index 6dea9295cdb5b1b1ed1e90a7a8f9c67cd0fec1ab..cd203a6698b400f68db4f35f1accc5107ae36c26 100644 (file)
--- a/hugolib/404_test.go
+++ b/hugolib/404_test.go
@@ -21,15 +21,41 @@ func Test404(t *testing.T) {
        t.Parallel()
 
        b := newTestSitesBuilder(t)
-       b.WithSimpleConfigFile().WithTemplatesAdded("404.html", "<html><body>Not Found! Parent: {{ .Parent.Kind }}</body></html>")
+       b.WithSimpleConfigFile().WithTemplatesAdded(
+               "404.html",
+               `
+{{ $home := site.Home }}
+404: 
+Parent: {{ .Parent.Kind }}
+IsAncestor: {{ .IsAncestor $home }}/{{ $home.IsAncestor . }}
+IsDescendant: {{ .IsDescendant $home }}/{{ $home.IsDescendant . }}
+CurrentSection: {{ .CurrentSection.Kind }}|
+FirstSection: {{ .FirstSection.Kind }}|
+InSection: {{ .InSection $home.Section }}|{{ $home.InSection . }}
+Sections: {{ len .Sections }}|
+Page: {{ .Page.RelPermalink }}|
+Data: {{ len .Data }}|
+
+`,
+       )
        b.Build(BuildCfg{})
 
        // Note: We currently have only 1 404 page. One might think that we should have
        // multiple, to follow the Custom Output scheme, but I don't see how that would work
        // right now.
        b.AssertFileContent("public/404.html", `
-Not Found
+
+  404:
 Parent: home
+IsAncestor: false/true
+IsDescendant: true/false
+CurrentSection: home|
+FirstSection: home|
+InSection: false|true
+Sections: 0|
+Page: /404.html|
+Data: 1|
+        
 `)
 
 }
diff --git a/hugolib/content_map_page.go b/hugolib/content_map_page.go
index 18b2ee1a775c7c168f12389896a6d0032824f8a9..143de535115b081e52e12468b1403676c8f3f634 100644 (file)
--- a/hugolib/content_map_page.go
+++ b/hugolib/content_map_page.go
@@ -806,6 +806,9 @@ func (b *pagesMapBucket) getPagesAndSections() page.Pages {
 
 func (b *pagesMapBucket) getSections() page.Pages {
        b.sectionsInit.Do(func() {
+               if b.owner.treeRef == nil {
+                       return
+               }
                b.sections = b.owner.treeRef.collectSections()
        })
 
diff --git a/hugolib/page__tree.go b/hugolib/page__tree.go
index 2c35289227b0d5ae32ff0d0944f7e6a302132906..bd7b586b982d60e9904dfbaabc3f2c224b8c4737 100644 (file)
--- a/hugolib/page__tree.go
+++ b/hugolib/page__tree.go
@@ -37,6 +37,15 @@ func (pt pageTree) IsAncestor(other interface{}) (bool, error) {
 
        ref1, ref2 := pt.p.getTreeRef(), tp.getTreeRef()
 
+       if ref1 == nil || ref2 == nil {
+               if ref1 == nil {
+                       // A 404 or other similar standalone page.
+                       return false, nil
+               }
+
+               return ref1.n.p.IsHome(), nil
+       }
+
        if !ref1.isSection() {
                return false, nil
        }
@@ -66,6 +75,15 @@ func (pt pageTree) IsDescendant(other interface{}) (bool, error) {
 
        ref1, ref2 := pt.p.getTreeRef(), tp.getTreeRef()
 
+       if ref1 == nil || ref2 == nil {
+               if ref2 == nil {
+                       // A 404 or other similar standalone page.
+                       return false, nil
+               }
+
+               return ref2.n.p.IsHome(), nil
+       }
+
        if !ref2.isSection() {
                return false, nil
        }
@@ -76,6 +94,9 @@ func (pt pageTree) IsDescendant(other interface{}) (bool, error) {
 
 func (pt pageTree) FirstSection() page.Page {
        ref := pt.p.getTreeRef()
+       if ref == nil {
+               return pt.p.s.home
+       }
        key := ref.key
        if !ref.isSection() {
                key = path.Dir(key)
@@ -99,6 +120,14 @@ func (pt pageTree) InSection(other interface{}) (bool, error) {
 
        ref1, ref2 := pt.p.getTreeRef(), tp.getTreeRef()
 
+       if ref1 == nil || ref2 == nil {
+               if ref1 == nil {
+                       // A 404 or other similar standalone page.
+                       return false, nil
+               }
+               return ref1.n.p.IsHome(), nil
+       }
+
        s1, _ := ref1.getCurrentSection()
        s2, _ := ref2.getCurrentSection()
 
@@ -123,7 +152,7 @@ func (pt pageTree) Parent() page.Page {
 
        tree := p.getTreeRef()
 
-       if pt.p.Kind() == page.KindTaxonomyTerm || tree == nil {
+       if tree == nil || pt.p.Kind() == page.KindTaxonomyTerm {
                return pt.p.s.home
        }
 
Unnamed repository; edit this file 'description' to name the repository.