Backups: Handle discarded changes v2

[mmohanb1]

Fixes #9361

• Change in CHANGELOG.md described in a way that is understandable for the average user (if applicable)
• Tests created for changes (if applicable)
• Manually tested changed features in running JabRef (always required)
• Screenshots added in PR description (for UI changes)
• Checked developer's documentation: Is the information available and up to date? If not, I outlined it in this pull request.
• Checked documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request to the documentation repository.

[koppor]

@mmohanb1 Thank you for continuously working on it. For the others, you could have explained why you opened a new pull request. You could also have done a force-push to the existing branch to discard all old commits.

Based on our discussions at #9361 and seeing the new release coming up soon, three members of the core team sat together this evening and worked on fixing the issue. It turned out that it was much harder than we thought. The test cases were very difficult to craft. It are more than 100 lines --> https://github.com/JabRef/jabref/pull/9458/files#diff-a7050d7b065fe4dfb3b7d7b02f0c4aa1743ae657992a882b0b4056c61c19275a.

The whole solution is available at #9458. Feel free to comment on the implementation or to add some more test cases.

Since the other pull request is more advanced (checkstyle is OK, test cases exist, seems to work).

[mmohanb1]

Thank you Oliver.

…

On Thu, Dec 15, 2022 at 7:40 PM Oliver Kopp ***@***.***> wrote: @mmohanb1 <https://github.com/mmohanb1> Thank you for continuously working on it. For the others, you could have explained why you opened a new pull request. You could also have done a force-push to the existing branch to discard all old commits. Based on our discussions at #9361 <#9361> and seeing the new release coming up soon, three members of the core team sat together this evening and worked on fixing the issue. It turned out that it was much harder than we thought. The test cases were very difficult to craft. It are more than 100 lines --> https://github.com/JabRef/jabref/pull/9458/files#diff-a7050d7b065fe4dfb3b7d7b02f0c4aa1743ae657992a882b0b4056c61c19275a . The whole solution is available at #9458 <#9458>. Feel free to comment on the implementation or to add some more test cases. Since the other pull request is more advanced (checkstyle is OK, test cases exist, seems to work). — Reply to this email directly, view it on GitHub <#9457 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AVTCRF577PE4MTGRHC4SETTWNO3BDANCNFSM6AAAAAATAILWYU> . You are receiving this because you were mentioned.Message ID: ***@***.***>