saving destroyed library: error when using a PDF file name with opening "{" or closing "}" ONLY inside file name

[ghost]

JabRef version

5.9 (latest release)

Operating system

Windows

Details on version and operating system

Windows 10 22H2

Checked with the latest development build

• I made a backup of my libraries before testing the latest development version.
• I have tested the latest development version and the problem persists

Steps to reproduce the behaviour

1. Create new library
2. Create test PDF file with opening "{" or closing "}" ONLY inside file name
3. Add new library entry
4. Assign this PDF file to the "file" field of the entry
5. Save the library
6. Error occurs:

Opening { error:

Closing } error:

When you try to open the library file (after closing JabRef), the library file is destroyed somewhere and can't be opened anymore!!!

The following errors occur when opening the library file again

Appendix

...

Log File

Paste an excerpt of your log file here

[Siedlerchr]

Uh yeah, that should not be allowed in the file field. Braces are bibtex' separators for the field values.
You can try to open it with a text editor (e.g. Notepad) and manually remove the offending braces, then it should be able to open again

[ghost]

Should jabref give message box maybe if you use not allowed characters on the file?

[Siedlerchr]

I could reproduce this in the latest dev version as well.
Although I say discard changes, the changes are still present and it kills the rest of the library entries

org.jabref.logic.exporter.SaveException: Problems saving: java.io.IOException: Error in field 'FILE of entry Kriegman1965': Braces don't match. Field value: :broken{ asdf.pdf:PDF
	at [email protected]/org.jabref.gui.exporter.SaveDatabaseAction.saveDatabase(SaveDatabaseAction.java:263)
	at [email protected]/org.jabref.gui.exporter.SaveDatabaseAction.save(SaveDatabaseAction.java:213)
	at [email protected]/org.jabref.gui.exporter.SaveDatabaseAction.save(SaveDatabaseAction.java:187)
	at [email protected]/org.jabref.gui.exporter.SaveDatabaseAction.save(SaveDatabaseAction.java:74)
	at [email protected]/org.jabref.gui.exporter.SaveAction.execute(SaveAction.java:41)
	at [email protected]/org.jabref.gui.actions.JabRefAction.lambda$new$3(JabRefAction.java:40)
	at [email protected]/org.controlsfx.control.action.Action.handle(Action.java:423)
	at [email protected]/org.controlsfx.control.action.Action.handle(Action.java:64)
	at javafx.base@20/com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(CompositeEventHandler.java:86)
	at javafx.base@20/com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:232)
	at javafx.base@20/com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:189)
	at javafx.base@20/com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:58)
	at javafx.base@20/com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
	at javafx.base@20/com.sun.javafx.event.EventUtil.fireEventImpl(EventUtil.java:74)
	at javafx.base@20/com.sun.javafx.event.EventUtil.fireEvent(EventUtil.java:49)
	at javafx.base@20/javafx.event.Event.fireEvent(Event.java:198)
	at javafx.controls@20/javafx.scene.control.MenuItem.fire(MenuItem.java:459)
	at javafx.controls@20/com.sun.javafx.scene.control.GlobalMenuAdapter.lambda$bindMenuItemProperties$2(GlobalMenuAdapter.java:150)
	at javafx.base@20/com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(CompositeEventHandler.java:86)
	at javafx.base@20/com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:232)
	at javafx.base@20/com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:189)
	at javafx.base@20/com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:58)
	at javafx.base@20/com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114)
	at javafx.base@20/com.sun.javafx.event.EventUtil.fireEventImpl(EventUtil.java:74)
	at javafx.base@20/com.sun.javafx.event.EventUtil.fireEvent(EventUtil.java:54)
	at javafx.base@20/javafx.event.Event.fireEvent(Event.java:198)
	at javafx.controls@20/javafx.scene.control.MenuItem.fire(MenuItem.java:459)
	at javafx.graphics@20/com.sun.javafx.tk.quantum.GlassSystemMenu$1.action(GlassSystemMenu.java:234)
Caused by: java.io.IOException: Error in field 'FILE of entry Kriegman1965': Braces don't match. Field value: :broken{ asdf.pdf:PDF
	at [email protected]/org.jabref.logic.bibtex.BibEntryWriter.writeField(BibEntryWriter.java:169)
	at [email protected]/org.jabref.logic.bibtex.BibEntryWriter.writeRequiredFieldsFirstRemainingFieldsSecond(BibEntryWriter.java:138)
	at [email protected]/org.jabref.logic.bibtex.BibEntryWriter.write(BibEntryWriter.java:73)
	at [email protected]/org.jabref.logic.exporter.BibtexDatabaseWriter.writeEntry(BibtexDatabaseWriter.java:158)
	at [email protected]/org.jabref.logic.exporter.BibDatabaseWriter.savePartOfDatabase(BibDatabaseWriter.java:228)
	at [email protected]/org.jabref.logic.exporter.BibDatabaseWriter.saveDatabase(BibDatabaseWriter.java:180)
	at [email protected]/org.jabref.gui.exporter.SaveDatabaseAction.saveDatabase(SaveDatabaseAction.java:252)
	... 27 more
Caused by: org.jabref.logic.bibtex.InvalidFieldValueException: Braces don't match. Field value: :broken{ asdf.pdf:PDF
	at [email protected]/org.jabref.logic.bibtex.FieldWriter.checkBraces(FieldWriter.java:71)
	at [email protected]/org.jabref.logic.bibtex.FieldWriter.formatWithoutResolvingStrings(FieldWriter.java:177)
	at [email protected]/org.jabref.logic.bibtex.FieldWriter.write(FieldWriter.java:89)
	at [email protected]/org.jabref.logic.bibtex.BibEntryWriter.writeField(BibEntryWriter.java:166)
	... 33 more

[Siedlerchr]

log.txt
Bakup files is still present https://docs.jabref.org/advanced/autosave#where-can-i-find-the-backup-files

[Siedlerchr]

Solution would be to check the filename on adding and trying to rename it to allowed characters

[ghost]

Solution would be to check the filename on adding and trying to rename it to allowed characters

Yes! This is why I reported it. It destroys your library. It happened here by accident when inserting a "{".
I have restored using my backup.

Maybe even more other characters in the file name can have conflicts?