Support for $\backslash$ in file paths#9906
Conversation
| if (value == null) { | ||
| this.value = null; | ||
| } else { | ||
| this.value = value.replace("$\\backslash$", "\\"); |
There was a problem hiding this comment.
Why not call the latex 2 Unicode formatter to capture more cases
There was a problem hiding this comment.
I would need to rewrite the logic of the latextounicode formatter:
Expected :[ParsedFileField{description='', link='C:/Users/XXXXXX/AppData/Local/Mendeley Ltd./Mendeley Desktop/Downloaded/Brown - 2017 - Physical test methods for elastomers.pdf', fileType='pdf'}]
Actual :[ParsedFileField{description='', link='Cbackslash{}{', fileType='}{/}Users/XXXXXX/AppData/Local/Mendeley Ltd./Mendeley Desktop/Downloaded/Brown - 2017 - Physical test methods for elastomers.pdf'}]
Expected :[ParsedFileField{description='test:;', link='wei2005ahp.pdf', fileType='PDF'}]
Actual :[ParsedFileField{description='test ', link='wei2005ahp.pdf', fileType='PDF'}]
Should I commit nevertheless?
There was a problem hiding this comment.
I would go with the simple solution first and leave this whooe latex removing for later
There was a problem hiding this comment.
One would probaly treat each path segment separately by the formatter
There was a problem hiding this comment.
In the original example, there were no umlauts there. I was thinking: better a quick fix than some effort on catching the special cases 😇
Fixes https://discourse.jabref.org/t/mendeley-bib-import-with-linked-files/3470. Fixes https://github.com/JabRef/jabref-issue-melting-pot/issues/178.