Add option to link to original post in ALLPOST

config.go

@@ -23,6 +23,10 @@ type PttwebConfig struct {
 
 	EnableOver18Cookie bool
 
+	// EnableLinkOriginalInAllPost indicates whether to parse the board name in
+	// ALLPOST board and link to original posts.
+	EnableLinkOriginalInAllPost bool
+
 	FeedPrefix            string
 	AtomFeedTitleTemplate string
 

pttbbs/string.go

@@ -24,13 +24,16 @@ var (
 const (
 	ArticleAuthor = "作者"
 	ArticleTitle  = "標題"
+
+	AllPostBrdName = "ALLPOST"
 )
 
 var (
-	validBrdNameRegexp  = regexp.MustCompile(`^[0-9a-zA-Z][0-9a-zA-Z_\.\-]+$`)
-	validFileNameRegexp = regexp.MustCompile(`^[MG]\.\d+\.A(\.[0-9A-F]+)?$`)
-	validUserIDRegexp   = regexp.MustCompile(`^[a-zA-Z][0-9a-zA-Z]{1,11}$`)
-	fileNameTimeRegexp  = regexp.MustCompile(`^[MG]\.(\d+)\.A(\.[0-9A-F]+)?$`)
+	validBrdNameRegexp        = regexp.MustCompile(`^[0-9a-zA-Z][0-9a-zA-Z_\.\-]+$`)
+	validFileNameRegexp       = regexp.MustCompile(`^[MG]\.\d+\.A(\.[0-9A-F]+)?$`)
+	validUserIDRegexp         = regexp.MustCompile(`^[a-zA-Z][0-9a-zA-Z]{1,11}$`)
+	fileNameTimeRegexp        = regexp.MustCompile(`^[MG]\.(\d+)\.A(\.[0-9A-F]+)?$`)
+	allPostBrdNameTitleRegexp = regexp.MustCompile(`\(([0-9a-zA-Z][0-9a-zA-Z_\.\-]+)\)$`)
 )
 
 func IsValidBrdName(brdname string) bool {
@@ -109,3 +112,13 @@ func countPrefixSpaces(s string) int {
 	}
 	return 0
 }
+
+// BrdNameFromAllPostTitle returns the board name parsed from the post title in
+// ALLPOST board.
+func BrdNameFromAllPostTitle(title string) (string, bool) {
+	m := allPostBrdNameTitleRegexp.FindStringSubmatch(title)
+	if len(m) == 0 {
+		return "", false
+	}
+	return m[1], true
+}

pttbbs/string_test.go

@@ -0,0 +1,44 @@
+package pttbbs
+
+import "testing"
+
+func TestBrdNameFromAllPostTitle(t *testing.T) {
+	for _, test := range []struct {
+		desc  string
+		title string
+		want  string
+	}{
+		{
+			desc:  "normal title with board name",
+			title: "測試 (Test)",
+			want:  "Test",
+		},
+		{
+			desc:  "empty title with board name",
+			title: "(SYSOP)",
+			want:  "SYSOP",
+		},
+		{
+			desc:  "normal title but bad board name",
+			title: "測試 (SYS%%) ",
+		},
+		{
+			desc:  "bad title",
+			title: "(Bad) ",
+		},
+		{
+			desc:  "empty title",
+			title: "",
+		},
+	} {
+		t.Run(test.desc, func(t *testing.T) {
+			wantOk := test.want != ""
+
+			got, gotOk := BrdNameFromAllPostTitle(test.title)
+			if got != test.want || gotOk != wantOk {
+				t.Errorf("BrdNameFromAllPostTitle(%q) = (%q, %t); want (%q, %t)",
+					test.title, got, gotOk, test.want, wantOk)
+			}
+		})
+	}
+}

pttweb.go

@@ -257,7 +257,12 @@ func templateFuncMap() template.FuncMap {
 		"valid_article": func(a pttbbs.Article) bool {
 			return pttbbs.IsValidArticleFileName(a.FileName)
 		},
-		"route_bbsarticle": func(brdname, filename string) (*url.URL, error) {
+		"route_bbsarticle": func(brdname, filename, title string) (*url.URL, error) {
+			if config.EnableLinkOriginalInAllPost && brdname == pttbbs.AllPostBrdName {
+				if origBrdName, ok := pttbbs.BrdNameFromAllPostTitle(title); ok {
+					brdname = origBrdName
+				}
+			}
 			return router.Get("bbsarticle").URLPath("brdname", brdname, "filename", filename)
 		},
 		"route_askover18": func() (*url.URL, error) {