Do not render alias paginator pages for non-HTML outputs

Update #6797

diff --git a/hugolib/paginator_test.go b/hugolib/paginator_test.go
index a97a59d04c853c5349807ed0cbe26169d6fa528a..e6a196150656f471ec1f577d1a4b89ab4876f97e 100644 (file)
--- a/hugolib/paginator_test.go
+++ b/hugolib/paginator_test.go
@@ -17,6 +17,8 @@ import (
        "fmt"
        "path/filepath"
        "testing"
+
+       qt "github.com/frankban/quicktest"
 )
 
 func TestPaginator(t *testing.T) {
@@ -105,3 +107,34 @@ func TestPaginateWithSort(t *testing.T) {
        b.Build(BuildCfg{}).AssertFileContent("public/index.html",
                filepath.FromSlash("|content/sect/doc1.nn.md|content/sect/doc1.nb.md|content/sect/doc1.fr.md|content/sect/doc1.en.md"))
 }
+
+// https://github.com/gohugoio/hugo/issues/6797
+func TestPaginateOutputFormat(t *testing.T) {
+       b := newTestSitesBuilder(t).WithSimpleConfigFile()
+       b.WithContent("_index.md", `---
+title: "Home"
+cascade:
+  outputs:
+    - JSON
+---`)
+
+       for i := 0; i < 22; i++ {
+               b.WithContent(fmt.Sprintf("p%d.md", i+1), fmt.Sprintf(`---
+title: "Page"
+weight: %d
+---`, i+1))
+       }
+
+       b.WithTemplatesAdded("index.json", `JSON: {{ .Paginator.TotalNumberOfElements }}: {{ range .Paginator.Pages }}|{{ .RelPermalink }}{{ end }}:DONE`)
+       b.Build(BuildCfg{})
+
+       b.AssertFileContent("public/index.json",
+               `JSON: 22
+|/p1/index.json|/p2/index.json|
+`)
+
+       // This looks odd, so are most bugs.
+       b.Assert(b.CheckExists("public/page/1/index.json/index.html"), qt.Equals, false)
+       b.Assert(b.CheckExists("public/page/1/index.json"), qt.Equals, false)
+       b.AssertFileContent("public/page/2/index.json", `JSON: 22: |/p11/index.json|/p12/index.json`)
+}

diff --git a/hugolib/site_render.go b/hugolib/site_render.go
index 59f26599628335029afc73ee5fbf66b067379149..dd79a726497c34cd40983b0d9b1df829044a24eb 100644 (file)
--- a/hugolib/site_render.go
+++ b/hugolib/site_render.go
@@ -205,12 +205,14 @@ func (s *Site) renderPaginator(p *pageState, templ tpl.Template) error {
                panic(fmt.Sprintf("invalid paginator state for %q", p.pathOrTitle()))
        }
 
-       // Write alias for page 1
-       d.Addends = fmt.Sprintf("/%s/%d", paginatePath, 1)
-       targetPaths := page.CreateTargetPaths(d)
+       if f.IsHTML {
+               // Write alias for page 1
+               d.Addends = fmt.Sprintf("/%s/%d", paginatePath, 1)
+               targetPaths := page.CreateTargetPaths(d)
 
-       if err := s.writeDestAlias(targetPaths.TargetFilename, p.Permalink(), f, nil); err != nil {
-               return err
+               if err := s.writeDestAlias(targetPaths.TargetFilename, p.Permalink(), f, nil); err != nil {
+                       return err
+               }
        }
 
        // Render pages for the rest