resources/page: Improve front matter date validation

Improve the error message and treat empty strings as zero dates.

Closes #12898
This commit is contained in:
Joe Mooring 2024-10-04 18:10:51 -07:00 committed by Bjørn Erik Pedersen
parent 0450d69fc6
commit ab03588db9
2 changed files with 55 additions and 2 deletions

View file

@ -728,7 +728,7 @@ func (f *frontmatterFieldHandlers) newDateFieldHandler(key string, setter func(d
return func(d *FrontMatterDescriptor) (bool, error) { return func(d *FrontMatterDescriptor) (bool, error) {
v, found := d.PageConfig.Params[key] v, found := d.PageConfig.Params[key]
if !found { if !found || v == "" {
return false, nil return false, nil
} }
@ -739,7 +739,7 @@ func (f *frontmatterFieldHandlers) newDateFieldHandler(key string, setter func(d
var err error var err error
date, err = htime.ToTimeInDefaultLocationE(v, d.Location) date, err = htime.ToTimeInDefaultLocationE(v, d.Location)
if err != nil { if err != nil {
return false, fmt.Errorf("invalid front matter: %s: %s: see %s", key, v, d.PathOrTitle) return false, fmt.Errorf("the %q front matter field is not a parsable date: see %s", key, d.PathOrTitle)
} }
d.PageConfig.Params[key] = date d.PageConfig.Params[key] = date
} }

View file

@ -14,6 +14,7 @@
package pagemeta_test package pagemeta_test
import ( import (
"strings"
"testing" "testing"
"github.com/gohugoio/hugo/hugolib" "github.com/gohugoio/hugo/hugolib"
@ -43,3 +44,55 @@ Lastmod: 2024-03-13 06:00:00 +0000 GMT
Eq: true Eq: true
`) `)
} }
func TestDateValidation(t *testing.T) {
t.Parallel()
files := `
-- hugo.toml --
disableKinds = ['page','rss','section','sitemap','taxonomy','term']
-- content/_index.md --
+++
date = DATE
+++
-- layouts/index.html --
{{ .Date.UTC.Format "2006-01-02" }}
--
`
errorMsg := `ERROR the "date" front matter field is not a parsable date`
// Valid (TOML)
f := strings.ReplaceAll(files, "DATE", "2024-10-01")
b := hugolib.Test(t, f)
b.AssertFileContent("public/index.html", "2024-10-01")
// Valid (string)
f = strings.ReplaceAll(files, "DATE", `"2024-10-01"`)
b = hugolib.Test(t, f)
b.AssertFileContent("public/index.html", "2024-10-01")
// Valid (empty string)
f = strings.ReplaceAll(files, "DATE", `""`)
b = hugolib.Test(t, f)
b.AssertFileContent("public/index.html", "0001-01-01")
// Valid (int)
f = strings.ReplaceAll(files, "DATE", "0")
b = hugolib.Test(t, f)
b.AssertFileContent("public/index.html", "1970-01-01")
// Invalid (string)
f = strings.ReplaceAll(files, "DATE", `"2024-42-42"`)
b, _ = hugolib.TestE(t, f)
b.AssertLogContains(errorMsg)
// Invalid (bool)
f = strings.ReplaceAll(files, "DATE", "true")
b, _ = hugolib.TestE(t, f)
b.AssertLogContains(errorMsg)
// Invalid (float)
f = strings.ReplaceAll(files, "DATE", "6.7")
b, _ = hugolib.TestE(t, f)
b.AssertLogContains(errorMsg)
}