diff options
| author | 2024-07-30 15:23:46 +0200 | |
|---|---|---|
| committer | 2024-07-30 15:23:46 +0200 | |
| commit | 6e01866845b31d8d78c3a11bccde1594ce0119a2 (patch) | |
| tree | 0f85c94686bcfdac24b5bbcfe238de6075c4b88c /app/Controllers | |
| parent | 5659b37948c17e5e68971a2464930bee3455dfa1 (diff) | |
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
Diffstat (limited to 'app/Controllers')
| -rw-r--r-- | app/Controllers/feedController.php | 3 |
1 files changed, 2 insertions, 1 deletions
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; } |
