From 120d6ec7d837eac2825de3edb870d63bd0493853 Mon Sep 17 00:00:00 2001 From: Frédéric Guillot Date: Sat, 30 Nov 2019 22:46:12 -0800 Subject: Do no rewrite Youtube description twice in "add_youtube_video" rule This is already done before in . --- reader/rewrite/rewrite_functions.go | 2 +- reader/rewrite/rewriter_test.go | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/reader/rewrite/rewrite_functions.go b/reader/rewrite/rewrite_functions.go index 8988cc5..8f83aa9 100644 --- a/reader/rewrite/rewrite_functions.go +++ b/reader/rewrite/rewrite_functions.go @@ -143,7 +143,7 @@ func addYoutubeVideo(entryURL, entryContent string) string { if len(matches) == 2 { video := `` - return video + "

" + replaceLineFeeds(replaceTextLinks(entryContent)) + "

" + return video + `
` + entryContent } return entryContent } diff --git a/reader/rewrite/rewriter_test.go b/reader/rewrite/rewriter_test.go index 27cf425..8c0db6c 100644 --- a/reader/rewrite/rewriter_test.go +++ b/reader/rewrite/rewriter_test.go @@ -36,8 +36,8 @@ func TestRewriteWithNoMatchingRule(t *testing.T) { } func TestRewriteWithYoutubeLink(t *testing.T) { - output := Rewriter("https://www.youtube.com/watch?v=1234", "Video Description\nhttp://example.org/path", ``) - expected := `

Video Description
http://example.org/path

` + output := Rewriter("https://www.youtube.com/watch?v=1234", "Video Description", ``) + expected := `
Video Description` if expected != output { t.Errorf(`Not expected output: got "%s" instead of "%s"`, output, expected) -- cgit v1.2.3