client: Fix “Mark all as read” button not hiding marked articles

Appears to have been broken by f3b279c. Fixes: #1388
fossar · Nov 14, 2022 · ec544b7 · ec544b7
1 parent fe5c3d4
commit ec544b7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git 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
@@ -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);