GroupByParamDate now supports datetimes
authorHorst Gutmann <zerok@zerokspot.com>
Mon, 28 Dec 2020 15:06:10 +0000 (16:06 +0100)
committerBjørn Erik Pedersen <bjorn.erik.pedersen@gmail.com>
Mon, 28 Dec 2020 16:50:55 +0000 (17:50 +0100)
Previously, the method only worked for date strings but not (for
instance) full RFC3339 datetime strings.

resources/page/pagegroup.go
resources/page/pagegroup_test.go

index d4ca8ca4161591eb4a0efcbe9b3bea4eb3d52081..e07efa7ca0d8740842503599c5c96b1482988912 100644 (file)
@@ -316,7 +316,7 @@ func (p Pages) GroupByParamDate(key string, format string, order ...string) (Pag
                var r Pages
 
                for _, p := range pages {
-                       param := resource.GetParamToLower(p, key)
+                       param := resource.GetParam(p, key)
                        var t time.Time
 
                        if param != nil {
index 218bd841dec12b9b95f1d08497cc8ee5ed70b72c..ef0d24471185b3cdd018406cefc14c9fe2e347f7 100644 (file)
@@ -34,7 +34,8 @@ var pageGroupTestSources = []pageGroupTestObject{
        {"/section1/testpage2.md", 3, "2012-01-01", "bar"},
        {"/section1/testpage3.md", 2, "2012-04-06", "foo"},
        {"/section2/testpage4.md", 1, "2012-03-02", "bar"},
-       {"/section2/testpage5.md", 1, "2012-04-06", "baz"},
+       // date might also be a full datetime:
+       {"/section2/testpage5.md", 1, "2012-04-06T00:00:00Z", "baz"},
 }
 
 func preparePageGroupTestPages(t *testing.T) Pages {