default backup path folder not initialized with wallet folder

Bug #1870433 reported by Jeff Fortin Tam
22
This bug affects 3 people
Affects Status Importance Assigned to Milestone
HomeBank
Fix Released
Low
Maxime DOYEN

Bug Description

Running Homebank 5.3.1 on Fedora 31.

I noticed that BackupPath defaults to the user's home directory. This is a little bit weird because I would either expect the backup files to be in same directory as the files they are backuping, or in a dedicated backup folder. Otherwise, if your main xhb files are not in the root of the home directory, it's a bit weird to have a bunch of .bak files laying around in plain sight.

If having the backup files in the same directory of the file they are backuping is not desirable, I'm thinking the XDG "shared data" folder (which would be, typically, ~/.local/share/homebank/backups/) might be more appropriate than "~" ? See also https://specifications.freedesktop.org/basedir-spec/latest/ (but as far as I can tell, Homebank already uses the XDG config directory for its preferences files, so I presume you know about the XDG user directories spec).

Revision history for this message
Maxime DOYEN (mdoyen) wrote :

The default backup folder is in fact the same than the default wallet folder.

I simply introduced a different path in 5.3 with #1847645, and forgot to initialize it with the wallet folder path that existing old user maybe have changed.

Revision history for this message
Maxime DOYEN (mdoyen) wrote :

I did fix the initialisation of backup folder for users < 5.3, it will copy the wallet folder, so that we keep the old behavior

Changed in homebank:
assignee: nobody → Maxime DOYEN (mdoyen)
importance: Undecided → Low
milestone: none → 5.4
status: New → Fix Committed
summary: - Default backup path folder location is the home directory
+ default backup path folder not initialized with wallet folder
Maxime DOYEN (mdoyen)
Changed in homebank:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.