Improve SpringerLink fetcher

ShikunXiong · Mar 15, 2020 · 3eeb86c · 3eeb86c
1 parent 15c7981
commit 3eeb86c
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 18 deletions.
diff --git a/src/main/java/org/jabref/logic/importer/fetcher/SpringerLink.java b/src/main/java/org/jabref/logic/importer/fetcher/SpringerLink.java
@@ -33,32 +33,32 @@ public class SpringerLink implements FulltextFetcher {
     @Override
     public Optional<URL> findFullText(BibEntry entry) throws IOException {
         Objects.requireNonNull(entry);
-        Optional<URL> pdfLink = Optional.empty();
 
         // Try unique DOI first
         Optional<DOI> doi = entry.getField(StandardField.DOI).flatMap(DOI::parse);
 
-        if (doi.isPresent()) {
-            // Available in catalog?
-            try {
-                HttpResponse<JsonNode> jsonResponse = Unirest.get(API_URL)
-                                                             .queryString("api_key", API_KEY)
-                                                             .queryString("q", String.format("doi:%s", doi.get().getDOI()))
-                                                             .asJson();
-                if (jsonResponse.getBody() != null) {
-                    JSONObject json = jsonResponse.getBody().getObject();
-                    int results = json.getJSONArray("result").getJSONObject(0).getInt("total");
+        if (!doi.isPresent()) {
+            return Optional.empty();
+        }
+        // Available in catalog?
+        try {
+            HttpResponse<JsonNode> jsonResponse = Unirest.get(API_URL)
+                                                         .queryString("api_key", API_KEY)
+                                                         .queryString("q", String.format("doi:%s", doi.get().getDOI()))
+                                                         .asJson();
+            if (jsonResponse.getBody() != null) {
+                JSONObject json = jsonResponse.getBody().getObject();
+                int results = json.getJSONArray("result").getJSONObject(0).getInt("total");
 
-                    if (results > 0) {
-                        LOGGER.info("Fulltext PDF found @ Springer.");
-                        pdfLink = Optional.of(new URL("http", CONTENT_HOST, String.format("/content/pdf/%s.pdf", doi.get().getDOI())));
-                    }
+                if (results > 0) {
+                    LOGGER.info("Fulltext PDF found @ Springer.");
+                    return Optional.of(new URL("http", CONTENT_HOST, String.format("/content/pdf/%s.pdf", doi.get().getDOI())));
                 }
-            } catch (UnirestException e) {
-                LOGGER.warn("SpringerLink API request failed", e);
             }
+        } catch (UnirestException e) {
+            LOGGER.warn("SpringerLink API request failed", e);
         }
-        return pdfLink;
+        return Optional.empty();
     }
 
     @Override

diff --git a/src/test/java/org/jabref/logic/importer/fetcher/SpringerLinkTest.java b/src/test/java/org/jabref/logic/importer/fetcher/SpringerLinkTest.java
@@ -53,4 +53,14 @@ public void notFoundByDOI() throws IOException {
 
         assertEquals(Optional.empty(), finder.findFullText(entry));
     }
+
+    @Test
+    void entityWithoutDoi() throws IOException {
+        assertEquals(Optional.empty(), finder.findFullText(entry));
+    }
+
+    @Test
+    void trustLevel() {
+        assertEquals(TrustLevel.PUBLISHER, finder.getTrustLevel());
+    }
 }