Oliver Kopp

Sanitize input for substring$

1

Small follow-up to https://github.com/JabRef/jabref/pull/11304. Should not change any functionality, but cleaner code. ### Mandatory checks - [ ] Change in `CHANGELOG.md` described in a way that is understandable for the...

Entries last recently added from an import should be assigned to this group.

One enters a a URL to a .bib file 

Current BibDesk behavior:    

This is more a discussion. I am applying [prettier-java](https://github.com/jhipster/prettier-java) to my project (https://github.com/JabRef/jabref/pull/10646). It turned out best (in comparison to GJF, palantir-java, spring-java-style). It has very good review-friendly line wrapping....

ReplaceLambdaWithMethodReference with two arguments

1

In case the method has two arguments and the first one is a lambda, then `ReplaceLambdaWithMethodReference` produces ``. ```diff - .filtered(Bindings.createObjectBinding(() -> this::showNode, filter)); + .filtered(Bindings.createObjectBinding(this::/*__TEMPLATE_STOP__*/ + , filter)); ```...

ReplaceStringBuilderWithString should keep newlines

8

```diff - String content = new StringBuilder() - .append(Localization.lang("A backup file for '%0' was found at [%1]", + String content = Localization.lang("A backup file for '%0' was found at [%1]",...

ReplaceStackWithDeque sometimes fails with an exception

1

> org.openrewrite.internal.RecipeRunException: Exception while visiting project file 'src\main\java\org\jabref\model\openoffice\ootext\OOTextI ntoOO.java (in OOTextIntoOO.write)', caused by: org.openrewrite.java.controlflow.ControlFlowIllegalStateException: Unable to create new J.VariableDeclarations with name `pairIterator` File: https://github.com/JabRef/jabref/blob/main/src/main/java/org/jabref/model/openoffice/ootext/OOTextIntoOO.java ``` org.openrewrite.internal.RecipeRunException: Exception while visiting project...

MinimumSwitchCases should keep comments

2

Current: ```diff - switch (keyBinding.get()) { - case DELETE_ENTRY: - new DeleteFileAction(dialogService, preferencesService, databaseContext, - viewModel, listView).execute(); - event.consume(); - break; - default: - // Pass other keys to children...

I experience the output of ´StandardCharsets.` and I don't know why. Old: ```java metaList.add(XmpUtilShared.parseXmpMetadata(new ByteArrayInputStream(xmpMetaString.getBytes()))); ``` https://github.com/JabRef/jabref/blob/913b46cf0c31a4f7f73c84e4a8584c9adb8c701f/src/main/java/org/jabref/logic/xmp/XmpUtilReader.java#L130 New: ```java metaList.add(XmpUtilShared.parseXmpMetadata(new ByteArrayInputStream(xmpMetaString.getBytes(StandardCharsets.)))); ```