From bd3f2ade87d63b5340bf3be26f33c6f74ac35f8c Mon Sep 17 00:00:00 2001
From: Andrew Dolgov <fox@bah.org.ru>
Date: Mon, 18 Jan 2010 13:55:28 +0300
Subject: [PATCH] editTagsSave: invalidate article cache

---
 functions.php           | 27 +++++++++++++++++++--------
 modules/backend-rpc.php |  7 +++++++
 viewfeed.js             |  2 ++
 3 files changed, 28 insertions(+), 8 deletions(-)

diff --git a/functions.php b/functions.php
index 7350ef28..9a9aab71 100644
--- a/functions.php
+++ b/functions.php
@@ -4496,19 +4496,30 @@
 
 	function get_article_tags($link, $id, $owner_uid = 0) {
 
+		global $memcache;
+
 		$a_id = db_escape_string($id);
 
 		if (!$owner_uid) $owner_uid = $_SESSION["uid"];
 
-		$tmp_result = db_query($link, "SELECT DISTINCT tag_name, 
+		$query = "SELECT DISTINCT tag_name, 
 			owner_uid as owner FROM
 			ttrss_tags WHERE post_int_id = (SELECT int_id FROM ttrss_user_entries WHERE
-				ref_id = '$a_id' AND owner_uid = '$owner_uid' LIMIT 1) ORDER BY tag_name");
+			ref_id = '$a_id' AND owner_uid = '$owner_uid' LIMIT 1) ORDER BY tag_name";
 
+		$obj_id = md5("TAGS:$owner_uid:$id");
 		$tags = array();	
-	
-		while ($tmp_line = db_fetch_assoc($tmp_result)) {
-			array_push($tags, $tmp_line["tag_name"]);				
+
+		if ($memcache && $obj = $memcache->get($obj_id)) {
+			$tags = $obj;
+		} else {
+			$tmp_result = db_query($link, $query);
+
+			while ($tmp_line = db_fetch_assoc($tmp_result)) {
+				array_push($tags, $tmp_line["tag_name"]);				
+			}
+
+			if ($memcache) $memcache->add($obj_id, $tags, 0, 3600);
 		}
 
 		return $tags;
@@ -6500,11 +6511,11 @@
 		$query = "SELECT * FROM ttrss_enclosures 
 			WHERE post_id = '$id' AND content_url != ''";
 
-		$cache_id = md5($query);
+		$obj_id = md5("ENCLOSURES:$id");
 
 		$rv = array();
 
-		if ($memcache && $obj = $memcache->get($cache_id)) {
+		if ($memcache && $obj = $memcache->get($obj_id)) {
 			$rv = $obj;
 		} else {
 			$result = db_query($link, $query);
@@ -6513,7 +6524,7 @@
 				while ($line = db_fetch_assoc($result)) {
 					array_push($rv, $line);
 				}
-				if ($memcache) $memcache->add($cache_id, $rv, 0, 3600);
+				if ($memcache) $memcache->add($obj_id, $rv, 0, 3600);
 			}
 		}
 
diff --git a/modules/backend-rpc.php b/modules/backend-rpc.php
index adefbd5a..f994c3e8 100644
--- a/modules/backend-rpc.php
+++ b/modules/backend-rpc.php
@@ -440,6 +440,8 @@
 
 		if ($subop == "setArticleTags") {
 
+			global $memcache;
+
 			$id = db_escape_string($_REQUEST["id"]);
 
 			$tags_str = db_escape_string($_REQUEST["tags_str"]);
@@ -480,6 +482,11 @@
 
 			db_query($link, "COMMIT");
 
+			if ($memcache) {
+				$obj_id = md5("TAGS:".$_SESSION["uid"].":$id");
+				$memcache->delete($obj_id);
+			}
+
 			$tags_str = format_tags_string(get_article_tags($link, $id), $id);
 
 			print "<rpc-reply>
diff --git a/viewfeed.js b/viewfeed.js
index b4a9228f..0ec443c6 100644
--- a/viewfeed.js
+++ b/viewfeed.js
@@ -1500,6 +1500,8 @@ function editTagsSave() {
 								if (tags) {
 									tags.innerHTML = tags_str.firstChild.nodeValue;
 								}
+
+								cache_invalidate(id);
 							}
 						}
 					}
-- 
2.39.2