tweak feed errors notification

2 files changed, 36 insertions, 21 deletions

diff --git a/modules/popup-dialog.php b/modules/popup-dialog.php
index 02282f46b..d3933ae77 100644
--- a/modules/popup-dialog.php
+++ b/modules/popup-dialog.php
@@ -240,6 +240,36 @@
 
 		}
 
+		if ($id == "feedUpdateErrors") {
+
+			print "<div id=\"infoBoxTitle\">Update Errors</div>";
+			print "<div class=\"infoBoxContents\">";
+
+			print "These feeds have not been updated because of errors:";
+
+			$result = db_query($link, "SELECT id,title,feed_url,last_error
+			FROM ttrss_feeds WHERE last_error != '' AND owner_uid = ".$_SESSION["uid"]);
+
+			print "<ul class='nomarks'>";
+
+			while ($line = db_fetch_assoc($result)) {
+				print "<li><b>" . $line["title"] . "</b> (" . $line["feed_url"] . "): " . 
+					"<em>" . $line["last_error"] . "</em>";
+			}
+
+			print "</ul>";
+			print "</div>";
+
+			print "<div align='center'>";
+
+			print "<input class=\"button\"
+				type=\"submit\" onclick=\"return closeInfoBox()\" 
+				value=\"Close\">";
+
+			print "</div>";
+
+		}
+
 		print "</div>";
 	}
 ?>
diff --git a/modules/pref-feeds.php b/modules/pref-feeds.php
index af8fd0936..0572978dd 100644
--- a/modules/pref-feeds.php
+++ b/modules/pref-feeds.php
@@ -710,31 +710,16 @@
 
 		if ($quiet) return;
 
-//		print "<h3>Edit Feeds</h3>";
-
-		$result = db_query($link, "SELECT id,title,feed_url,last_error
+		$result = db_query($link, "SELECT COUNT(id) AS num_errors
 			FROM ttrss_feeds WHERE last_error != '' AND owner_uid = ".$_SESSION["uid"]);
 
-		if (db_num_rows($result) > 0) {
-		
-			print "<div class=\"warning\">";
-			
-//			print"<img class=\"closeButton\" 
-//				onclick=\"javascript:hideParentElement(this);\" src=\"images/close.png\">";
-	
-			print "<a href=\"javascript:showBlockElement('feedUpdateErrors')\">
-				<b>Some feeds have update errors (click for details)</b></a>";
-
-			print "<ul id=\"feedUpdateErrors\" class=\"nomarks\">";
-						
-			while ($line = db_fetch_assoc($result)) {
-				print "<li>" . $line["title"] . " (" . $line["feed_url"] . "): " . 
-					$line["last_error"];
-			}
+		$num_errors = db_fetch_result($result, 0, "num_errors");
 
-			print "</ul>";
-			print "</div>";
+		if ($num_errors > 0) {
 
+			print "<a href=\"javascript:showFeedsWithErrors()\" 
+				class=\"feedUpdErrLink\">Some feeds have update errors (click 
+				for details)</a>";
 		}
 
 		$feed_search = db_escape_string($_GET["search"]);