-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Open
Labels
good second issueIssues that involve a tour of two or three interweaved components in JabRefIssues that involve a tour of two or three interweaved components in JabRef
Description
Context: #12532 (comment)
JabRef does not support \ to be used as escaping for the keyword separator
Task:
- Add test cases for
\,in org.jabref.model.entry.KeywordListTest - Adapt
org.jabref.model.entry.KeywordList#parse(java.lang.String, java.lang.Character, java.lang.Character)to implement parsing the escaping - Adapt
org.jabref.model.entry.KeywordList#serializeto escape properly
Please also keep in mind that the GUI renders differently than when serializing. This should be done if steps 2 and 3 were correctly done. -- You can double check with org.jabref.gui.fieldeditors.KeywordsEditorViewModel#serializeKeywords - and simply try out at the UI.
There was code proposed multiple times - but without any timely follow-up. Please ready through and implement a good solution
- good test cases https://github.com/JabRef/jabref/pull/12888/files#diff-405010f9d24d966c1c8d319857c458e7a81761922448a5ff981e645d4ad43efc.
- Add #12810: Implement escaping for keyword separators (FKA #12888) #13583
When working on this address #12888 (comment)
ryan-carpenter
Metadata
Metadata
Assignees
Labels
good second issueIssues that involve a tour of two or three interweaved components in JabRefIssues that involve a tour of two or three interweaved components in JabRef
Type
Projects
Status
Free to take