From 51f6f91758bfc57e52e8e9316495ae723d7c2332 Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@bah.org.ru>
Date: Tue, 3 Feb 2009 21:45:31 +0300
Subject: [PATCH] remove manual download dialog; make offline updates automatic

---
 modules/backend-rpc.php  |  8 +++++---
 modules/popup-dialog.php |  4 ++--
 tt-rss.js                | 30 +++++++++++++-----------------
 3 files changed, 20 insertions(+), 22 deletions(-)

diff --git a/modules/backend-rpc.php b/modules/backend-rpc.php
index 3b5b2d63..967e97ae 100644
--- a/modules/backend-rpc.php
+++ b/modules/backend-rpc.php
@@ -517,10 +517,12 @@
 		if ($subop == "download") {
 			$stage = (int) $_REQUEST["stage"];
 			$cid = db_escape_string($_REQUEST["cid"]);
-			$amount = (int) $_REQUEST["amount"];
-			$unread_only = db_escape_string($_REQUEST["unread_only"]);
+			//$amount = (int) $_REQUEST["amount"];
+			//$unread_only = db_escape_string($_REQUEST["unread_only"]);
+			//if (!$amount) $amount = 50;
 
-			if (!$amount) $amount = 50;
+			$amount = 100;
+			$unread_only = true;
 
 			print "<rpc-reply>";
 
diff --git a/modules/popup-dialog.php b/modules/popup-dialog.php
index 8a1b2020..2ba0740c 100644
--- a/modules/popup-dialog.php
+++ b/modules/popup-dialog.php
@@ -461,7 +461,7 @@
 			return;
 		}
 
-		if ($id == "offlineDownload") {
+/*		if ($id == "offlineDownload") {
 			print "<div id=\"infoBoxTitle\">".__('Download articles')."</div>";
 			print "<div class=\"infoBoxContents\">";
 
@@ -500,7 +500,7 @@
 			print "</div>";
 
 			return;
-		}
+		} */
 
 
 		print "<div id='infoBoxTitle'>Internal Error</div>
diff --git a/tt-rss.js b/tt-rss.js
index ab62a85f..b4f2d2ad 100644
--- a/tt-rss.js
+++ b/tt-rss.js
@@ -670,10 +670,10 @@ function quickMenuGo(opid) {
 			resize_headlines();
 		}
 
-		if (opid == "qmcDownload") {
+/*		if (opid == "qmcDownload") {
 			displayDlg("offlineDownload");
 			return;
-		}
+		} */
 
 		if (opid == "qmcResetCats") {
 
@@ -1308,7 +1308,7 @@ function hotkey_handler(e) {
 				}
 			}
 
-			if (keycode == 68 && shift_key) { // D
+/*			if (keycode == 68 && shift_key) { // D
 				initiate_offline_download();
 				return false;
 			}
@@ -1316,7 +1316,7 @@ function hotkey_handler(e) {
 			if (keycode == 68) { // d
 				displayDlg("offlineDownload");
 				return false;
-			}
+			} */
 
 			if (keycode == 87) { // w
 				feeds_sort_by_unread = !feeds_sort_by_unread;
@@ -1532,8 +1532,7 @@ function init_gears() {
 
 			db.execute("CREATE TABLE if not exists articles (id integer, feed_id integer, title text, link text, guid text, updated text, content text, tags text, unread text, marked text, added text)");
 
-			var qmcDownload = document.getElementById("qmcDownload");
-			if (qmcDownload) Element.show(qmcDownload);
+			window.setTimeout("update_offline_data(0)", 100);
 
 		}	
 	
@@ -1583,7 +1582,7 @@ function offline_download_parse(stage, transport) {
 						[id, title, has_icon]);
 				}
 		
-				window.setTimeout("initiate_offline_download("+(stage+1)+")", 50);
+				window.setTimeout("update_offline_data("+(stage+1)+")", 50);
 			} else {
 
 				var articles = transport.responseXML.getElementsByTagName("article");
@@ -1610,10 +1609,10 @@ function offline_download_parse(stage, transport) {
 				}
 
 				if (articles_found > 0) {
-					window.setTimeout("initiate_offline_download("+(stage+1)+")", 50);
+					window.setTimeout("update_offline_data("+(stage+1)+")", 50);
 				} else {
-					notify_info("All done.");
-					closeInfoBox();
+//					notify_info("All done.");
+//					closeInfoBox();
 				}
 			}
 
@@ -1623,13 +1622,14 @@ function offline_download_parse(stage, transport) {
 	}
 }
 
-function initiate_offline_download(stage, caller) {
+function update_offline_data(stage) {
 	try {
 
 		if (!stage) stage = 0;
-		if (caller) caller.disabled = true;
 
-		notify_progress("Loading, please wait... (" + stage +")", true);
+		debug("update_offline_data: stage " + stage);
+
+//		notify_progress("Loading, please wait... (" + stage +")", true);
 
 		var query = "backend.php?op=rpc&subop=download&stage=" + stage;
 
@@ -1644,10 +1644,6 @@ function initiate_offline_download(stage, caller) {
 			query = query + "&cid=" + offline_dl_max_id;
 		}
 
-		if (document.getElementById("download_ops_form")) {
-			query = query + "&" + Form.serialize("download_ops_form");
-		}
-
 		new Ajax.Request(query, {
 			onComplete: function(transport) { 
 				offline_download_parse(stage, transport);				
-- 
2.39.2