Reformat codebase

.jbang/CheckoutPR.java

@@ -15,7 +15,7 @@
 import org.kohsuke.github.GitHubBuilder;
 import org.kohsuke.github.PagedIterator;
 
-///usr/bin/env jbang "$0" "$@" ; exit $?
+/// usr/bin/env jbang "$0" "$@" ; exit $?
 
 //JAVA 21+
 //RUNTIME_OPTIONS --enable-native-access=ALL-UNNAMED
@@ -62,7 +62,7 @@ public static void main(String[] args) throws Exception {
             while (prIterator.hasNext()) {
                 pr = prIterator.next();
                 if ((contributor.isEmpty() || pr.getHead().getUser().getLogin().equals(contributor)) &&
-                    pr.getHead().getRef().equals(branchName)) {
+                        pr.getHead().getRef().equals(branchName)) {
                     found = true;
                     System.out.println("Found pull request #" + pr.getNumber());
                     break;
@@ -165,9 +165,9 @@ private static void checkoutUpstreamMain() throws Exception {
             // Check if a remote pointing to JabRef/jabref already exists
             List<RemoteConfig> remotes = git.remoteList().call();
             Optional<RemoteConfig> jabrefRemote = remotes.stream()
-                // We use "contains", because there could be SSH remote URLs
-                .filter(r -> r.getURIs().stream().anyMatch(uri -> uri.toString().contains("JabRef/jabref")))
-                .findFirst();
+                                                         // We use "contains", because there could be SSH remote URLs
+                                                         .filter(r -> r.getURIs().stream().anyMatch(uri -> uri.toString().contains("JabRef/jabref")))
+                                                         .findFirst();
 
             String remoteToUse;
             if (jabrefRemote.isPresent()) {
@@ -176,9 +176,9 @@ private static void checkoutUpstreamMain() throws Exception {
             } else {
                 System.out.println("Adding remote 'upstream' pointing to " + jabrefRepoUrl);
                 git.remoteAdd()
-                    .setName(upstreamName)
-                    .setUri(new URIish(jabrefRepoUrl))
-                    .call();
+                   .setName(upstreamName)
+                   .setUri(new URIish(jabrefRepoUrl))
+                   .call();
                 remoteToUse = upstreamName;
             }
 

.jbang/CloneJabRef.java

@@ -1,7 +1,7 @@
 import java.nio.file.Files;
 import java.nio.file.Path;
 
-///usr/bin/env jbang "$0" "$@" ; exit $?
+/// usr/bin/env jbang "$0" "$@" ; exit $?
 
 //JAVA 21+
 //RUNTIME_OPTIONS --enable-native-access=ALL-UNNAMED

.jbang/JabKitLauncher.java

@@ -1,4 +1,4 @@
-///usr/bin/env jbang "$0" "$@" ; exit $?
+/// usr/bin/env jbang "$0" "$@" ; exit $?
 
 //DESCRIPTION jabkit - mange BibTeX files using JabRef
 

.jbang/JabLsLauncher.java

@@ -1,4 +1,4 @@
-///usr/bin/env jbang "$0" "$@" ; exit $?
+/// usr/bin/env jbang "$0" "$@" ; exit $?
 
 //DESCRIPTION jabls - start a bibtex languageserver
 

.jbang/JabSrvLauncher.java

@@ -1,4 +1,4 @@
-///usr/bin/env jbang "$0" "$@" ; exit $?
+/// usr/bin/env jbang "$0" "$@" ; exit $?
 
 //DESCRIPTION jabsrv - serve BibTeX files using JabRef
 

build-support/src/main/java/CitationStyleCatalogGenerator.java

@@ -1,4 +1,4 @@
-///usr/bin/env jbang "$0" "$@" ; exit $?
+/// usr/bin/env jbang "$0" "$@" ; exit $?
 
 //JAVA 24
 //RUNTIME_OPTIONS --enable-native-access=ALL-UNNAMED

build-support/src/main/java/JournalListMvGenerator.java

@@ -1,4 +1,4 @@
-///usr/bin/env jbang "$0" "$@" ; exit $?
+/// usr/bin/env jbang "$0" "$@" ; exit $?
 
 //JAVA 24
 //RUNTIME_OPTIONS --enable-native-access=ALL-UNNAMED
@@ -44,7 +44,6 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-
 /// Has to be started in the root of the repository due to <https://github.com/jbangdev/jbang-gradle-plugin/issues/11>
 public class JournalListMvGenerator {
 

build-support/src/main/java/LtwaListMvGenerator.java

@@ -1,4 +1,4 @@
-///usr/bin/env jbang "$0" "$@" ; exit $?
+/// usr/bin/env jbang "$0" "$@" ; exit $?
 
 //JAVA 24
 //RUNTIME_OPTIONS --enable-native-access=ALL-UNNAMED

config/checkstyle/checkstyle.xml

@@ -135,7 +135,9 @@
               public String getTabName() { return Localization.lang("XMP metadata"); }
 
         -->
-        <module name="LeftCurly"/>
+        <module name="LeftCurly">
+            <property name="id" value="LeftCurly"/>
+        </module>
 
         <module name="RightCurly"/>
 

config/checkstyle/suppressions.xml

@@ -15,4 +15,5 @@
     <suppress checks="[a-zA-Z0-9]*" files="CustomTitledPaneSkin.java" />
     <!-- Following contain examples or comments with "}}" that are necessary -->
     <suppress id="NoMultipleClosingBracesAtEndOfLine" files="(BibtexParserTest|StringUtil).java" />
+    <suppress id="LeftCurly" files="(BibtexParser|BstFunctions|CitationKeyGenerator|GenerateSummaryTask|INSPIREBibtexFilterReader|LinkedFileHandler).java" />
 </suppressions>

jabgui/src/main/java/org/jabref/cli/CliImportHelper.java

@@ -30,8 +30,8 @@ public class CliImportHelper {
      * @param location URL or file path to import
      */
     public static Optional<ParserResult> importFile(String location,
-                                             CliPreferences cliPreferences,
-                                             boolean porcelain) {
+                                                    CliPreferences cliPreferences,
+                                                    boolean porcelain) {
         LOGGER.debug("Importing file from locaiton {}", location);
         String[] data = location.split(",");
 
@@ -42,7 +42,7 @@ public static Optional<ParserResult> importFile(String location,
             try {
                 file = new URLDownload(address).toTemporaryFile();
             } catch (FetcherException |
-                    MalformedURLException e) {
+                     MalformedURLException e) {
                 System.err.println(Localization.lang("Problem downloading from %0: %1", address, e.getLocalizedMessage()));
                 return Optional.empty();
             }
@@ -63,9 +63,9 @@ public static Optional<ParserResult> importFile(String location,
         return importResult;
     }
 
-   public static Optional<ParserResult> importFile(Path file,
-                                             CliPreferences cliPreferences,
-                                             boolean porcelain) {
+    public static Optional<ParserResult> importFile(Path file,
+                                                    CliPreferences cliPreferences,
+                                                    boolean porcelain) {
         try {
             ImportFormatReader importFormatReader = new ImportFormatReader(
                     cliPreferences.getImporterPreferences(),

jabgui/src/main/java/org/jabref/gui/JabRefDialogService.java

@@ -446,23 +446,23 @@ public void notify(String message) {
         LOGGER.info(message);
 
         UiTaskExecutor.runInJavaFXThread(() ->
-            Notifications.create()
-                         .text(message)
-                         .position(Pos.BOTTOM_CENTER)
-                         .hideAfter(TOAST_MESSAGE_DISPLAY_TIME)
-                         .owner(mainWindow)
-                         .threshold(5,
-                            Notifications.create()
-                                         .title(Localization.lang("Last notification"))
-                                         .text(
-                                            "(" + Localization.lang("Check the event log to see all notifications") + ")"
-                                            + "\n\n" + message)
-                                         .onAction(e -> {
-                                            ErrorConsoleAction ec = new ErrorConsoleAction();
-                                            ec.execute();
-                                         }))
-                         .hideCloseButton()
-                         .show());
+                Notifications.create()
+                             .text(message)
+                             .position(Pos.BOTTOM_CENTER)
+                             .hideAfter(TOAST_MESSAGE_DISPLAY_TIME)
+                             .owner(mainWindow)
+                             .threshold(5,
+                                     Notifications.create()
+                                                  .title(Localization.lang("Last notification"))
+                                                  .text(
+                                                          "(" + Localization.lang("Check the event log to see all notifications") + ")"
+                                                                  + "\n\n" + message)
+                                                  .onAction(e -> {
+                                                      ErrorConsoleAction ec = new ErrorConsoleAction();
+                                                      ec.execute();
+                                                  }))
+                             .hideCloseButton()
+                             .show());
     }
 
     @Override

jabgui/src/main/java/org/jabref/gui/JabRefGUI.java

@@ -326,7 +326,7 @@ public void onShowing(WindowEvent event) {
 
         // Open last edited databases
         if (uiCommands.stream().noneMatch(UiCommand.BlankWorkspace.class::isInstance)
-            && preferences.getWorkspacePreferences().shouldOpenLastEdited()) {
+                && preferences.getWorkspacePreferences().shouldOpenLastEdited()) {
             mainFrame.openLastEditedDatabases();
         }
 

jabgui/src/main/java/org/jabref/gui/LibraryTab.java

@@ -205,8 +205,8 @@ private LibraryTab(@NonNull BibDatabaseContext bibDatabaseContext,
 
         stateManager.activeDatabaseProperty().addListener((_, _, _) -> {
             if (preferences.getSearchPreferences().isFulltext()) {
-              mainTable.getTableModel().refreshSearchMatches();
-           }
+                mainTable.getTableModel().refreshSearchMatches();
+            }
         });
     }
 
@@ -968,7 +968,7 @@ public void resetChangedProperties() {
      * Creates a new library tab. Contents are loaded by the {@code dataLoadingTask}. Most of the other parameters are required by {@code resetChangeMonitor()}.
      *
      * @param dataLoadingTask The task to execute to load the data asynchronously.
-     * @param file the path to the file (loaded by the dataLoadingTask)
+     * @param file            the path to the file (loaded by the dataLoadingTask)
      */
     public static LibraryTab createLibraryTab(BackgroundTask<ParserResult> dataLoadingTask,
                                               Path file,

jabgui/src/main/java/org/jabref/gui/ai/components/aichat/AiChatComponent.java

@@ -95,8 +95,8 @@ public AiChatComponent(AiService aiService,
         aiService.getIngestionService().ingest(name, ListUtil.getLinkedFiles(entries).toList(), bibDatabaseContext);
 
         ViewLoader.view(this)
-                .root(this)
-                .load();
+                  .root(this)
+                  .load();
     }
 
     @FXML
@@ -215,10 +215,11 @@ private List<Notification> updateNotificationsForEntry(BibEntry entry) {
 
         entry.getFiles().stream().map(file -> aiService.getIngestionService().ingest(file, bibDatabaseContext)).forEach(ingestionStatus -> {
             switch (ingestionStatus.getState()) {
-                case PROCESSING -> notifications.add(new Notification(
-                    Localization.lang("File %0 is currently being processed", ingestionStatus.getObject().getLink()),
-                    Localization.lang("After the file is ingested, you will be able to chat with it.")
-                ));
+                case PROCESSING ->
+                        notifications.add(new Notification(
+                                Localization.lang("File %0 is currently being processed", ingestionStatus.getObject().getLink()),
+                                Localization.lang("After the file is ingested, you will be able to chat with it.")
+                        ));
 
                 case ERROR -> {
                     assert ingestionStatus.getException().isPresent(); // When the state is ERROR, the exception must be present.
@@ -229,7 +230,8 @@ private List<Notification> updateNotificationsForEntry(BibEntry entry) {
                     ));
                 }
 
-                case SUCCESS -> { }
+                case SUCCESS -> {
+                }
             }
         });
 

jabgui/src/main/java/org/jabref/gui/ai/components/aichat/chatprompt/ChatPromptComponent.java

@@ -127,9 +127,9 @@ private void initialize() {
                     // The easy way to get rid of this ambiguity is to disallow scrolling when there are new lines in the prompt.
                     // But the exception to this situation is when the caret position is at the beginning of the prompt.
                     history.get().stream()
-                            .skip(newValue.intValue())
-                            .findFirst()
-                            .ifPresent(message -> userPromptTextArea.setText(message));
+                           .skip(newValue.intValue())
+                           .findFirst()
+                           .ifPresent(message -> userPromptTextArea.setText(message));
                 }
             } else {
                 // When currentUserMessageScroll is set to NEW_NON_EXISTENT_MESSAGE, then we should:

jabgui/src/main/java/org/jabref/gui/ai/components/util/notifications/Notification.java

@@ -9,4 +9,7 @@
  * about possible problems with entries (because that will affect LLM output), but on the other hand the user would
  * like to chat with all available entries in the group, even if some of them are not valid.
  */
-public record Notification(String title, String message) { }
+public record Notification(
+        String title,
+        String message) {
+}

jabgui/src/main/java/org/jabref/gui/autocompleter/PersonNameStringConverter.java

@@ -41,7 +41,8 @@ public PersonNameStringConverter(AutoCompletePreferences preferences) {
     public String toString(Author author) {
         if (autoCompLF) {
             switch (autoCompleteFirstNameMode) {
-                case ONLY_ABBREVIATED, BOTH:
+                case ONLY_ABBREVIATED,
+                     BOTH:
                     return author.getFamilyGiven(true);
                 case ONLY_FULL:
                     return author.getFamilyGiven(false);
@@ -51,7 +52,8 @@ public String toString(Author author) {
         }
         if (autoCompFF) {
             switch (autoCompleteFirstNameMode) {
-                case ONLY_ABBREVIATED, BOTH:
+                case ONLY_ABBREVIATED,
+                     BOTH:
                     return author.getGivenFamily(true);
                 case ONLY_FULL:
                     return author.getGivenFamily(false);

jabgui/src/main/java/org/jabref/gui/autocompleter/SuggestionProvider.java

@@ -1,7 +1,7 @@
 /**
  * Copyright (c) 2014, 2016 ControlsFX
  * All rights reserved.
- *
+ * <p>
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
  * * Redistributions of source code must retain the above copyright
@@ -12,7 +12,7 @@
  * * Neither the name of ControlsFX, any associated website, nor the
  * names of its contributors may be used to endorse or promote products
  * derived from this software without specific prior written permission.
- *
+ * <p>
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS" AND
  * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
  * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE

jabgui/src/main/java/org/jabref/gui/autosaveandbackup/AutosaveManager.java

@@ -44,8 +44,8 @@ private AutosaveManager(BibDatabaseContext bibDatabaseContext, CoarseChangeFilte
         this.executor.scheduleAtFixedRate(
                 () -> {
                     if (needsSave) {
-                       eventBus.post(new AutosaveEvent());
-                       needsSave = false;
+                        eventBus.post(new AutosaveEvent());
+                        needsSave = false;
                     }
                 },
                 DELAY_BETWEEN_AUTOSAVE_ATTEMPTS_IN_SECONDS,

jabgui/src/main/java/org/jabref/gui/autosaveandbackup/BackupManager.java

@@ -137,11 +137,10 @@ public static void shutdown(BibDatabaseContext bibDatabaseContext, Path backupDi
     /**
      * Checks whether a backup file exists for the given database file. If it exists, it is checked whether it is
      * newer and different from the original.
-     *
+     * <p>
      * In case a discarded file is present, the method also returns <code>false</code>, See also {@link #discardBackup(Path)}.
      *
      * @param originalPath Path to the file a backup should be checked for. Example: jabref.bib.
-     *
      * @return <code>true</code> if backup file exists AND differs from originalPath. <code>false</code> is the
      * "default" return value in the good case. In case a discarded file exists, <code>false</code> is returned, too.
      * In the case of an exception <code>true</code> is returned to ensure that the user checks the output.
@@ -303,7 +302,7 @@ private static Path determineDiscardedFile(Path file, Path backupDir) {
 
     /**
      * Marks the backups as discarded.
-     *
+     * <p>
      * We do not delete any files, because the user might want to recover old backup files.
      * Therefore, we mark discarded backups by a --discarded file.
      */
@@ -340,11 +339,11 @@ private void startBackupTask(Path backupDir) {
         fillQueue(backupDir);
 
         executor.scheduleAtFixedRate(
-                                     // We need to determine the backup path on each action, because we use the timestamp in the filename
-                                     () -> determineBackupPathForNewBackup(backupDir).ifPresent(this::performBackup),
-                                     DELAY_BETWEEN_BACKUP_ATTEMPTS_IN_SECONDS,
-                                     DELAY_BETWEEN_BACKUP_ATTEMPTS_IN_SECONDS,
-                                     TimeUnit.SECONDS);
+                // We need to determine the backup path on each action, because we use the timestamp in the filename
+                () -> determineBackupPathForNewBackup(backupDir).ifPresent(this::performBackup),
+                DELAY_BETWEEN_BACKUP_ATTEMPTS_IN_SECONDS,
+                DELAY_BETWEEN_BACKUP_ATTEMPTS_IN_SECONDS,
+                TimeUnit.SECONDS);
     }
 
     private void fillQueue(Path backupDir) {
@@ -370,7 +369,7 @@ private void fillQueue(Path backupDir) {
      * Unregisters the BackupManager from the eventBus of {@link BibDatabaseContext}.
      * This method should only be used when closing a database/JabRef in a normal way.
      *
-     * @param backupDir The backup directory
+     * @param backupDir    The backup directory
      * @param createBackup If the backup manager should still perform a backup
      */
     private void shutdown(Path backupDir, boolean createBackup) {

jabgui/src/main/java/org/jabref/gui/auximport/NewSubLibraryAction.java

@@ -9,7 +9,7 @@
 
 /**
  * The action concerned with generate a new (sub-)database from latex AUX file.
- *
+ * <p>
  * A new library is created by {@link org.jabref.gui.importer.NewDatabaseAction}
  */
 public class NewSubLibraryAction extends SimpleCommand {