Skip to content

Commit

Permalink
tpl/time: Handle nil values in time.AsTime
Browse files Browse the repository at this point in the history
Closes #8865
  • Loading branch information
bep committed Aug 9, 2021
1 parent 9bba9a3 commit 3e11072
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
5 changes: 5 additions & 0 deletions tpl/time/time.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@ import (
"time"
_time "time"

"github.com/gohugoio/hugo/common/hreflect"

"github.com/gohugoio/hugo/common/htime"

"github.com/gohugoio/locales"
Expand All @@ -43,6 +45,9 @@ type Namespace struct {
// AsTime converts the textual representation of the datetime string into
// a time.Time interface.
func (ns *Namespace) AsTime(v interface{}, args ...interface{}) (interface{}, error) {
if !hreflect.IsTruthful(v) {
return time.Time{}, nil
}
loc := ns.location
if len(args) > 0 {
locStr, err := cast.ToStringE(args[0])
Expand Down
5 changes: 4 additions & 1 deletion tpl/time/time_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ func TestTimeLocation(t *testing.T) {

for i, test := range []struct {
name string
value string
value interface{}
location interface{}
expect interface{}
}{
Expand All @@ -39,6 +39,9 @@ func TestTimeLocation(t *testing.T) {
{"New York EST", "2020-01-20", "America/New_York", "2020-01-20 00:00:00 -0500 EST"},
{"Empty location, time", "2020-10-20 20:33:59", "", "2020-10-20 20:33:59 +0000 UTC"},
{"New York, time", "2020-10-20 20:33:59", "America/New_York", "2020-10-20 20:33:59 -0400 EDT"},
{"Nil value", nil, "", "0001-01-01 00:00:00 +0000"},
{"Empty value", "", "", "0001-01-01 00:00:00 +0000"},

// The following have an explicit offset specified. In this case, it overrides timezone
{"Offset minus 0700, empty location", "2020-09-23T20:33:44-0700", "", "2020-09-23 20:33:44 -0700 -0700"},
{"Offset plus 0200, empty location", "2020-09-23T20:33:44+0200", "", "2020-09-23 20:33:44 +0200 +0200"},
Expand Down

0 comments on commit 3e11072

Please sign in to comment.