diff --git a/jabkit/src/main/java/org/jabref/cli/CheckConsistency.java b/jabkit/src/main/java/org/jabref/cli/CheckConsistency.java index 5fdc327435c..5afff87aa70 100644 --- a/jabkit/src/main/java/org/jabref/cli/CheckConsistency.java +++ b/jabkit/src/main/java/org/jabref/cli/CheckConsistency.java @@ -66,7 +66,9 @@ public void run() { BibliographyConsistencyCheck consistencyCheck = new BibliographyConsistencyCheck(); BibliographyConsistencyCheck.Result result = consistencyCheck.check(entries, (count, total) -> { - System.out.println(Localization.lang("Checking consistency for entry type %0 of %1", count + 1, total)); + if (!sharedOptions.porcelain) { + System.out.println(Localization.lang("Checking consistency for entry type %0 of %1", count + 1, total)); + } }); Writer writer = new OutputStreamWriter(System.out); diff --git a/jabkit/src/test/java/org/jabref/cli/ArgumentProcessorTest.java b/jabkit/src/test/java/org/jabref/cli/ArgumentProcessorTest.java index 9bb546ae28b..e0146fcfdbb 100644 --- a/jabkit/src/test/java/org/jabref/cli/ArgumentProcessorTest.java +++ b/jabkit/src/test/java/org/jabref/cli/ArgumentProcessorTest.java @@ -146,6 +146,7 @@ void checkConsistency() throws URISyntaxException { commandLine.execute(args.toArray(String[]::new)); String output = outContent.toString(); + assertTrue(output.contains("Checking consistency for entry type 1 of 1\n")); assertTrue(output.contains("Consistency check completed")); System.setOut(System.out); @@ -165,7 +166,7 @@ void checkConsistencyPorcelain() throws URISyntaxException { commandLine.execute(args.toArray(String[]::new)); String output = outContent.toString(); - assertEquals("Checking consistency for entry type 1 of 1\n", output); + assertEquals("", output); System.setOut(System.out); } diff --git a/jablib/src/main/java/org/jabref/logic/quality/consistency/BibliographyConsistencyCheck.java b/jablib/src/main/java/org/jabref/logic/quality/consistency/BibliographyConsistencyCheck.java index c233f67a4dd..ca0f2691da5 100644 --- a/jablib/src/main/java/org/jabref/logic/quality/consistency/BibliographyConsistencyCheck.java +++ b/jablib/src/main/java/org/jabref/logic/quality/consistency/BibliographyConsistencyCheck.java @@ -83,13 +83,13 @@ private static void collectEntriesIntoMaps(List entries, Map fieldsInAnyEntry = entryTypeToFieldsInAnyEntryMap.computeIfAbsent(entryType, k -> new HashSet<>()); + Set fieldsInAnyEntry = entryTypeToFieldsInAnyEntryMap.computeIfAbsent(entryType, _ -> new HashSet<>()); fieldsInAnyEntry.addAll(entry.getFields()); - Set fieldsInAllEntries = entryTypeToFieldsInAllEntriesMap.computeIfAbsent(entryType, k -> new HashSet<>(entry.getFields())); + Set fieldsInAllEntries = entryTypeToFieldsInAllEntriesMap.computeIfAbsent(entryType, _ -> new HashSet<>(entry.getFields())); fieldsInAllEntries.retainAll(entry.getFields()); - Set entriesOfType = entryTypeToEntriesMap.computeIfAbsent(entryType, k -> new HashSet<>()); + Set entriesOfType = entryTypeToEntriesMap.computeIfAbsent(entryType, _ -> new HashSet<>()); entriesOfType.add(entry); } }