From cc38c8e5497a8a86c3cc23f4f9e279a412c57727 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@madoka.volgo-balt.ru>
Date: Sat, 16 Mar 2013 16:07:11 +0400
Subject: [PATCH] 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

---
 include/functions.php           | 16 ++--------------
 include/rssfuncs.php            |  4 ++--
 plugins/af_buttersafe/init.php  |  2 +-
 plugins/af_explosm/init.php     |  2 +-
 plugins/af_gocomics/init.php    |  2 +-
 plugins/af_pennyarcade/init.php |  2 +-
 plugins/af_redditimgur/init.php |  2 +-
 7 files changed, 9 insertions(+), 21 deletions(-)

diff --git a/include/functions.php b/include/functions.php
index 4e16c9f1..dd40b952 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 0b1d0656..feffe001 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;
 			}
diff --git a/plugins/af_buttersafe/init.php b/plugins/af_buttersafe/init.php
index 12833540..c9f6505c 100644
--- a/plugins/af_buttersafe/init.php
+++ b/plugins/af_buttersafe/init.php
@@ -44,7 +44,7 @@ class Af_Buttersafe extends Plugin {
 					}
 
 					if ($basenode) {
-						$article["content"] = $doc->saveXML($basenode, LIBXML_NOEMPTYTAG);
+						$article["content"] = $doc->saveXML($basenode);
 						$article["plugin_data"] = "buttersafe,$owner_uid:" . $article["plugin_data"];
 					}
 				}
diff --git a/plugins/af_explosm/init.php b/plugins/af_explosm/init.php
index 701eafee..2a8fab49 100644
--- a/plugins/af_explosm/init.php
+++ b/plugins/af_explosm/init.php
@@ -44,7 +44,7 @@ class Af_Explosm extends Plugin {
 					}
 
 					if ($basenode) {
-						$article["content"] = $doc->saveXML($basenode, LIBXML_NOEMPTYTAG);
+						$article["content"] = $doc->saveXML($basenode);
 						$article["plugin_data"] = "explosm,$owner_uid:" . $article["plugin_data"];
 					}
 				}
diff --git a/plugins/af_gocomics/init.php b/plugins/af_gocomics/init.php
index da524259..466ec968 100644
--- a/plugins/af_gocomics/init.php
+++ b/plugins/af_gocomics/init.php
@@ -44,7 +44,7 @@ class Af_GoComics extends Plugin {
 					}
 
 					if ($basenode) {
-						$article["content"] = $doc->saveXML($basenode, LIBXML_NOEMPTYTAG);
+						$article["content"] = $doc->saveXML($basenode);
 						$article["plugin_data"] = "gocomics,$owner_uid:" . $article["plugin_data"];
 					}
 				}
diff --git a/plugins/af_pennyarcade/init.php b/plugins/af_pennyarcade/init.php
index bf07b94a..26d19bc0 100644
--- a/plugins/af_pennyarcade/init.php
+++ b/plugins/af_pennyarcade/init.php
@@ -44,7 +44,7 @@ class Af_PennyArcade extends Plugin {
 					}
 
 					if ($basenode) {
-						$article["content"] = $doc->saveXML($basenode, LIBXML_NOEMPTYTAG);
+						$article["content"] = $doc->saveXML($basenode);
 						$article["plugin_data"] = "pennyarcade,$owner_uid:" . $article["plugin_data"];
 					}
 				}
diff --git a/plugins/af_redditimgur/init.php b/plugins/af_redditimgur/init.php
index b2bc975e..2bb44f6d 100644
--- a/plugins/af_redditimgur/init.php
+++ b/plugins/af_redditimgur/init.php
@@ -113,7 +113,7 @@ class Af_RedditImgur extends Plugin {
 					$node = $doc->getElementsByTagName('body')->item(0);
 
 					if ($node && $found) {
-						$article["content"] = $doc->saveXML($node, LIBXML_NOEMPTYTAG);
+						$article["content"] = $doc->saveXML($node);
 						if (!$force) $article["plugin_data"] = "redditimgur,$owner_uid:" . $article["plugin_data"];
 					}
 				}