shithub: hugo

Download patch

ref: c1425a166d3a26fa1789a63a1b6fd0c813dec15e
parent: 831bfd36aaf18de8ffa6a8f63519ab3e541fa4dd
author: Bjørn Erik Pedersen <[email protected]>
date: Thu Feb 23 05:03:48 EST 2017 

hugolib: Fix preserveTaxonomyNames regression

Fixes #3070


--- a/hugolib/hugo_sites.go
+++ b/hugolib/hugo_sites.go
@@ -326,6 +326,8 @@
 				foundTaxonomyTermsPage := false
 				for key := range tax {
 					foundTaxonomyPage := false
+					origKey := key
+
 					if s.Info.preserveTaxonomyNames {
 						key = s.PathSpec.MakePathSanitized(key)
 					}
@@ -344,7 +346,7 @@
 
 					if s.isEnabled(KindTaxonomy) {
 						if !foundTaxonomyPage {
-							n := s.newTaxonomyPage(plural, key)
+							n := s.newTaxonomyPage(plural, origKey)
 							s.Pages = append(s.Pages, n)
 							newPages = append(newPages, n)
 						}
--- a/hugolib/site.go
+++ b/hugolib/site.go
@@ -2097,12 +2097,11 @@
 	p.sections = []string{plural, key}
 
 	if s.Info.preserveTaxonomyNames {
+		// Keep (mostly) as is in the title
+		// We make the first character upper case, mostly because
+		// it is easier to reason about in the tests.
+		p.Title = helpers.FirstUpper(key)
 		key = s.PathSpec.MakePathSanitized(key)
-	}
-
-	if s.Info.preserveTaxonomyNames {
-		// keep as is in the title
-		p.Title = key
 	} else {
 		p.Title = strings.Replace(strings.Title(key), "-", " ", -1)
 	}
--- a/hugolib/taxonomy_test.go
+++ b/hugolib/taxonomy_test.go
@@ -49,12 +49,21 @@
 	}
 }
 
-// Issue #2992
 func TestTaxonomiesWithAndWithoutContentFile(t *testing.T) {
+	for _, preserveTaxonomyNames := range []bool{false, true} {
+		t.Run(fmt.Sprintf("preserveTaxonomyNames %t", preserveTaxonomyNames), func(t *testing.T) {
+			doTestTaxonomiesWithAndWithoutContentFile(t, preserveTaxonomyNames)
+		})
+
+	}
+}
+
+func doTestTaxonomiesWithAndWithoutContentFile(t *testing.T, preserveTaxonomyNames bool) {
 	t.Parallel()
 
 	siteConfig := `
 baseURL = "http://example.com/blog"
+preserveTaxonomyNames = %t
 
 paginate = 1
 defaultContentLanguage = "en"
@@ -77,6 +86,8 @@
 # Doc
 `
 
+	siteConfig = fmt.Sprintf(siteConfig, preserveTaxonomyNames)
+
 	th, h := newTestSitesFromConfigWithDefaultTemplates(t, siteConfig)
 	require.Len(t, h.Sites, 1)
 
@@ -85,6 +96,7 @@
 	writeSource(t, fs, "content/p1.md", fmt.Sprintf(pageTemplate, "t1/c1", "- tag1", "- cat1", "- o1"))
 	writeSource(t, fs, "content/p2.md", fmt.Sprintf(pageTemplate, "t2/c1", "- tag2", "- cat1", "- o1"))
 	writeSource(t, fs, "content/p3.md", fmt.Sprintf(pageTemplate, "t2/c12", "- tag2", "- cat2", "- o1"))
+	writeSource(t, fs, "content/p4.md", fmt.Sprintf(pageTemplate, "Hello World", "", "", "- \"Hello Hugo world\""))
 
 	writeNewContentFile(t, fs, "Category Terms", "2017-01-01", "content/categories/_index.md", 10)
 	writeNewContentFile(t, fs, "Tag1 List", "2017-01-01", "content/tags/tag1/_index.md", 10)
@@ -121,5 +133,16 @@
 	require.Len(t, cat.Pages, 3)
 	require.Len(t, cat.Data["Pages"], 3)
 	require.Equal(t, "t1/c1", cat.Pages[0].Title)
+
+	// Issue #3070 preserveTaxonomyNames
+	if preserveTaxonomyNames {
+		helloWorld := s.getPage(KindTaxonomy, "others", "Hello Hugo world")
+		require.NotNil(t, helloWorld)
+		require.Equal(t, "Hello Hugo world", helloWorld.Title)
+	} else {
+		helloWorld := s.getPage(KindTaxonomy, "others", "hello-hugo-world")
+		require.NotNil(t, helloWorld)
+		require.Equal(t, "Hello Hugo World", helloWorld.Title)
+	}
 
 }