From 6e01866845b31d8d78c3a11bccde1594ce0119a2 Mon Sep 17 00:00:00 2001 From: Alexandre Alapetite Date: Tue, 30 Jul 2024 15:23:46 +0200 Subject: Fix regression on mark duplicate titles as read for modified articles (#6664) When the option *mark as read if an identical title already exists in the top n newest articles*, that should not apply to updated articles, as they risk being marked as read due to themselves having the same title. Identifying the different sub-cases would require a more complicated logic, so disable for now. Regression due to https://github.com/FreshRSS/FreshRSS/pull/6334 --- app/Controllers/feedController.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'app/Controllers/feedController.php') diff --git a/app/Controllers/feedController.php b/app/Controllers/feedController.php index 730c13a23..c8d534545 100644 --- a/app/Controllers/feedController.php +++ b/app/Controllers/feedController.php @@ -583,7 +583,8 @@ class FreshRSS_feed_Controller extends FreshRSS_ActionController { continue; } - $entry->applyFilterActions($titlesAsRead); + // NB: Do not mark updated articles as read based on their title, as the duplicate title maybe be from the same article. + $entry->applyFilterActions([]); if ($readWhenSameTitleInFeed > 0) { $titlesAsRead[$entry->title()] = true; } -- cgit v1.2.3