Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix filtered posts showing during "auto refresh" #255

Merged
merged 1 commit into from
Jan 9, 2023

Conversation

thiagojedi
Copy link

@thiagojedi thiagojedi commented Jan 3, 2023

I've been experiencing a bug where, after the app is idle for a while, filtered ("hide") posts are appearing in the home timeline. I could not reproduce in debug mode, but it is happening somewhat consistently in the Play Store apk for me.

I guess it is related to empty wordFilters list as it's the only difference between manual refresh and auto refresh. Is it possible that we're not calling maybeUpdateLocalInfo()?

Anyway, I removed the filters.isEmpty() validation so status with filtered can still be checked. The semantics is the same as the constructor StatusFilterPredicate(String accountID, Filter.FilterContext context) still checks for wordFilters.

@thiagojedi thiagojedi changed the title Fix filtered posts showing during "autoupdate" Fix filtered posts showing during "auto refresh" Jan 5, 2023
@sk22 sk22 merged commit 32bb3fa into sk22:main Jan 9, 2023
@sk22 sk22 mentioned this pull request Jan 9, 2023
@thiagojedi thiagojedi deleted the fix/4.0-filters-on-timeline-autoupdate branch January 9, 2023 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants