"create backup file while saving" option is non-functional.

[buhtz]

JabRef 5.1--2020-03-10--d5ee20b
Windows 10 10.0 amd64
Java 13.0.2

In the settings I deactivated the "create backup file while saving" option.
I also deleted the *.sav file from the filesystem.

But the savfile is still created when starting JabRef.
When closing JabRef it is gone.

EDIT: The behaviour is not always reproducable. But please see the attached video which contains different behaviours.

6092.zip

Makes no sense for me.

[buhtz]

The problem is still reproducable with the current dev - also with the release version. I can not understand why there was a release in the last days but JabRef is still unstable.

[koppor]

The option "create backup file while saving" is for the .bak file, not for the .sav file. - Because of the while saving.

The issue is that the setting is without implementation. See

jabref/src/main/java/org/jabref/logic/exporter/AtomicFileOutputStream.java

Line 185 in 209d336

Files.copy(targetFile, backupFile, StandardCopyOption.REPLACE_EXISTING);

.

Options:

1. Remove setting from the GUI.
2. Remove creation of .bak while saving - accepting that the user cannot go back one revision manually.
3. Implement code to respect setting.

[buhtz]

What is the difference between bak and sav file?

[koppor]

sav == autosave each 15sec bak == version saved before last save. -- HEAD~ in nerd speak Reason for bak: non IT users can restore something if they accidentally saved. -- Very old functionality. Could be done differently these days. See IntelliJ for instance... Codeberg-AsGithubAlternative-buhtz <notifications@github.com> schrieb am Sa., 14. März 2020, 07:57:

…

What is the difference between bak and sav file? — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#6092 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAKNU7UT5PELJEV4B7YYQE3RHMTFPANCNFSM4LFAS3TQ> .

[stefan-kolb]

The option "create backup file while saving" is for the .bak file, not for the .sav file. - Because of the while saving.

The issue is that the setting is without implementation. See

jabref/src/main/java/org/jabref/logic/exporter/AtomicFileOutputStream.java

Line 185 in 209d336

Files.copy(targetFile, backupFile, StandardCopyOption.REPLACE_EXISTING);

.
Options:

1. Remove setting from the GUI.
2. Remove creation of .bak while saving - accepting that the user cannot go back one revision manually.
3. Implement code to respect setting.

I guess we can remove that option from the UI/preferences. @JabRef/developers WDYT? Consens or dev call?

[Siedlerchr]

I'm voting for remove

[calixtus]

If it's of no use, remove it.
There are probably some more things that could be removed from the preferences: Like the special option to select the sumatra pdf reader. This should work with the application binding in the external file types manager dialog...

[koppor]

Decision: Remove option, but keep functionality.