Merge pull request '[GITEA] Revert "Make user-content-* consistent with github (#26388)"' (#1945) from earl-warren/forgejo:wip-v1.21-markdown-anchors into v1.21/forgejo

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/1945
Reviewed-by: Gusted <gusted@noreply.codeberg.org>

modules/markup/common/footnote.go

@@ -29,12 +29,17 @@ func CleanValue(value []byte) []byte {
 	value = bytes.TrimSpace(value)
 	rs := bytes.Runes(value)
 	result := make([]rune, 0, len(rs))
+	needsDash := false
 	for _, r := range rs {
-		if unicode.IsLetter(r) || unicode.IsNumber(r) || r == '_' || r == '-' {
+		switch {
+		case unicode.IsLetter(r) || unicode.IsNumber(r) || r == '_':
+			if needsDash && len(result) > 0 {
+				result = append(result, '-')
+			}
+			needsDash = false
 			result = append(result, unicode.ToLower(r))
-		}
+		default:
-		if unicode.IsSpace(r) {
+			needsDash = true
-			result = append(result, '-')
 		}
 	}
 	return []byte(string(result))

modules/markup/common/footnote_test.go

@@ -1,4 +1,5 @@
 // Copyright 2023 The Gitea Authors. All rights reserved.
+// Copyright 2023 The Forgejo Authors. All rights reserved.
 // SPDX-License-Identifier: MIT
 package common
 
@@ -15,44 +16,45 @@ func TestCleanValue(t *testing.T) {
 	}{
 		// Github behavior test cases
 		{"", ""},
-		{"test(0)", "test0"},
+		{"test.0.1", "test-0-1"},
-		{"test!1", "test1"},
+		{"test(0)", "test-0"},
-		{"test:2", "test2"},
+		{"test!1", "test-1"},
-		{"test*3", "test3"},
+		{"test:2", "test-2"},
-		{"test！4", "test4"},
+		{"test*3", "test-3"},
-		{"test：5", "test5"},
+		{"test！4", "test-4"},
-		{"test*6", "test6"},
+		{"test：5", "test-5"},
-		{"test：6 a", "test6-a"},
+		{"test*6", "test-6"},
-		{"test：6 !b", "test6-b"},
+		{"test：6 a", "test-6-a"},
-		{"test：ad # df", "testad--df"},
+		{"test：6 !b", "test-6-b"},
-		{"test：ad #23 df 2*/*", "testad-23-df-2"},
+		{"test：ad # df", "test-ad-df"},
-		{"test：ad 23 df 2*/*", "testad-23-df-2"},
+		{"test：ad #23 df 2*/*", "test-ad-23-df-2"},
-		{"test：ad # 23 df 2*/*", "testad--23-df-2"},
+		{"test：ad 23 df 2*/*", "test-ad-23-df-2"},
+		{"test：ad # 23 df 2*/*", "test-ad-23-df-2"},
 		{"Anchors in Markdown", "anchors-in-markdown"},
 		{"a_b_c", "a_b_c"},
 		{"a-b-c", "a-b-c"},
-		{"a-b-c----", "a-b-c----"},
+		{"a-b-c----", "a-b-c"},
-		{"test：6a", "test6a"},
+		{"test：6a", "test-6a"},
-		{"test：a6", "testa6"},
+		{"test：a6", "test-a6"},
-		{"tes a a   a  a", "tes-a-a---a--a"},
+		{"tes a a   a  a", "tes-a-a-a-a"},
-		{"  tes a a   a  a  ", "tes-a-a---a--a"},
+		{"  tes a a   a  a  ", "tes-a-a-a-a"},
 		{"Header with \"double quotes\"", "header-with-double-quotes"},
-		{"Placeholder to force scrolling on link's click", "placeholder-to-force-scrolling-on-links-click"},
+		{"Placeholder to force scrolling on link's click", "placeholder-to-force-scrolling-on-link-s-click"},
 		{"tes（）", "tes"},
-		{"tes（0）", "tes0"},
+		{"tes（0）", "tes-0"},
-		{"tes{0}", "tes0"},
+		{"tes{0}", "tes-0"},
-		{"tes[0]", "tes0"},
+		{"tes[0]", "tes-0"},
-		{"test【0】", "test0"},
+		{"test【0】", "test-0"},
-		{"tes…@a", "tesa"},
+		{"tes…@a", "tes-a"},
 		{"tes￥& a", "tes-a"},
 		{"tes= a", "tes-a"},
-		{"tes|a", "tesa"},
+		{"tes|a", "tes-a"},
-		{"tes\\a", "tesa"},
+		{"tes\\a", "tes-a"},
-		{"tes/a", "tesa"},
+		{"tes/a", "tes-a"},
 		{"a啊啊b", "a啊啊b"},
-		{"c🤔️🤔️d", "cd"},
+		{"c🤔️🤔️d", "c-d"},
-		{"a⚡a", "aa"},
+		{"a⚡a", "a-a"},
-		{"e.~f", "ef"},
+		{"e.~f", "e-f"},
 	}
 	for _, test := range tests {
 		assert.Equal(t, []byte(test.expect), CleanValue([]byte(test.param)), test.param)