Check integrity edition check implemented

CHANGELOG.md

@@ -18,7 +18,7 @@ We refer to [GitHub issues](https://github.com/JabRef/jabref/issues) by using `#
 - [#1813](https://github.com/JabRef/jabref/issues/1813) Import/Export preferences dialog default directory set to working directory
 - [#1897](https://github.com/JabRef/jabref/issues/1897) Implemented integrity check for `year` field: Last four nonpunctuation characters should be numerals
 - Address in MS-Office 2007 xml format is now imported as `location`
-
+- [#1912](https://github.com/JabRef/jabref/issues/1912) Implemented integrity check for `edition` field: Should have the first letter capitalized (BibTeX), Should contain an integer or a literal (BibLaTeX)
 
 ### Fixed
 - Fixed selecting an entry out of multiple duplicates

src/main/java/net/sf/jabref/logic/integrity/EditionChecker.java

@@ -0,0 +1,60 @@
+package net.sf.jabref.logic.integrity;
+
+import java.util.Collections;
+import java.util.List;
+import java.util.Objects;
+import java.util.Optional;
+import java.util.function.Predicate;
+import java.util.regex.Pattern;
+
+import net.sf.jabref.BibDatabaseContext;
+import net.sf.jabref.logic.integrity.IntegrityCheck.Checker;
+import net.sf.jabref.logic.l10n.Localization;
+import net.sf.jabref.model.entry.BibEntry;
+import net.sf.jabref.model.entry.FieldName;
+
+public class EditionChecker implements Checker {
+
+    private static final Predicate<String> FIRST_LETTER_CAPITALIZED = Pattern.compile("^[A-Z]").asPredicate();
+    private static final Predicate<String> ONLY_NUMERALS_OR_LITERALS = Pattern.compile("^([0-9]+|[^0-9]+)$")
+            .asPredicate();
+
+    private final BibDatabaseContext bibDatabaseContextEdition;
+
+
+    public EditionChecker(BibDatabaseContext bibDatabaseContext) {
+        this.bibDatabaseContextEdition = Objects.requireNonNull(bibDatabaseContext);
+    }
+
+    /**
+     * Checks, if field contains only an integer or a literal (BibLaTeX mode)
+     * Checks, if the first letter is capitalized (BibTeX mode)
+     * BibLaTex:
+     * The edition of a printed publication. This must be an integer, not an ordinal.
+     * It is also possible to give the edition as a literal string, for example "Third, revised and expanded edition".
+     * Official bibtex spec:
+     * The edition of a book-for example, "Second".
+     * This should be an ordinal, and should have the first letter capitalized.
+     */
+    @Override
+    public List<IntegrityMessage> check(BibEntry entry) {
+        Optional<String> value = entry.getField(FieldName.EDITION);
+        if (!value.isPresent()) {
+            return Collections.emptyList();
+        }
+
+        //BibLaTeX
+        if (bibDatabaseContextEdition.isBiblatexMode() && !ONLY_NUMERALS_OR_LITERALS.test(value.get().trim())) {
+            return Collections.singletonList(new IntegrityMessage(
+                    Localization.lang("should contain an integer or a literal"), entry, FieldName.EDITION));
+        }
+
+        //BibTeX
+        if (!bibDatabaseContextEdition.isBiblatexMode() && !FIRST_LETTER_CAPITALIZED.test(value.get().trim())) {
+            return Collections.singletonList(new IntegrityMessage(
+                    Localization.lang("should have the first letter capitalized"), entry, FieldName.EDITION));
+        }
+
+        return Collections.emptyList();
+    }
+}

src/main/java/net/sf/jabref/logic/integrity/IntegrityCheck.java

@@ -66,6 +66,7 @@ private List<IntegrityMessage> checkBibtexEntry(BibEntry entry) {
 
         result.addAll(new BracketChecker(FieldName.TITLE).check(entry));
         result.addAll(new YearChecker().check(entry));
+        result.addAll(new EditionChecker(bibDatabaseContext).check(entry));
         result.addAll(new UrlChecker().check(entry));
         result.addAll(new FileChecker(bibDatabaseContext, fileDirectoryPreferences).check(entry));
         result.addAll(new TypeChecker().check(entry));

src/main/resources/l10n/JabRef_da.properties

@@ -1775,6 +1775,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_de.properties

@@ -2490,3 +2490,6 @@ last_four_nonpunctuation_characters_should_be_numerals=Die letzten vier Nichtint
 Remember_password?=Passwort_merken?
 
 shared=geteilt
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=

src/main/resources/l10n/JabRef_en.properties

@@ -2306,3 +2306,5 @@ Look_up_full_text_documents=Look_up_full_text_documents
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=You_are_about_to_look_up_full_text_documents_for_%0_entries.
 last_four_nonpunctuation_characters_should_be_numerals=last_four_nonpunctuation_characters_should_be_numerals
 shared=shared
+should_contain_an_integer_or_a_literal=should_contain_an_integer_or_a_literal
+should_have_the_first_letter_capitalized=should_have_the_first_letter_capitalized

src/main/resources/l10n/JabRef_es.properties

@@ -1676,6 +1676,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_fa.properties

@@ -2456,6 +2456,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_fr.properties

@@ -1726,3 +1726,5 @@ last_four_nonpunctuation_characters_should_be_numerals=Les_4_derniers_caractère
 Remember_password?=
 
 shared=
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=

src/main/resources/l10n/JabRef_in.properties

@@ -1692,6 +1692,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_it.properties

@@ -1793,6 +1793,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_ja.properties

@@ -2433,6 +2433,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_nl.properties

@@ -2465,6 +2465,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_no.properties

@@ -2857,6 +2857,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_pt_BR.properties

@@ -1689,6 +1689,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_ru.properties

@@ -2434,6 +2434,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_sv.properties

@@ -1637,6 +1637,10 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
 Remember_password?=Kom_ihåg_lösenord?
 
 shared=delad
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=

src/main/resources/l10n/JabRef_tr.properties

@@ -1710,6 +1710,9 @@ Look_up_full_text_documents=Tam_netin_belgeri_ara
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=%0_adet_girdi_için_tam_metin_belgeleri_aramak_üzeresiniz.
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_vi.properties

@@ -2460,6 +2460,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/main/resources/l10n/JabRef_zh.properties

@@ -1701,6 +1701,9 @@ Look_up_full_text_documents=
 You_are_about_to_look_up_full_text_documents_for_%0_entries.=
 
 last_four_nonpunctuation_characters_should_be_numerals=
+
+should_contain_an_integer_or_a_literal=
+should_have_the_first_letter_capitalized=
 Remember_password?=
 
 shared=

src/test/java/net/sf/jabref/logic/integrity/IntegrityCheckTest.java

@@ -60,6 +60,20 @@ public void testYearChecks() {
         assertWrong(createContext("year", "1986a(){},.;!?<>%&$"));
     }
 
+    @Test
+    public void testEditionChecks() {
+        assertCorrect(withMode(createContext("edition", "Second"), BibDatabaseMode.BIBTEX));
+        assertCorrect(withMode(createContext("edition", "Third"), BibDatabaseMode.BIBTEX));
+        assertWrong(withMode(createContext("edition", "second"), BibDatabaseMode.BIBTEX));
+        assertWrong(withMode(createContext("edition", "2"), BibDatabaseMode.BIBTEX));
+        assertWrong(withMode(createContext("edition", "2nd"), BibDatabaseMode.BIBTEX));
+        assertCorrect(withMode(createContext("edition", "2"), BibDatabaseMode.BIBLATEX));
+        assertCorrect(withMode(createContext("edition", "10"), BibDatabaseMode.BIBLATEX));
+        assertCorrect(
+                withMode(createContext("edition", "Third, revised and expanded edition"), BibDatabaseMode.BIBLATEX));
+        assertWrong(withMode(createContext("edition", "2nd"), BibDatabaseMode.BIBLATEX));
+    }
+
     @Test
     public void testBracketChecks() {
         assertCorrect(createContext("title", "x"));