Fixing journal abbreviation roundtrip

CHANGELOG.md

@@ -92,6 +92,7 @@ Note that this project **does not** adhere to [Semantic Versioning](http://semve
 - We fixed an issue where journal abbreviations would not abbreviate journal titles with escaped ampersands (\\&). [#8948](https://github.com/JabRef/jabref/issues/8948)
 - We fixed an issue where font size preferences did not apply correctly to preference dialog window and the menu bar. [#8386](https://github.com/JabRef/jabref/issues/8386) and [#9279](https://github.com/JabRef/jabref/issues/9279)
 - We fixed an issue when using an unsafe character in the citation key, the auto-linking feature fails to link files. [#9267](https://github.com/JabRef/jabref/issues/9267)
+- We fixed an issue where a known journal's medline/dot-less abbreviation does not switch to the full name. [#9370](https://github.com/JabRef/jabref/issues/9370)
 
 ### Removed
 

src/main/java/org/jabref/logic/journals/JournalAbbreviationRepository.java

@@ -8,6 +8,7 @@
 import java.util.Optional;
 import java.util.Set;
 import java.util.regex.Matcher;
+import java.util.regex.Pattern;
 import java.util.stream.Collectors;
 
 import org.h2.mvstore.MVMap;
@@ -56,7 +57,8 @@ public boolean isKnownName(String journalName) {
             return true;
         }
 
-        return fullToAbbreviation.containsKey(journal) || abbreviationToFull.containsKey(journal);
+        return fullToAbbreviation.containsKey(journal) || abbreviationToFull.containsKey(journal)
+                || findDottedAbbrFromDotless(journal).length() > 0;
     }
 
     /**
@@ -67,7 +69,30 @@ public boolean isAbbreviatedName(String journalName) {
         String journal = journalName.trim();
 
         return customAbbreviations.stream().anyMatch(abbreviation -> isMatchedAbbreviated(journal, abbreviation))
-                || abbreviationToFull.containsKey(journal);
+                || abbreviationToFull.containsKey(journal) || findDottedAbbrFromDotless(journal).length() > 0;
+    }
+
+    public String findDottedAbbrFromDotless(String journalName) {
+        String foundKey = "";
+
+        // check for a dot-less abbreviation
+        if (!Pattern.compile("\\.").matcher(journalName).find()) {
+            // use dot-less abbr to find full name using regex
+            String[] journalSplit = journalName.split(" ");
+
+            for (int i = 0; i < journalSplit.length; i++) {
+                String word = journalSplit[i] + "[\\.\\s]*";
+                journalSplit[i] = word;
+            }
+
+            String joined = String.join("", journalSplit);
+
+            foundKey = abbreviationToFull.keySet().stream()
+                                         .filter(s -> Pattern.compile(joined).matcher(s).find())
+                                         .collect(Collectors.joining());
+        }
+
+        return foundKey;
     }
 
     /**
@@ -87,7 +112,17 @@ public Optional<Abbreviation> get(String input) {
 
         return Optional.ofNullable(fullToAbbreviation.get(journal))
                        .map(abbreviation -> new Abbreviation(journal, abbreviation))
-                       .or(() -> Optional.ofNullable(abbreviationToFull.get(journal)).map(fullName -> new Abbreviation(fullName, journal)));
+                       .or(() -> {
+                           String abbr = "";
+
+                           // check for dot-less abbr
+                           if (isKnownName(journal) && isAbbreviatedName(journal)) {
+                               abbr = findDottedAbbrFromDotless(journal);
+                           }
+
+                           return Optional.ofNullable(abbreviationToFull.get(abbr.equals("") ? journal : abbr))
+                                          .map(fullName -> new Abbreviation(fullName, journal));
+                       });
     }
 
     public void addCustomAbbreviation(Abbreviation abbreviation) {

src/test/java/org/jabref/logic/journals/JournalAbbreviationRepositoryTest.java

@@ -252,4 +252,19 @@ void testUnabbreviateWithJournalExistsAndFJournalExists() {
                 .withField(StandardField.JOURNAL, "ACS Applied Materials & Interfaces");
         assertEquals(expectedAbbreviatedJournalEntry, abbreviatedJournalEntry);
     }
+
+    @Test
+    void testJournalDotlessAbbreviation() {
+        BibDatabase bibDatabase = new BibDatabase();
+        JournalAbbreviationRepository journalAbbreviationRepository = JournalAbbreviationLoader.loadBuiltInRepository();
+        UndoableUnabbreviator undoableUnabbreviator = new UndoableUnabbreviator(journalAbbreviationRepository);
+
+        BibEntry abbreviatedJournalEntry = new BibEntry(StandardEntryType.Article);
+        abbreviatedJournalEntry.setField(StandardField.JOURNAL, "ACS Appl Mater Interfaces");
+
+        undoableUnabbreviator.unabbreviate(bibDatabase, abbreviatedJournalEntry, StandardField.JOURNAL, new CompoundEdit());
+        BibEntry expectedAbbreviatedJournalEntry = new BibEntry(StandardEntryType.Article)
+                .withField(StandardField.JOURNAL, "ACS Applied Materials & Interfaces");
+        assertEquals(expectedAbbreviatedJournalEntry, abbreviatedJournalEntry);
+    }
 }