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

Incorrect total entries count #510

Closed
pandafeliz opened this issue Sep 28, 2021 · 2 comments
Closed

Incorrect total entries count #510

pandafeliz opened this issue Sep 28, 2021 · 2 comments
Labels

Comments

@pandafeliz
Copy link

LRR Version and OS
Version 0.8.0, "Black Tie White Noise" OS: Windows 10

Bug Details
The total entries count shows a number above my current filecount, I executed the clean database and reset search cache commands but it remains the same. This was working correctly in 0.7.9 so not sure if is a visual bug or old entries that were deleted before and the clean database command can't catch.

Matching Logs
None

Screenshots

This is my current filecount
XYplorer_CRs1bxATmi

and this is what is displayed in the application without any filters applied

chrome_3e1Z9NXATb

as you can see the total filecount goes up but the filtered count is the correct one.

@Difegue
Copy link
Owner

Difegue commented Sep 28, 2021

I'm aware of this issue; I made the total count use the keycount of the redis database so I can save an extra DB call, but it's slightly incorrect due to the DB having a few more keys for configuration.

my $total = $redis->dbsize; #TODO this is incorrect by a few keys due to the config keys being strewn about on the DB root.

The total count is just used for display so there shouldn't be undeletable entries strewn about or anything like that. I'll figure out a better way to handle this in a next release.

@Difegue Difegue added the bug label Sep 28, 2021
@pandafeliz
Copy link
Author

alright, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants