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.

Sign up for GitHub

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

Jump to bottom

Column sorting is not saved after showing expired entries #8036

Closed
syakovyn opened this issue May 11, 2022 · 4 comments · Fixed by #8435
Closed

Column sorting is not saved after showing expired entries #8036

syakovyn opened this issue May 11, 2022 · 4 comments · Fixed by #8435
Assignees
@droidmonkey
Labels
bug user interface
Milestone
v2.7.2

Comments

@syakovyn
Copy link

Overview

I sort entries by Title. However, it is not kept after relaunching the app.

Before restarting the app:
image

After restarting the app:
image

Steps to Reproduce

  1. Start the app
  2. Open a group with entries.
  3. Click on a column header to sort by that column.
  4. Lock the database and exit or just immediately exit.
  5. Open the app again. Go to the same group.

Expected Behavior

Entries are sorted.

Actual Behavior

Entries are not sorted.

Context

Checked the suggestion from #7686 (comment) for ~/Library/Caches/KeePassXC/keepassxc.ini as I'm on Mac. But, it is not applicable as it is not on a RAM disk.
Tried removing keepassxc.ini but that didn't help either.

KeePassXC - Version 2.7.1
Revision: 5916a8f

Qt 5.15.2
Debugging mode is disabled.

Operating system: macOS 12.3
CPU architecture: arm64
Kernel: darwin 21.4.0

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • SSH Agent
  • KeeShare
  • YubiKey
  • Quick Unlock

Cryptographic libraries:

  • Botan 2.19.1
@syakovyn syakovyn added the bug label May 11, 2022
@droidmonkey
Copy link
Member

Delete the cache ini and reopen keepassxc, sometimes it gets corrupted.

@ba32107
Copy link
Contributor

ba32107 commented May 12, 2022

I have the same problem, on Ubuntu. I tried removing the cache ini, but it didn't help.

Steps I followed:

  1. KeePassXC open, sort order is messed up
  2. Sorted the columns
  3. Made a dummy change, so the DB is saved
  4. Deleted the cache ini
  5. Closed & reopened KeePassXC
  6. Expired entry virtual group shows
  7. When leaving the virtual group, order is messed up

If I take care of the expired entries (step 6 never happens), then the order is correct. So, I'm pretty sure the problem is related to the expired entry group.

@sandwm
Copy link

sandwm commented May 12, 2022

I can confirm that not showing the expired / about to expire entries works around the problem:
Tools -> Settings -> General -> disable "On database unlock, show entries that ..."

@droidmonkey
Copy link
Member

I am fixing this with my tag enhancement PR.

@droidmonkey droidmonkey self-assigned this May 14, 2022
@droidmonkey droidmonkey added this to the v2.7.2 milestone May 14, 2022
@droidmonkey droidmonkey changed the title Column sorting is not saved after exit Column sorting is not saved after showing expired entries Jun 16, 2022
droidmonkey added a commit that referenced this issue Sep 1, 2022
@droidmonkey
WIP: Enhance Tags / Saved Searches
da999f7 
TODO: Context menu to set tag on one or more entries

* Separate searching for all entries and resetting the search
* Support selecting multiple tags to search against
* Fix using escaped quotes in search terms
* Make tag searching more precise
* Support `is:expired-#` to search for entries expiring within # days. Exclude recycled entries from expired search.
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 1, 2022
@droidmonkey
WIP: Enhance Tags / Saved Searches
e1516cd 
TODO: Context menu to set tag on one or more entries

* Separate searching for all entries and resetting the search
* Support selecting multiple tags to search against
* Fix using escaped quotes in search terms
* Make tag searching more precise
* Support `is:expired-#` to search for entries expiring within # days. Exclude recycled entries from expired search.
* Fix #8036 - use search interface to display expiring entries on first unlock.
@droidmonkey droidmonkey mentioned this issue Sep 1, 2022
Merged
droidmonkey added a commit that referenced this issue Sep 3, 2022
@droidmonkey
Use search for showing expired entries on unlock
2114c7d 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 3, 2022
@droidmonkey
Use search for showing expired entries on unlock
202335c 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 5, 2022
@droidmonkey
Use search for showing expired entries on unlock
d36c42f 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 5, 2022
@droidmonkey
Use search for showing expired entries on unlock
17947bd 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 5, 2022
@droidmonkey
Use search for showing expired entries on unlock
f21bd3b 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 5, 2022
@droidmonkey
Use search for showing expired entries on unlock
127b9f0 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 5, 2022
@droidmonkey
Use search for showing expired entries on unlock
70b2cf0 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 7, 2022
@droidmonkey
Use search for showing expired entries on unlock
4ff0ed9 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 7, 2022
@droidmonkey
Use search for showing expired entries on unlock
720680e 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 7, 2022
@droidmonkey
Use search for showing expired entries on unlock
96f439a 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 7, 2022
@droidmonkey
Use search for showing expired entries on unlock
f91f9a8 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 7, 2022
@droidmonkey
Use search for showing expired entries on unlock
c1493a1 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 7, 2022
@droidmonkey
Use search for showing expired entries on unlock
ee64334 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 8, 2022
@droidmonkey
Use search for showing expired entries on unlock
e5bd5f3 
* Fix #8036 - use search interface to display expiring entries on first unlock.
pull bot pushed a commit to annihilatorrrr/keepassxc that referenced this issue Sep 8, 2022
@droidmonkey @pull
Use search for showing expired entries on unlock
eda21eb 
* Fix keepassxreboot#8036 - use search interface to display expiring entries on first unlock.
pull bot pushed a commit to tigerwill90/keepassxc that referenced this issue Sep 8, 2022
@droidmonkey @pull
Use search for showing expired entries on unlock
44077af 
* Fix keepassxreboot#8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 11, 2022
@droidmonkey
Use search for showing expired entries on unlock
0799010 
* Fix #8036 - use search interface to display expiring entries on first unlock.
droidmonkey added a commit that referenced this issue Sep 22, 2022
@droidmonkey
Use search for showing expired entries on unlock
de16895 
* Fix #8036 - use search interface to display expiring entries on first unlock.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@droidmonkey droidmonkey

Labels
bug user interface
Projects
None yet
Milestone
v2.7.2
Development

Successfully merging a pull request may close this issue.

Enhance Tags / Saved Searches keepassxreboot/keepassxc
4 participants
@syakovyn @droidmonkey @sandwm @ba32107