Keepassdx: Write failures not properly shown when saving an entry

Created on 12 Jan 2020  路  6Comments  路  Source: Kunzisoft/KeePassDX

Version 2.5.0beta25 from F-Droid via Android 5.1.1

To Reproduce
use KpDX to open a read-only file using a third party file explorer, and create an entry.

* maybe from Android itself might work. i used Material Files from f-droid.

saving an entry should save and let you know about the progress of it. if any errors occurs, the usual red bad should appear at the bottom like the other errors in the app.

what actually happens is that the progress bar flashes and nothing else happens, and the file is not edited, nor an error about the read-only state (or "failed to read") is shown.

bug completed developments

Most helpful comment

Indeed, I tried to resolve the bug quickly but there is more work than I thought. I will try to do a better management of the exceptions raised by the save service.

All 6 comments

Tell me if the problem is solved for you with the new version 2.5.0.0beta26?

the update hasn't shown up in F-Droid yet. unstable updates is enabled.

i'll get the github release

for saving new entries, edit current ones, creating groups, and save the database via the entry list, it is solved, although only the save database option has the "could not save database" text (others still have the java.io exception text)

the issue persists when editing database settings, such as passwords, compression, names and titles. (my fault, i didn't think of this when first writing this issue).

it likely needs a text for moving entries, too ("you can not move here" can be misleading in this case)

Indeed, I tried to resolve the bug quickly but there is more work than I thought. I will try to do a better management of the exceptions raised by the save service.

OK, it's solved!
Which text would you like for the error of moving entries?

the write failure message should be enough. case-specific errors like moving maybe should be of a lesser priority (as in which text should appear when two conflicts happens at once).

errors from moving would probably need some additional help texts or rephrased strings, too, but that would be something for another issue.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

gihucom picture gihucom  路  3Comments

kermit-frog picture kermit-frog  路  4Comments

stp-ip picture stp-ip  路  3Comments

adlisval picture adlisval  路  7Comments

axel-dd picture axel-dd  路  5Comments