From badac687b01db1491d3917e45c478b3c9d7a9a06 Mon Sep 17 00:00:00 2001 From: Andrew Dolgov Date: Sat, 7 Feb 2009 13:40:43 +0300 Subject: [PATCH] synchronize unread status from offline to online DB --- functions.php | 7 +++ modules/backend-rpc.php | 43 ++++++++++++++- offline.js | 116 ++++++++++++++++++++++++++++++++++++++-- 3 files changed, 161 insertions(+), 5 deletions(-) diff --git a/functions.php b/functions.php index 7d3bae49c..08e71b9d9 100644 --- a/functions.php +++ b/functions.php @@ -1996,6 +1996,13 @@ } } + function bool_to_sql_bool($s) { + if ($s) { + return "true"; + } else { + return "false"; + } + } function toggleEvenOdd($a) { if ($a == "even") diff --git a/modules/backend-rpc.php b/modules/backend-rpc.php index 2e9915f53..9996f7f41 100644 --- a/modules/backend-rpc.php +++ b/modules/backend-rpc.php @@ -518,6 +518,7 @@ $stage = (int) $_REQUEST["stage"]; $cidt = db_escape_string($_REQUEST["cidt"]); $cidb = db_escape_string($_REQUEST["cidb"]); + $sync = db_escape_string($_REQUEST["sync"]); //$amount = (int) $_REQUEST["amount"]; //$unread_only = db_escape_string($_REQUEST["unread_only"]); //if (!$amount) $amount = 50; @@ -527,6 +528,47 @@ print ""; + $sync = split(";", $sync); + + print ""; + + if (count($sync) > 0) { + if (strtotime($sync[0])) { + $last_online = db_escape_string($sync[0]); + + print ""; + + for ($i = 1; $i < count($sync); $i++) { + $e = split(",", $sync[$i]); + + if (count($e) == 3) { + + $id = (int) $e[0]; + $unread = bool_to_sql_bool((bool) $e[1]); + $marked = bool_to_sql_bool((bool) $e[2]); + + /* Marked status is not synchronized yet */ + + $query = "UPDATE ttrss_user_entries SET + unread = $unread, + last_read = '$last_online' + WHERE ref_id = '$id' AND + (last_read IS NULL OR last_read < '$last_online') AND + owner_uid = ".$_SESSION["uid"]; + + $result = db_query($link, $query); + +// if (db_affected_rows($result) > 0) { + print ""; +// } + + } + } + } + } + + print ""; + if ($stage == 0) { print ""; @@ -596,7 +638,6 @@ } if ($stage > 0) { - print ""; $limit = 50; diff --git a/offline.js b/offline.js index f9ccf2998..6718733fa 100644 --- a/offline.js +++ b/offline.js @@ -600,6 +600,18 @@ function offline_download_parse(stage, transport) { try { if (transport.responseXML) { + var sync_ok = transport.responseXML.getElementsByTagName("sync-ok"); + + if (sync_ok.length > 0) { + for (var i = 0; i < sync_ok.length; i++) { + var id = sync_ok[i].getAttribute("id"); + if (id) { + debug("synced offline info for id " + id); + db.execute("UPDATE articles SET modified = '' WHERE id = ?", [id]); + } + } + } + if (stage == 0) { var feeds = transport.responseXML.getElementsByTagName("feed"); @@ -693,9 +705,12 @@ function offline_download_parse(stage, transport) { debug("downloaded articles: " + articles_found + " limit: " + limit); - if (articles_found >= limit) { + var has_sync_data = has_local_sync_data(); + + if (articles_found >= limit || has_sync_data) { window.setTimeout("update_offline_data("+(stage+1)+")", 10*1000); - debug("update_offline_data: done " + stage); + debug("update_offline_data: done " + stage + " HSD: " + + has_sync_data + ""); } else { window.setTimeout("update_offline_data(0)", 1800*1000); debug("update_offline_data: finished"); @@ -740,7 +755,16 @@ function update_offline_data(stage) { rs.close(); + var to_sync = prepare_local_sync_data(); + + if (to_sync != "") { + to_sync = "?sync=" + param_escape(to_sync); + } + + debug(query + "/" + to_sync); + new Ajax.Request(query, { + parameters: to_sync, onComplete: function(transport) { offline_download_parse(stage, transport); } }); @@ -928,6 +952,8 @@ function init_gears() { db.execute("DROP INDEX IF EXISTS article_labels_label_id_idx"); db.execute("DROP INDEX IF EXISTS articles_unread_idx"); db.execute("DROP INDEX IF EXISTS articles_feed_id_idx"); + db.execute("DROP INDEX IF EXISTS articles_id_idx"); + db.execute("DROP INDEX IF EXISTS article_labels_id_idx"); db.execute("DROP TABLE IF EXISTS version"); db.execute("DROP TRIGGER IF EXISTS articles_update_unread"); db.execute("DROP TRIGGER IF EXISTS articles_update_marked"); @@ -950,6 +976,8 @@ function init_gears() { db.execute("CREATE INDEX IF NOT EXISTS articles_unread_idx ON articles(unread)"); db.execute("CREATE INDEX IF NOT EXISTS article_labels_label_id_idx ON article_labels(label_id)"); db.execute("CREATE INDEX IF NOT EXISTS articles_feed_id_idx ON articles(feed_id)"); + db.execute("CREATE INDEX IF NOT EXISTS articles_id_idx ON articles(id)"); + db.execute("CREATE INDEX IF NOT EXISTS article_labels_id_idx ON article_labels(id)"); db.execute("CREATE TABLE IF NOT EXISTS syncdata (key integer, value text)"); @@ -1258,11 +1286,91 @@ function init_local_sync_data() { } } +function has_local_sync_data() { + try { + + var rs = db.execute("SELECT id FROM articles "+ + "WHERE modified > (SELECT value FROM syncdata WHERE key = 'last_online') "+ + "LIMIT 1"); + + var tmp = 0; + + if (rs.isValidRow()) { + tmp = rs.field(0); + } + + rs.close(); + + return tmp != 0; + + } catch (e) { + exception_error("has_local_sync_data", e); + } +} + +function prepare_local_sync_data() { + try { + var rs = db.execute("SELECT value FROM syncdata WHERE key = 'last_online'"); + + var last_online = ""; + + if (rs.isValidRow()) { + last_online = rs.field(0); + } + + rs.close(); + + var rs = db.execute("SELECT id,unread,marked FROM articles "+ + "WHERE modified > ? LIMIT 200", [last_online]); + + var tmp = last_online + ";"; + + var entries = 0; + + while (rs.isValidRow()) { + var e = new Array(); + + tmp = tmp + rs.field(0) + "," + rs.field(1) + "," + rs.field(2) + ";"; + entries++; + + rs.next(); + } + + rs.close(); + + if (entries > 0) { + return tmp; + } else { + return ''; + } + + } catch (e) { + exception_error("prepare_local_sync_data", e); + } +} + function update_local_sync_data() { try { if (db && !offline_mode) { - db.execute("UPDATE syncdata SET value = DATETIME('NOW', 'localtime') "+ - "WHERE key = 'last_online'"); + + var rs = db.execute("SELECT id FROM articles "+ + "WHERE modified > (SELECT value FROM syncdata WHERE "+ + "key = 'last_online') LIMIT 1") + + var f_id = 0; + + if (rs.isValidRow()) { + f_id = rs.field(0); + } + + rs.close(); + + /* no pending articles to sync */ + + if (f_id == 0) { + db.execute("UPDATE syncdata SET value = DATETIME('NOW', 'localtime') "+ + "WHERE key = 'last_online'"); + } } } catch (e) {