From 4ffdabf5c99583430f568f98f4ac30e305465b5b Mon Sep 17 00:00:00 2001
From: Siedlerchr <siedlerkiller@gmail.com>
Date: Sat, 4 Nov 2017 12:45:02 +0100
Subject: [PATCH] Move isOnlineLink back to linked file

---
 .../java/org/jabref/model/entry/LinkedFile.java  | 16 +++++++++++++---
 .../java/org/jabref/model/util/FileHelper.java   |  9 ---------
 2 files changed, 13 insertions(+), 12 deletions(-)

diff --git a/src/main/java/org/jabref/model/entry/LinkedFile.java b/src/main/java/org/jabref/model/entry/LinkedFile.java
index 5214576a684..d644aca1a13 100644
--- a/src/main/java/org/jabref/model/entry/LinkedFile.java
+++ b/src/main/java/org/jabref/model/entry/LinkedFile.java
@@ -35,7 +35,7 @@ public LinkedFile(String description, String link, String fileType) {
         this.description.setValue(Objects.requireNonNull(description));
 
         String fileLink = Objects.requireNonNull(link);
-        if (!FileHelper.isOnlineLink(fileLink)) {
+        if (!isOnlineLink(fileLink)) {
             this.link.setValue(fileLink.replace("\\", "/"));
         } else {
             this.link.setValue(fileLink);
@@ -69,7 +69,7 @@ public String getLink() {
     }
 
     public void setLink(String link) {
-        if (!FileHelper.isOnlineLink(link)) {
+        if (!isOnlineLink(link)) {
             this.link.setValue(link.replace("\\", "/"));
         } else {
             this.link.setValue(link);
@@ -117,6 +117,15 @@ private void readObject(ObjectInputStream in) throws IOException {
         description = new SimpleStringProperty(in.readUTF());
     }
 
+    /**
+     * Checks if the given String is an online link
+     * @param toCheck The String to check
+     * @return True if it starts with http://, https:// or contains www; false otherwise
+     */
+    private boolean isOnlineLink(String toCheck) {
+        return toCheck.startsWith("http://") || toCheck.startsWith("https://") || toCheck.contains("www.");
+    }
+
     @Override
     public int hashCode() {
         return Objects.hash(description.get(), link.get(), fileType.get());
@@ -136,7 +145,7 @@ public boolean isEmpty() {
     }
 
     public boolean isOnlineLink() {
-        return FileHelper.isOnlineLink(link.get());
+        return isOnlineLink(link.get());
     }
 
     public Optional<Path> findIn(BibDatabaseContext databaseContext, FileDirectoryPreferences fileDirectoryPreferences) {
@@ -152,4 +161,5 @@ public Optional<Path> findIn(List<Path> directories) {
             return FileHelper.expandFilenameAsPath(link.get(), directories);
         }
     }
+
 }
diff --git a/src/main/java/org/jabref/model/util/FileHelper.java b/src/main/java/org/jabref/model/util/FileHelper.java
index 3b0f3e11421..bd75c54ca95 100644
--- a/src/main/java/org/jabref/model/util/FileHelper.java
+++ b/src/main/java/org/jabref/model/util/FileHelper.java
@@ -14,15 +14,6 @@
 
 public class FileHelper {
 
-    /**
-     * Checks if the given String is an online link
-     * @param toCheck The String to check
-     * @return True if it starts with http://, https:// or contains www; false otherwise
-     */
-    public static boolean isOnlineLink(String toCheck) {
-        return toCheck.startsWith("http://") || toCheck.startsWith("https://") || toCheck.contains("www.");
-    }
-
     /**
      * Returns the extension of a file or Optional.empty() if the file does not have one (no . in name).
      *