hugolib: Fix relative .Page.GetPage from bundle

Fixes #6705

diff --git a/hugolib/hugo_modules_test.go b/hugolib/hugo_modules_test.go
index 1c665e6e3716feeefc00961d2b997370bc0e2195..14085e2c09c20be56c88ed28cb3e23d0cdd35ed3 100644 (file)
--- a/hugolib/hugo_modules_test.go
+++ b/hugolib/hugo_modules_test.go
@@ -585,9 +585,19 @@ workingDir = %q
 {{ $mypage := .Site.GetPage "/blog/mypage.md" }}
 {{ with $mypage }}MYPAGE: {{ .Title }}|Path: {{ path.Join .File.Path }}|FilePath: {{ path.Join .File.FileInfo.Meta.PathFile }}|{{ end }}
 
+`, "_default/_markup/render-link.html", `
+{{ $link := .Destination }}
+{{ $isRemote := strings.HasPrefix $link "http" }}
+{{- if not $isRemote -}}
+{{ $url := urls.Parse .Destination }}
+{{ $fragment := "" }}
+{{- with $url.Fragment }}{{ $fragment = printf "#%s" . }}{{ end -}}
+{{- with .Page.GetPage $url.Path }}{{ $link = printf "%s%s" .Permalink $fragment }}{{ end }}{{ end -}}
+<a href="{{ $link | safeURL }}"{{ with .Title}} title="{{ . }}"{{ end }}{{ if $isRemote }} target="_blank"{{ end }}>{{ .Text | safeHTML }}</a>
 `)
 
        os.Mkdir(filepath.Join(workingDir, "mycontent"), 0777)
+       os.Mkdir(filepath.Join(workingDir, "mycontent", "mybundle"), 0777)
 
        b.WithSourceFile("README.md", `---
 title: "Readme Title"
@@ -600,6 +610,23 @@ Readme Content.
 title: "My Page"
 ---
 
+
+* [Relative Link From Page](mybundle)
+* [Relative Link From Page, filename](mybundle/index.md)
+* [Link using original path](/mycontent/mybundle/index.md)
+
+
+`, filepath.Join("mycontent", "mybundle", "index.md"), `
+---
+title: "My Bundle"
+---
+
+* [Dot Relative Link From Bundle](../mypage.md)
+* [Link using original path](/mycontent/mypage.md)
+* [Link to Home](/)
+* [Link to Home, README.md](/README.md)
+* [Link to Home, _index.md](/_index.md)
+
 `)
 
        b.Build(BuildCfg{})
@@ -610,7 +637,19 @@ README: Readme Title
 Readme Content.
 MYPAGE: My Page|Path: blog/mypage.md|FilePath: mycontent/mypage.md|
 `)
-       b.AssertFileContent("public/blog/mypage/index.html", "Single: My Page")
+       b.AssertFileContent("public/blog/mypage/index.html", `
+<a href="https://example.com/blog/mybundle/">Relative Link From Page</a>
+<a href="https://example.com/blog/mybundle/">Relative Link From Page, filename</a>
+<a href="https://example.com/blog/mybundle/">Link using original path</a>
+
+`)
+       b.AssertFileContent("public/blog/mybundle/index.html", `
+<a href="https://example.com/blog/mypage/">Dot Relative Link From Bundle</a>
+<a href="https://example.com/blog/mypage/">Link using original path</a>
+<a href="https://example.com/">Link to Home</a>
+<a href="https://example.com/">Link to Home, README.md</a>
+<a href="https://example.com/">Link to Home, _index.md</a>
+`)
 
        b.EditFiles("README.md", `---
 title: "Readme Edit"
@@ -622,6 +661,7 @@ title: "Readme Edit"
        b.AssertFileContent("public/index.html", `
 Readme Edit
 `)
+
 }
 
 // https://github.com/gohugoio/hugo/issues/6299

diff --git a/hugolib/page.go b/hugolib/page.go
index f5031332c67bef3d99652d0df9421a819cb2c49a..8aad8dcc7e050d017b1c438f084597091fa9ae28 100644 (file)
--- a/hugolib/page.go
+++ b/hugolib/page.go
@@ -954,7 +954,7 @@ func (p *pageState) sourceRefs() []string {
                path := meta.PathFile()
 
                if path != "" {
-                       ref := "/" + path
+                       ref := "/" + filepath.ToSlash(path)
                        if ref != refs[0] {
                                refs = append(refs, ref)
                        }

diff --git a/hugolib/pagecollections.go b/hugolib/pagecollections.go
index 57878881a3d5d5e2af38421a8ddf67338c7e19de..023222bddb750ab91fabd8a922b893d48932e952 100644 (file)
--- a/hugolib/pagecollections.go
+++ b/hugolib/pagecollections.go
@@ -267,7 +267,14 @@ func (c *PageCollections) getPageNew(context page.Page, ref string) (page.Page,
 
        } else if context != nil {
                // Try the page-relative path.
-               ppath := path.Join("/", strings.ToLower(context.SectionsPath()), ref)
+               var dir string
+               if !context.File().IsZero() {
+                       dir = filepath.ToSlash(context.File().Dir())
+               } else {
+                       dir = context.SectionsPath()
+               }
+               ppath := path.Join("/", strings.ToLower(dir), ref)
+
                p, err := c.getFromCache(ppath)
                if err == nil && p != nil {
                        return p, nil

diff --git a/hugolib/testhelpers_test.go b/hugolib/testhelpers_test.go
index 93ea9477cecb928116eacfa720e6a2dc69a8fc09..44185eadce365cc989295a07f7a22ae82c86dba4 100644 (file)
--- a/hugolib/testhelpers_test.go
+++ b/hugolib/testhelpers_test.go
@@ -466,6 +466,7 @@ func (s *sitesBuilder) CreateSitesE() error {
                        for _, dir := range []string{
                                "content/sect",
                                "layouts/_default",
+                               "layouts/_default/_markup",
                                "layouts/partials",
                                "layouts/shortcodes",
                                "data",