From 4c5f3bbd9b81e73316c794bd47debf1659e7723c Mon Sep 17 00:00:00 2001 From: Alexandre Alapetite Date: Tue, 9 May 2023 22:31:43 +0200 Subject: Fix markAsReadUponGone (#5382) Fix regression from https://github.com/FreshRSS/FreshRSS/pull/5315 which indroduced a bug for cached feeds. We now update the `lastSeen` property of entries to account for the fact that they are unchanged but still existing. --- app/Models/EntryDAO.php | 32 +++++++++++++++++++++++++++++++- 1 file changed, 31 insertions(+), 1 deletion(-) (limited to 'app/Models/EntryDAO.php') diff --git a/app/Models/EntryDAO.php b/app/Models/EntryDAO.php index 36d633c6e..50bd0584c 100644 --- a/app/Models/EntryDAO.php +++ b/app/Models/EntryDAO.php @@ -1274,7 +1274,7 @@ SQL; /** * @param array $guids - * @return int|false The number of affected feeds, or false if error + * @return int|false The number of affected entries, or false if error */ public function updateLastSeen(int $id_feed, array $guids, int $mtime = 0) { if (count($guids) < 1) { @@ -1308,6 +1308,36 @@ SQL; } } + /** + * Update (touch) the last seen attribute of the latest entries of a given feed. + * Useful when a feed is unchanged / cached. + * @return int|false The number of affected entries, or false in case of error + */ + public function updateLastSeenUnchanged(int $id_feed, int $mtime = 0) { + $sql = <<pdo->prepare($sql); + if ($mtime <= 0) { + $mtime = time(); + } + if ($stm !== false && + $stm->bindValue(':mtime', $mtime, PDO::PARAM_INT) && + $stm->bindValue(':id_feed1', $id_feed, PDO::PARAM_INT) && + $stm->bindValue(':id_feed2', $id_feed, PDO::PARAM_INT) && + $stm->execute()) { + return $stm->rowCount(); + } else { + $info = $stm == null ? $this->pdo->errorInfo() : $stm->errorInfo(); + Minz_Log::error('SQL error ' . __METHOD__ . json_encode($info) . ' while updating feed ' . $id_feed); + return false; + } + } + /** @return array */ public function countUnreadRead(): array { $sql = <<<'SQL' -- cgit v1.2.3