Fix line breaking about jabref

[stefan-kolb]

The bug should be caused by FieldContentParser.java

Test:

abstract = {The Quick Brown Fox Jumps Over The Lazy Dog. The Quick Brown Fox Jumps\r\n
    \tOver The Lazy Dog. The Quick Brown Fox Jumps Over The Lazy Dog. The

Code (near l.64):

 if (content.length() > i + 1 && content.charAt(i + 1) == '\t' && (content.length() == i + 2 || !Character.isWhitespace(content.charAt(i + 2)))) {
// We have either \n\t followed by non-whitespace, or \n\t at the
// end. Both cases indicate a wrap made by JabRef. Remove and insert space if necessary.
// @formatter:on
content.deleteCharAt(i); // \n
content.deleteCharAt(i); // \t
}

[stefan-kolb]

The question here is, which fields should accept line breaks?

[koppor]

All multiline fields 😂

Arent't they enlisted in the code somewhere?

Multiline: Abstract, Review
Unsure: comment
Non-multiline: all others

Side note: No one has complained about the difference between UI and serialization. To have the .bib files editable, I would keep the autowrap-feature (refs #123).

Update It seems that somehow the behavior changed from an old version (v2.9.2?) to v2.10 and JabRef removes all wraps and just has a single line. Maybe we should revert that behavior? Currently unsure.

[sbitzer]

Hi, I'm very much for keeping some line breaks in multiline fields. I previously used the review field to save my own thoughts about a paper. For some entries this has been quite a bit of text which I had separated into paragraphs by putting an empty line between paragraphs in the entry editor. I recently noticed that linebreaks are removed when the database is saved. When it is saved the first time, my empty lines are gone. When it is saved the second time, the remaining linebreaks are gone, too. Needless to say that I found that all my old reviews are only a single paragraph mess now. I don't think that this was an issue earlier and I'm a bit upset about the change in behaviour.

My jabref version is 2.10b2.

[koppor]

Related issue: https://sourceforge.net/p/jabref/feature-requests/854/

[lenhard]

Fixed #187