hugolib: Fix relative .Page.GetPage from bundle

Fixes #6705
This commit is contained in:
Bjørn Erik Pedersen 2020-01-05 12:56:41 +01:00
parent 9b6e61464b
commit 196a9df585
4 changed files with 51 additions and 3 deletions

View file

@ -585,9 +585,19 @@ workingDir = %q
{{ $mypage := .Site.GetPage "/blog/mypage.md" }} {{ $mypage := .Site.GetPage "/blog/mypage.md" }}
{{ with $mypage }}MYPAGE: {{ .Title }}|Path: {{ path.Join .File.Path }}|FilePath: {{ path.Join .File.FileInfo.Meta.PathFile }}|{{ end }} {{ 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"), 0777)
os.Mkdir(filepath.Join(workingDir, "mycontent", "mybundle"), 0777)
b.WithSourceFile("README.md", `--- b.WithSourceFile("README.md", `---
title: "Readme Title" title: "Readme Title"
@ -600,6 +610,23 @@ Readme Content.
title: "My Page" 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{}) b.Build(BuildCfg{})
@ -610,7 +637,19 @@ README: Readme Title
Readme Content. Readme Content.
MYPAGE: My Page|Path: blog/mypage.md|FilePath: mycontent/mypage.md| 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", `--- b.EditFiles("README.md", `---
title: "Readme Edit" title: "Readme Edit"
@ -622,6 +661,7 @@ title: "Readme Edit"
b.AssertFileContent("public/index.html", ` b.AssertFileContent("public/index.html", `
Readme Edit Readme Edit
`) `)
} }
// https://github.com/gohugoio/hugo/issues/6299 // https://github.com/gohugoio/hugo/issues/6299

View file

@ -954,7 +954,7 @@ func (p *pageState) sourceRefs() []string {
path := meta.PathFile() path := meta.PathFile()
if path != "" { if path != "" {
ref := "/" + path ref := "/" + filepath.ToSlash(path)
if ref != refs[0] { if ref != refs[0] {
refs = append(refs, ref) refs = append(refs, ref)
} }

View file

@ -267,7 +267,14 @@ func (c *PageCollections) getPageNew(context page.Page, ref string) (page.Page,
} else if context != nil { } else if context != nil {
// Try the page-relative path. // 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) p, err := c.getFromCache(ppath)
if err == nil && p != nil { if err == nil && p != nil {
return p, nil return p, nil

View file

@ -466,6 +466,7 @@ func (s *sitesBuilder) CreateSitesE() error {
for _, dir := range []string{ for _, dir := range []string{
"content/sect", "content/sect",
"layouts/_default", "layouts/_default",
"layouts/_default/_markup",
"layouts/partials", "layouts/partials",
"layouts/shortcodes", "layouts/shortcodes",
"data", "data",