From a0f4015f7d9e74762bc444ab405dbebba50326ed Mon Sep 17 00:00:00 2001
From: NiLuJe <ninuje@gmail.com>
Date: Mon, 14 Dec 2020 18:43:23 +0100
Subject: [PATCH] Workaround luxl's issue with self-closing tags once and for
 all (#6994)

* luxl and self-closing tags, once more with feeling...

Fix #6993

* More generic fix to the self-closing tag issue w/ luxl

Re #6902 & co.
---
 frontend/ui/opdsparser.lua | 11 +++--------
 1 file changed, 3 insertions(+), 8 deletions(-)

diff --git a/frontend/ui/opdsparser.lua b/frontend/ui/opdsparser.lua
index a1321b3a9..91ba4cfa8 100644
--- a/frontend/ui/opdsparser.lua
+++ b/frontend/ui/opdsparser.lua
@@ -78,16 +78,11 @@ function OPDSParser:parse(text)
     text = text:gsub('<content type="xhtml">.-</content>', '')
     -- luxl doesn't handle XML comments, so strip them
     text = text:gsub("<!%-%-.-%-%->", "")
-    -- luxl prefers <br />, the other two forms are valid in HTML, but will kick luxl's ass
+    -- luxl is also particular about the syntax for self-closing, empty & orphaned tags...
+    text = text:gsub("<(%l+)/>", "<%1 />")
+    -- We also need to handle the slash-less variants for br & hr...
     text = text:gsub("<br>", "<br />")
-    text = text:gsub("<br/>", "<br />")
-    -- Same deal with hr
     text = text:gsub("<hr>", "<hr />")
-    text = text:gsub("<hr/>", "<hr />")
-    -- It's also allergic to orphaned <em/> (As opposed to a balanced <em></em> pair)...
-    text = text:gsub("<em/>", "")
-    -- Let's assume it might also happen to strong...
-    text = text:gsub("<strong/>", "")
     -- Some OPDS catalogs wrap text in a CDATA section, remove it as it causes parsing problems
     text = text:gsub("<!%[CDATA%[(.-)%]%]>", function (s)
         return s:gsub( "%p", {["&"] = "&amp;", ["<"] = "&lt;", [">"] = "&gt;" } )