Merge branch 'master' of github.com:JabRef/jabref into fix-for-issue-6224

Author: catarinagomespt

.mailmap

@@ -193,3 +193,4 @@ Christoph Treude <[email protected]>
 Goutam Lavudiya <[email protected]>
 Shikun Xiong <[email protected]>
 Fabiani Giovanni <[email protected]>
+Dimitra Karadima <[email protected]>

AUTHORS

@@ -41,6 +41,7 @@ Ayachi Nene
 Bartosz J. Kaczkowski
 Bartłomiej Dach
 Behrouz Javanmardi
+Benedikt Tutzer
 Benjamin Köhler
 Berk Gureken
 Bernd Kalbfuss
@@ -88,6 +89,7 @@ Dennis Tschechlov
 DevSiroukane
 Dilan Coss
 dima
+Dimitra Karadima
 Domenico Cufalo
 Dominik Schrempf
 Dominik Traczyk
@@ -135,6 +137,7 @@ Guillaume Gardey
 Hakan Duran
 Hannes Restel
 Harinda Samarasekara
+Hussain Arif
 Igor Chernyavsky
 Igor Steinmacher
 Illes Solt
@@ -213,6 +216,7 @@ Martin Stolle
 Martina Catizone
 Mathias Walter
 Matthias Geiger
+Matthias Mayr
 Matthias Mueller
 Mattia Bunel
 Mattias Ulbrich

CHANGELOG.md

@@ -21,6 +21,7 @@ Note that this project **does not** adhere to [Semantic Versioning](http://semve
 - We implemented an option to download fulltext files while importing. [#6381](https://github.com/JabRef/jabref/pull/6381)
 - We added a progress-indicator showing the average progress of background tasks to the toolbar. Clicking it reveals a pop-over with a list of running background tasks. [6443](https://github.com/JabRef/jabref/pull/6443)
 - We fixed the bug when strike the delete key in the text field. [#6421](https://github.com/JabRef/jabref/issues/6421)
+- We added a BibTex key modifier for truncating strings. [#3915](https://github.com/JabRef/jabref/issues/3915)
 - We added support for jumping to target entry when typing letter/digit after sorting a column in maintable [#6146](https://github.com/JabRef/jabref/issues/6146)
 
 ### Changed
@@ -57,6 +58,8 @@ Note that this project **does not** adhere to [Semantic Versioning](http://semve
 - We fixed an issue where JabRef switched to discrete graphics under macOS [#5935](https://github.com/JabRef/jabref/issues/5935)
 - We fixed an issue where the Preferences entry preview will be unexpected modified leads to Value too long exception [#6198](https://github.com/JabRef/jabref/issues/6198)
 - We fixed an issue where custom jstyles for Open/LibreOffice would only be valid if a layout line for the entry type `default` was at the end of the layout section [#6303](https://github.com/JabRef/jabref/issues/6303)
+- We fixed an issue where long directory names created from patterns could create an exception. [#3915](https://github.com/JabRef/jabref/issues/3915)
+
 - We fixed an issue where sort on numeric cases was broken. [#6349](https://github.com/JabRef/jabref/issues/6349)
 - We fixed an issue where year and month fields were not cleared when converting to biblatex [#6224](https://github.com/JabRef/jabref/issues/6224)
 - We fixed an issue where an "Not on FX thread" exception occured when saving on linux [#6453](https://github.com/JabRef/jabref/issues/6453)

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

docs/images/intellij-checkstyle-settings.png

@@ -59,12 +59,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/Globals.java

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

src/main/java/org/jabref/JabRefException.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/JabRefExecutorService.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/JabRefMain.java

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

src/main/java/org/jabref/architecture/ApacheCommonsLang3Allowed.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/ArgumentProcessor.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/CrossrefFetcherEvaluator.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}");
-
     }
-
 }