From ec544b7ae066050dcdb43e362f38854d4fdb7bea Mon Sep 17 00:00:00 2001 From: Jan Tojnar Date: Mon, 14 Nov 2022 04:58:48 +0100 Subject: [PATCH] =?UTF-8?q?client:=20Fix=20=E2=80=9CMark=20all=20as=20read?= =?UTF-8?q?=E2=80=9D=20button=20not=20hiding=20marked=20articles?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Appears to have been broken by https://github.com/fossar/selfoss/commit/f3b279c808e8a5d7e3ab5c8254bdda83c094b50e. Fixes: https://github.com/fossar/selfoss/issues/1388 --- NEWS.md | 1 + assets/js/templates/EntriesPage.jsx | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/NEWS.md b/NEWS.md index 29c4b89582..18a7d8d82b 100644 --- a/NEWS.md +++ b/NEWS.md @@ -5,6 +5,7 @@ ### Bug fixes - Configuration parser was changed to *raw* method, which relaxes the requirement to quote option values containing special characters in `config.ini`. ([#1371](https://github.com/fossar/selfoss/issues/1371)) +- Fix “Mark all as read” button not hiding marked articles in unread view. ([#1388](https://github.com/fossar/selfoss/issues/1388)) ### Other changes - `tidy` PHP extension is now required if you want to use “Content extractor” spout. ([#1392](https://github.com/fossar/selfoss/pull/1392)) diff --git a/assets/js/templates/EntriesPage.jsx b/assets/js/templates/EntriesPage.jsx index 97ca84c5e4..48eff9b7b9 100644 --- a/assets/js/templates/EntriesPage.jsx +++ b/assets/js/templates/EntriesPage.jsx @@ -706,8 +706,8 @@ export default class StateHolder extends React.Component { // close opened entry and list this.setExpandedEntries({}); - if (ids.length !== 0 && this.props.match.filter === FilterType.UNREAD) { - markedEntries = markedEntries.filter(({ id }) => ids.includes(id)); + if (ids.length !== 0 && this.props.match.params.filter === FilterType.UNREAD) { + markedEntries = markedEntries.filter(({ id }) => !ids.includes(id)); } this.setLoadingState(LoadingState.LOADING);