pngcrush.sh

[tt-rss.git] / classes / ccache.php

diff --git a/classes/ccache.php b/classes/ccache.php
index 88e28e9e83e6ba23642fc90693f9b9fb720c2ad1..9c5547e7d88a6c2346ca903ebd1cf54bbdedff42 100644 (file)
--- a/classes/ccache.php
+++ b/classes/ccache.php
@@ -14,6 +14,8 @@ class CCache {
 
        static function remove($feed_id, $owner_uid, $is_cat = false) {
 
+               $feed_id = (int) $feed_id;
+
                if (!$is_cat) {
                        $table = "ttrss_counters_cache";
                } else {
@@ -62,7 +64,12 @@ class CCache {
        static function find($feed_id, $owner_uid, $is_cat = false,
                                                 $no_update = false) {
 
-               if (!is_numeric($feed_id)) return;
+               // "" (null) is valid and should be cast to 0 (uncategorized)
+               // everything else i.e. tags are not
+               if (!is_numeric($feed_id) && $feed_id)
+                       return;
+
+               $feed_id = (int) $feed_id;
 
                if (!$is_cat) {
                        $table = "ttrss_counters_cache";
@@ -93,7 +100,12 @@ class CCache {
        static function update($feed_id, $owner_uid, $is_cat = false,
                                                   $update_pcat = true, $pcat_fast = false) {
 
-               if (!is_numeric($feed_id)) return;
+               // "" (null) is valid and should be cast to 0 (uncategorized)
+               // everything else i.e. tags are not
+               if (!is_numeric($feed_id) && $feed_id)
+                       return;
+
+               $feed_id = (int) $feed_id;
 
                $prev_unread = CCache::find($feed_id, $owner_uid, $is_cat, true);
 
@@ -118,18 +130,18 @@ class CCache {
 
                        if (!$pcat_fast) {
                                $sth = $pdo->prepare("SELECT id FROM ttrss_feeds
-                                               WHERE owner_uid = :uid AND 
+                                               WHERE owner_uid = :uid AND
                                                        (cat_id = :cat OR (:cat = 0 AND cat_id IS NULL))");
                                $sth->execute([":uid" => $owner_uid, ":cat" => $feed_id]);
 
                                while ($line = $sth->fetch()) {
-                                       CCache::update($line["id"], $owner_uid, false, false);
+                                       CCache::update((int)$line["id"], $owner_uid, false, false);
                                }
                        }
 
                        $sth = $pdo->prepare("SELECT SUM(value) AS sv
                                FROM ttrss_counters_cache, ttrss_feeds
-                               WHERE id = feed_id AND 
+                               WHERE id = feed_id AND
                                (cat_id = :cat OR (:cat = 0 AND cat_id IS NULL)) AND
                                ttrss_counters_cache.owner_uid = :uid AND
                                ttrss_feeds.owner_uid = :uid");
@@ -185,7 +197,7 @@ class CCache {
                                        $sth->execute([$owner_uid, $feed_id]);
 
                                        if ($row = $sth->fetch()) {
-                                               CCache::update($row["cat_id"], $owner_uid, true, true, true);
+                                               CCache::update((int)$row["cat_id"], $owner_uid, true, true, true);
                                        }
                                }
                        }