Fix section menus for lazy blogger

IsMenuCurrent was always returning false for the top level entries.

Fixes #2065

diff --git a/hugolib/menu_test.go b/hugolib/menu_test.go
index f8012bf9c2c49dc8c4c923a720f7243e0ed63832..cd5128543b0ddffcc4eee546c032ef8237d16692 100644 (file)
--- a/hugolib/menu_test.go
+++ b/hugolib/menu_test.go
@@ -435,9 +435,6 @@ func doTestSectionPagesMenu(canonifyUrls bool, t *testing.T) {
        assert.NotNil(t, fishySectionMenuEntry)
        assert.NotNil(t, nodeFishy)
 
-       firstSectionMenuEntry.URL = firstSectionMenuEntry.URL + "/"
-       secondSectionMenuEntry.URL = secondSectionMenuEntry.URL + "/"
-       fishySectionMenuEntry.URL = fishySectionMenuEntry.URL + "/"
        assert.True(t, nodeFirst.IsMenuCurrent("spm", firstSectionMenuEntry))
        assert.False(t, nodeFirst.IsMenuCurrent("spm", secondSectionMenuEntry))
        assert.False(t, nodeFirst.IsMenuCurrent("spm", fishySectionMenuEntry))

diff --git a/hugolib/site.go b/hugolib/site.go
index 1c6d0257cdc56eb328d2fa9f7aaeb9b1b4b7118c..4cb65b7fc6f3ed5f238f541f188e93c147472d1c 100644 (file)
--- a/hugolib/site.go
+++ b/hugolib/site.go
@@ -1192,7 +1192,7 @@ func (s *Site) assembleMenus() {
                if sectionPagesMenu != "" {
                        if _, ok := sectionPagesMenus[p.Section()]; !ok {
                                if p.Section() != "" {
-                                       me := MenuEntry{Identifier: p.Section(), Name: helpers.MakeTitle(helpers.FirstUpper(p.Section())), URL: s.Info.createNodeMenuEntryURL("/" + p.Section())}
+                                       me := MenuEntry{Identifier: p.Section(), Name: helpers.MakeTitle(helpers.FirstUpper(p.Section())), URL: s.Info.createNodeMenuEntryURL("/" + p.Section() + "/")}
                                        if _, ok := flat[twoD{sectionPagesMenu, me.KeyName()}]; ok {
                                                // menu with same id defined in config, let that one win
                                                continue