Fix threading issue in abbreviate action

Fixes #12634

Author: Siedlerchr

CHANGELOG.md

@@ -75,7 +75,8 @@ Note that this project **does not** adhere to [Semantic Versioning](https://semv
 - We fixed an issue where month values 21–24 (ISO 8601-2019 season codes) in Biblatex date fields were not recognized as seasons during parsing. [#12437](https://github.com/JabRef/jabref/issues/12437)
 - We fixed an issue where migration of "Search groups" would fail with an exception when the search query is invalid. [#12555](https://github.com/JabRef/jabref/issues/12555)
 - We fixed an issue where not all linked files from BibDesk in the field `bdsk-file-...` were parsed. [#12555](https://github.com/JabRef/jabref/issues/12555)
-- We fixed an issue where JabRef displayed an incorrect deletion notification when canceling entry deletion [#12645](https://github.com/JabRef/jabref/issues/12645)
+- We fixed an issue where JabRef displayed an incorrect deletion notification when canceling entry deletion. [#12645](https://github.com/JabRef/jabref/issues/12645)
+- We fixed an issue where an exception would occur when running abbreviate journals for multiple entries. [#12634](https://github.com/JabRef/jabref/issues/12634)
 
 ### Removed
 

src/main/java/org/jabref/gui/journals/AbbreviateAction.java

@@ -104,24 +104,9 @@ private String abbreviate(BibDatabaseContext databaseContext, List<BibEntry> ent
 
         NamedCompound ce = new NamedCompound(Localization.lang("Abbreviate journal names"));
 
-        // Collect all callables to execute in one collection.
-        Set<Callable<Boolean>> tasks = entries.stream().<Callable<Boolean>>map(entry -> () ->
-                FieldFactory.getJournalNameFields().stream().anyMatch(journalField ->
-                        undoableAbbreviator.abbreviate(databaseContext.getDatabase(), entry, journalField, ce)))
-                .collect(Collectors.toSet());
-
-        // Execute the callables and wait for the results.
-        List<Future<Boolean>> futures = HeadlessExecutorService.INSTANCE.executeAll(tasks);
-
-        // Evaluate the results of the callables.
-        long count = futures.stream().filter(future -> {
-            try {
-                return future.get();
-            } catch (InterruptedException | ExecutionException exception) {
-                LOGGER.error("Unable to retrieve value.", exception);
-                return false;
-            }
-        }).count();
+        int count = entries.stream().mapToInt(entry ->
+                (int) FieldFactory.getJournalNameFields().stream().filter(journalField ->
+                        undoableAbbreviator.abbreviate(databaseContext.getDatabase(), entry, journalField, ce)).count()).sum();
 
         if (count == 0) {
             return Localization.lang("No journal names could be abbreviated.");