Enforce LeftCurly rule

config/IntelliJ Code Style.xml

@@ -1,37 +1,38 @@
-<code_scheme name="JabRef">
-  <option name="CLASS_COUNT_TO_USE_IMPORT_ON_DEMAND" value="999" />
-  <option name="NAMES_COUNT_TO_USE_IMPORT_ON_DEMAND" value="999" />
-  <option name="PACKAGES_TO_USE_IMPORT_ON_DEMAND">
-    <value />
-  </option>
-  <option name="IMPORT_LAYOUT_TABLE">
-    <value>
-      <package name="java" withSubpackages="true" static="false" />
-      <emptyLine />
-      <package name="javax" withSubpackages="true" static="false" />
-      <emptyLine />
-      <package name="javafx" withSubpackages="true" static="false" />
-      <emptyLine />
-      <package name="org.jabref" withSubpackages="true" static="false" />
-      <emptyLine />
-      <package name="" withSubpackages="true" static="false" />
-      <emptyLine />
-      <package name="" withSubpackages="true" static="true" />
-    </value>
-  </option>
-  <option name="JD_P_AT_EMPTY_LINES" value="false" />
-  <option name="JD_KEEP_EMPTY_PARAMETER" value="false" />
-  <option name="JD_KEEP_EMPTY_EXCEPTION" value="false" />
-  <option name="JD_KEEP_EMPTY_RETURN" value="false" />
-  <option name="WRAP_COMMENTS" value="true" />
+<code_scheme name="JabRef" version="173">
+  <option name="RIGHT_MARGIN" value="0" />
+  <option name="SOFT_MARGINS" value="120" />
   <JavaCodeStyleSettings>
     <option name="DO_NOT_WRAP_AFTER_SINGLE_ANNOTATION" value="true" />
+    <option name="SPACE_INSIDE_ONE_LINE_ENUM_BRACES" value="true" />
+    <option name="CLASS_COUNT_TO_USE_IMPORT_ON_DEMAND" value="999" />
+    <option name="NAMES_COUNT_TO_USE_IMPORT_ON_DEMAND" value="999" />
+    <option name="PACKAGES_TO_USE_IMPORT_ON_DEMAND">
+      <value />
+    </option>
+    <option name="IMPORT_LAYOUT_TABLE">
+      <value>
+        <package name="java" withSubpackages="true" static="false" />
+        <emptyLine />
+        <package name="javax" withSubpackages="true" static="false" />
+        <emptyLine />
+        <package name="javafx" withSubpackages="true" static="false" />
+        <emptyLine />
+        <package name="org.jabref" withSubpackages="true" static="false" />
+        <emptyLine />
+        <package name="" withSubpackages="true" static="false" />
+        <emptyLine />
+        <package name="" withSubpackages="true" static="true" />
+      </value>
+    </option>
   </JavaCodeStyleSettings>
   <codeStyleSettings language="JAVA">
+    <option name="KEEP_CONTROL_STATEMENT_IN_ONE_LINE" value="false" />
     <option name="KEEP_BLANK_LINES_IN_DECLARATIONS" value="1" />
     <option name="KEEP_BLANK_LINES_IN_CODE" value="1" />
     <option name="KEEP_BLANK_LINES_BEFORE_RBRACE" value="0" />
     <option name="ALIGN_MULTILINE_CHAINED_METHODS" value="true" />
+    <option name="SPACE_BEFORE_ARRAY_INITIALIZER_LBRACE" value="true" />
+    <option name="WRAP_COMMENTS" value="true" />
     <arrangement>
       <rules>
         <section>
@@ -292,4 +293,4 @@
       </rules>
     </arrangement>
   </codeStyleSettings>
-</code_scheme>
+</code_scheme>

config/checkstyle/checkstyle.xml

@@ -26,6 +26,10 @@
     <module name="FileTabCharacter"/>
     <module name="NewlineAtEndOfFile"/>
 
+    <!-- Checks for size violations: https://checkstyle.sourceforge.io/config_sizes.html -->
+
+    <!-- LineLength not in place as PreviewerViewer and RelatedArticlesTab have line length with more than 500 charachters -->
+
     <module name="TreeWalker">
         <!-- Checks for Javadoc comments: https://checkstyle.org/config_javadoc.html -->
         <module name="InvalidJavadocPosition"/>
@@ -35,6 +39,7 @@
         </module>
 
         <!-- Checks for imports: https://checkstyle.org/config_import.html -->
+
         <module name="UnusedImports"/>
         <module name="RedundantImport"/>
         <module name="AvoidStarImport"/>
@@ -85,6 +90,9 @@
         </module>
 
         <!-- Checks for blocks: https://checkstyle.org/config_blocks.html -->
+
+        <module name="AvoidNestedBlocks"/>
+
         <module name="NeedBraces"/>
 
         <module name="EmptyBlock">
@@ -99,16 +107,25 @@
 
               public String getTabName() { return Localization.lang("XMP metadata"); }
 
-            Since it is too much effort to reformat all code, it is currently not enabled -->
-        <!-- <module name="LeftCurly"/> -->
+        -->
+        <module name="LeftCurly"/>
 
         <module name="RightCurly"/>
+
         <!-- coding - https://checkstyle.sourceforge.io/config_coding.html -->
 
         <module name="AvoidDoubleBraceInitialization"/>
 
         <module name="CovariantEquals"/>
 
+        <module name="MultipleVariableDeclarations"/>
+
+        <module name="OneStatementPerLine">
+            <property name="treatTryResourcesAsStatement" value="true"/>
+        </module>
+
+        <module name="UnnecessarySemicolonInTryWithResources"/>
+
         <!-- Checks for common coding problems: https://checkstyle.org/config_coding.html -->
 
         <module name="DeclarationOrder"/>

docs/guidelines-for-setting-up-a-local-workspace.md

@@ -170,9 +170,10 @@ Contributions to JabRef's source code need to have a code formatting that is con
    4. Check "Store relative to project location"
    5. Click "Next" and "Finish"
    6. Activate the CheckStyle configuration file by ticking it in the list
-   7. Ensure that CheckStyle version 8.31 (or higher) is selected
-   8. Save settings by clicking "OK"
-   9. Your configuration should now look like this:  
+   7. Ensure that the latest CheckStyle version is selected (8.32 or higher)
+   8. Set the "Scan Scope" to "Only Java sources (including tests)
+   9. Save settings by clicking "OK"
+   10. Your configuration should now look like this:  
       ![checkstyle settings](images/intellij-checkstyle-settings.png)
 
 #### Troubleshooting when using both IDEA and Eclipse

src/main/java/org/jabref/Globals.java

@@ -54,12 +54,14 @@ public class Globals {
 
     /**
      * This field is initialized upon startup.
+     * <p>
      * Only GUI code is allowed to access it, logic code should use dependency injection.
      */
     public static JournalAbbreviationRepository journalAbbreviationRepository;
 
     /**
      * This field is initialized upon startup.
+     * <p>
      * Only GUI code is allowed to access it, logic code should use dependency injection.
      */
     public static ProtectedTermsLoader protectedTermsLoader;

src/main/java/org/jabref/JabRefException.java

@@ -39,5 +39,4 @@ public String getLocalizedMessage() {
             return localizedMessage;
         }
     }
-
 }

src/main/java/org/jabref/JabRefExecutorService.java

@@ -28,7 +28,6 @@ public class JabRefExecutorService {
         thread.setName("JabRef CachedThreadPool");
         thread.setUncaughtExceptionHandler(new FallbackExceptionHandler());
         return thread;
-
     });
     private final ExecutorService lowPriorityExecutorService = Executors.newCachedThreadPool(r -> {
         Thread thread = new Thread(r);
@@ -165,5 +164,4 @@ public void run() {
             }
         }
     }
-
 }

src/main/java/org/jabref/JabRefMain.java

@@ -114,9 +114,9 @@ private static void ensureCorrectJavaVersion() {
 
         if (java9Fail || versionFail) {
             StringBuilder versionError = new StringBuilder(
-                                                           Localization.lang("Your current Java version (%0) is not supported. Please install version %1 or higher.",
-                                                                             checker.getJavaVersion(),
-                                                                   buildInfo.minRequiredJavaVersion));
+                    Localization.lang("Your current Java version (%0) is not supported. Please install version %1 or higher.",
+                            checker.getJavaVersion(),
+                            buildInfo.minRequiredJavaVersion));
 
             versionError.append("\n");
             versionError.append(Localization.lang("Your Java Runtime Environment is located at %0.", checker.getJavaInstallationDirectory()));
@@ -165,7 +165,7 @@ private static void applyPreferences(JabRefPreferences preferences) {
 
         // Build list of Import and Export formats
         Globals.IMPORT_FORMAT_READER.resetImportFormats(Globals.prefs.getImportFormatPreferences(),
-                                                        Globals.prefs.getXMPPreferences(), Globals.getFileUpdateMonitor());
+                Globals.prefs.getXMPPreferences(), Globals.getFileUpdateMonitor());
         Globals.entryTypesManager.addCustomOrModifiedTypes(preferences.loadBibEntryTypes(BibDatabaseMode.BIBTEX),
                 preferences.loadBibEntryTypes(BibDatabaseMode.BIBLATEX));
         Globals.exportFactory = Globals.prefs.getExporterFactory(Globals.journalAbbreviationRepository);

src/main/java/org/jabref/architecture/ApacheCommonsLang3Allowed.java

@@ -8,5 +8,4 @@
 
     // The rationale
     String value();
-
 }

src/main/java/org/jabref/cli/ArgumentProcessor.java

@@ -266,7 +266,7 @@ private boolean exportMatches(List<ParserResult> loaded) {
                     break;
                 default:
                     System.err.println(Localization.lang("Output file missing").concat(". \n \t ")
-                            .concat(Localization.lang("Usage")).concat(": ") + JabRefCLI.getExportMatchesSyntax());
+                                                   .concat(Localization.lang("Usage")).concat(": ") + JabRefCLI.getExportMatchesSyntax());
                     noGUINeeded = true;
                     return false;
             }
@@ -442,7 +442,7 @@ private void exportFile(List<ParserResult> loaded, String[] data) {
                 try {
                     exporter.get().export(pr.getDatabaseContext(), Path.of(data[0]),
                             pr.getDatabaseContext().getMetaData().getEncoding()
-                                    .orElse(Globals.prefs.getDefaultEncoding()),
+                              .orElse(Globals.prefs.getDefaultEncoding()),
                             pr.getDatabaseContext().getDatabase().getEntries());
                 } catch (Exception ex) {
                     System.err.println(Localization.lang("Could not export file") + " '" + data[0] + "': "
@@ -516,8 +516,7 @@ private void regenerateBibtexKeys(List<ParserResult> loaded) {
     /**
      * Run an entry fetcher from the command line.
      *
-     * @param fetchCommand A string containing both the name of the fetcher to use and
-     *                     the search query, separated by a :
+     * @param fetchCommand A string containing both the name of the fetcher to use and the search query, separated by a :
      * @return A parser result containing the entries fetched or null if an error occurred.
      */
     private Optional<ParserResult> fetch(String fetchCommand) {

src/main/java/org/jabref/cli/CrossrefFetcherEvaluator.java

@@ -71,7 +71,6 @@ public void run() {
                             } catch (FetcherException e) {
                                 e.printStackTrace();
                             }
-
                         } else {
                             try {
                                 Optional<DOI> crossrefDOI = new CrossRef().findIdentifier(entry);
@@ -86,7 +85,6 @@ public void run() {
                         countDownLatch.countDown();
                     }
                 });
-
             }
             countDownLatch.await();
 

src/main/java/org/jabref/cli/GenerateCharacterTable.java

@@ -39,14 +39,12 @@ public static void main(String[] args) {
             System.out
                     .println(
                             character.getKey() + " & "
-                                    + ((character.getKey() > 128) ? String
-                                            .valueOf(Character.toChars(character.getKey())) : "")
+                                    + ((character.getKey() > 128) ?
+                                    String.valueOf(Character.toChars(character.getKey())) : "")
                                     + " & \\symbol{" + Integer.toString(character.getKey()) + "} & "
                                     + character.getValue() + " & \\verb¤" + character.getValue() + "¤ \\\\");
         }
         System.out.println("\\end{supertabular}");
         System.out.println("\\end{document}");
-
     }
-
 }

src/main/java/org/jabref/cli/JabRefCLI.java

@@ -164,73 +164,73 @@ private static Options getOptions() {
 
         options.addOption(Option.builder("i").
                 longOpt("import").
-                desc(String.format("%s: '%s'", Localization.lang("Import file"), "-i library.bib")).
-                hasArg().
-                argName("FILE[,FORMAT]").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Import file"), "-i library.bib")).
+                                        hasArg().
+                                        argName("FILE[,FORMAT]").
+                                        build());
 
         options.addOption(Option.builder().
                 longOpt("importToOpen").
-                desc(Localization.lang("Same as --import, but will be imported to the opened tab")).
-                hasArg().
-                argName("FILE[,FORMAT]").
-                build());
+                                        desc(Localization.lang("Same as --import, but will be imported to the opened tab")).
+                                        hasArg().
+                                        argName("FILE[,FORMAT]").
+                                        build());
 
         options.addOption(Option.builder("ib").
                 longOpt("importBibtex").
-                desc(String.format("%s: '%s'", Localization.lang("Import BibTeX"), "-ib @article{entry}")).
-                hasArg().
-                argName("BIBTEXT_STRING").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Import BibTeX"), "-ib @article{entry}")).
+                                        hasArg().
+                                        argName("BIBTEXT_STRING").
+                                        build());
 
         options.addOption(Option.builder("o").
                 longOpt("output").
-                desc(String.format("%s: '%s'", Localization.lang("Export an input to a file"), "-i db.bib -o db.htm,html")).
-                hasArg().
-                argName("FILE[,FORMAT]").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Export an input to a file"), "-i db.bib -o db.htm,html")).
+                                        hasArg().
+                                        argName("FILE[,FORMAT]").
+                                        build());
 
         options.addOption(Option.builder("m").
                 longOpt("exportMatches").
-                desc(String.format("%s: '%s'", Localization.lang("Matching"), "-i db.bib -m author=Newton,search.htm,html")).
-                hasArg().
-                argName("QUERY,FILE[,FORMAT]").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Matching"), "-i db.bib -m author=Newton,search.htm,html")).
+                                        hasArg().
+                                        argName("QUERY,FILE[,FORMAT]").
+                                        build());
 
         options.addOption(Option.builder("f").
                 longOpt("fetch").
-                desc(String.format("%s: '%s'", Localization.lang("Run fetcher"), "-f Medline/PubMed:cancer")).
-                hasArg().
-                argName("FETCHER:QUERY").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Run fetcher"), "-f Medline/PubMed:cancer")).
+                                        hasArg().
+                                        argName("FETCHER:QUERY").
+                                        build());
 
         options.addOption(Option.builder("a").
                 longOpt("aux").
-                desc(String.format("%s: '%s'", Localization.lang("Sublibrary from AUX to BibTeX"), "-a thesis.aux,new.bib")).
-                hasArg().
-                argName("FILE[.aux],FILE[.bib] FILE").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Sublibrary from AUX to BibTeX"), "-a thesis.aux,new.bib")).
+                                        hasArg().
+                                        argName("FILE[.aux],FILE[.bib] FILE").
+                                        build());
 
         options.addOption(Option.builder("x").
                 longOpt("prexp").
-                desc(String.format("%s: '%s'", Localization.lang("Export preferences to a file"), "-x prefs.xml")).
-                hasArg().
-                argName("[FILE]").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Export preferences to a file"), "-x prefs.xml")).
+                                        hasArg().
+                                        argName("[FILE]").
+                                        build());
 
         options.addOption(Option.builder("p").
                 longOpt("primp").
-                desc(String.format("%s: '%s'", Localization.lang("Import preferences from a file"), "-p prefs.xml")).
-                hasArg().
-                argName("[FILE]").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Import preferences from a file"), "-p prefs.xml")).
+                                        hasArg().
+                                        argName("[FILE]").
+                                        build());
 
         options.addOption(Option.builder("d").
                 longOpt("prdef").
-                desc(String.format("%s: '%s'", Localization.lang("Reset preferences"), "-d mainFontSize,newline' or '-d all")).
-                hasArg().
-                argName("KEY1[,KEY2][,KEYn] | all").
-                build());
+                                        desc(String.format("%s: '%s'", Localization.lang("Reset preferences"), "-d mainFontSize,newline' or '-d all")).
+                                        hasArg().
+                                        argName("KEY1[,KEY2][,KEYn] | all").
+                                        build());
 
         return options;
     }