remove LIBXML_NOEMPTYTAG because of double <br/>s - the #357 issue with

htmlpurifier might not be relevant anymore because of htmLawed switch,
but <br/>s are annoying

2 files changed, 4 insertions, 16 deletions

diff --git a/include/functions.php b/include/functions.php
index 4e16c9f19..dd40b9529 100644
--- a/include/functions.php
+++ b/include/functions.php
@@ -2575,7 +2575,6 @@
 		$xpath = new DOMXPath($doc);
 
 		$entries = $xpath->query('(//a[@href]|//img[@src])');
-		$br_inserted = 0;
 
 		foreach ($entries as $entry) {
 
@@ -2594,22 +2593,11 @@
 			if (strtolower($entry->nodeName) == "a") {
 				$entry->setAttribute("target", "_blank");
 			}
-
-			if (strtolower($entry->nodeName) == "img" && !$br_inserted) {
-				$br = $doc->createElement("br");
-
-				if ($entry->parentNode->nextSibling) {
-					$entry->parentNode->insertBefore($br, $entry->nextSibling);
-					$br_inserted = 1;
-				}
-
-			}
 		}
 
 		$node = $doc->getElementsByTagName('body')->item(0);
 
-		// http://tt-rss.org/redmine/issues/357
-		return $doc->saveXML($node, LIBXML_NOEMPTYTAG);
+		return $doc->saveXML($node);
 	}
 
 	function check_for_update($link) {
@@ -3774,7 +3762,7 @@
 
 		// http://tt-rss.org/forum/viewtopic.php?f=1&t=970
 		if ($node)
-			return $doc->saveXML($node, LIBXML_NOEMPTYTAG);
+			return $doc->saveXML($node);
 		else
 			return $html;
 	}
diff --git a/include/rssfuncs.php b/include/rssfuncs.php
index 0b1d06564..feffe0019 100644
--- a/include/rssfuncs.php
+++ b/include/rssfuncs.php
@@ -1082,7 +1082,7 @@
 
 		$node = $doc->getElementsByTagName('body')->item(0);
 
-		return $doc->saveXML($node, LIBXML_NOEMPTYTAG);
+		return $doc->saveXML($node);
 	}
 
 	function expire_lock_files($debug) {
@@ -1280,7 +1280,7 @@
 			$node = $doc->getElementsByTagName('body')->item(0);
 
 			if ($node) {
-				$content = $doc->saveXML($node, LIBXML_NOEMPTYTAG);
+				$content = $doc->saveXML($node);
 
 				return $content;
 			}