hugolib, output: Restrict Render to regular Pages

Using it for list pages doesn't work and has potential weird side-effects.

The user probably meant to range over .Site.ReqularPages, and that is now marked clearly in the log.

4 files changed, 33 insertions, 9 deletions

diff --git a/hugolib/page.go b/hugolib/page.go
index 84dfa2f43..fa9f40922 100644
--- a/hugolib/page.go
+++ b/hugolib/page.go
@@ -1721,3 +1721,11 @@ func (p *Page) setValuesForKind(s *Site) {
 		p.URLPath.URL = "/" + path.Join(p.sections...) + "/"
 	}
 }
+
+// Used in error logs.
+func (p *Page) pathOrTitle() string {
+	if p.Path() != "" {
+		return p.Path()
+	}
+	return p.Title
+}
diff --git a/hugolib/page_output.go b/hugolib/page_output.go
index ed0678964..837e41549 100644
--- a/hugolib/page_output.go
+++ b/hugolib/page_output.go
@@ -21,6 +21,7 @@ import (
 
 	"github.com/spf13/hugo/media"
 
+	"github.com/spf13/hugo/helpers"
 	"github.com/spf13/hugo/output"
 )
 
@@ -85,9 +86,9 @@ func (p *PageOutput) copy() *PageOutput {
 	return c
 }
 
-func (p *PageOutput) layouts(layouts ...string) []string {
+func (p *PageOutput) layouts(layouts ...string) ([]string, error) {
 	if len(layouts) == 0 && p.selfLayout != "" {
-		return []string{p.selfLayout}
+		return []string{p.selfLayout}, nil
 	}
 
 	layoutOverride := ""
@@ -106,7 +107,11 @@ func (p *PageOutput) Render(layout ...string) template.HTML {
 		return template.HTML("")
 	}
 
-	l := p.layouts(layout...)
+	l, err := p.layouts(layout...)
+	if err != nil {
+		helpers.DistinctErrorLog.Printf("in .Render: Failed to resolve layout %q for page %q", layout, p.pathOrTitle())
+		return template.HTML("")
+	}
 	return p.s.Tmpl.ExecuteTemplateToHTML(p, l...)
 }
 
@@ -122,7 +127,7 @@ func (p *Page) Render(layout ...string) template.HTML {
 		pageOutput, err := newPageOutput(p, true, outFormat)
 
 		if err != nil {
-			p.s.Log.ERROR.Printf("Failed to create output page for type %q for page %q: %s", outFormat.Name, p, err)
+			p.s.Log.ERROR.Printf("Failed to create output page for type %q for page %q: %s", outFormat.Name, p.pathOrTitle(), err)
 			return
 		}
 
@@ -137,7 +142,7 @@ func (p *Page) Render(layout ...string) template.HTML {
 // for list pages.
 func (p *Page) checkRender() bool {
 	if p.Kind != KindPage {
-		p.s.Log.ERROR.Printf(".Render only available for regular pages, not for %q of kind %q", p.Path(), p.Kind)
+		helpers.DistinctErrorLog.Printf(".Render only available for regular pages, not for of kind %q. You probably meant .Site.RegularPages and not.Site.Pages.", p.Kind)
 		return false
 	}
 	return true
diff --git a/hugolib/site.go b/hugolib/site.go
index 9d88d7605..4aa68aacf 100644
--- a/hugolib/site.go
+++ b/hugolib/site.go
@@ -1668,7 +1668,7 @@ func (s *Site) kindFromSections(sections []string) string {
 	return KindSection
 }
 
-func (s *Site) layouts(p *PageOutput) []string {
+func (s *Site) layouts(p *PageOutput) ([]string, error) {
 	return s.layoutHandler.For(p.layoutDescriptor, "", p.outputFormat)
 }
 
diff --git a/hugolib/site_render.go b/hugolib/site_render.go
index 330601013..700912e65 100644
--- a/hugolib/site_render.go
+++ b/hugolib/site_render.go
@@ -91,7 +91,11 @@ func pageRenderer(s *Site, pages <-chan *Page, results chan<- error, wg *sync.Wa
 			if page.selfLayout != "" {
 				layouts = []string{page.selfLayout}
 			} else {
-				layouts = s.layouts(pageOutput)
+				layouts, err = s.layouts(pageOutput)
+				if err != nil {
+					s.Log.ERROR.Printf("Failed to resolve layout output %q for page %q: %s", outFormat.Name, page, err)
+					continue
+				}
 			}
 
 			switch pageOutput.outputFormat.Name {
@@ -161,7 +165,11 @@ func (s *Site) renderPaginator(p *PageOutput) error {
 			pageNumber := i + 1
 			addend := fmt.Sprintf("/%s/%d", paginatePath, pageNumber)
 			targetPath, _ := p.targetPath(addend)
-			layouts := p.layouts()
+			layouts, err := p.layouts()
+
+			if err != nil {
+				return err
+			}
 
 			if err := s.renderAndWritePage(
 				pagerNode.Title,
@@ -200,10 +208,13 @@ func (s *Site) renderRSS(p *PageOutput) error {
 		p.Data["Pages"] = p.Pages
 	}
 
-	layouts := s.layoutHandler.For(
+	layouts, err := s.layoutHandler.For(
 		p.layoutDescriptor,
 		"",
 		p.outputFormat)
+	if err != nil {
+		return err
+	}
 
 	// TODO(bep) output deprecate/handle rssURI
 	targetPath, err := p.targetPath()