cleaning up some potential bugs

RipMeApp · Apr 25, 2014 · 554026b · 554026b
1 parent 746dd3f
commit 554026b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/src/main/java/com/rarchives/ripme/ripper/rippers/ImagefapRipper.java b/src/main/java/com/rarchives/ripme/ripper/rippers/ImagefapRipper.java
@@ -104,10 +104,7 @@ public void rip() throws IOException {
     }
 
     public boolean canRip(URL url) {
-        if (!url.getHost().endsWith(DOMAIN)) {
-            return false;
-        }
-        return true;
+        return url.getHost().endsWith(DOMAIN);
     }
 
 }
diff --git a/src/main/java/com/rarchives/ripme/utils/Utils.java b/src/main/java/com/rarchives/ripme/utils/Utils.java
@@ -245,9 +245,13 @@ public static String shortenPath(File file) {
     }
 
     public static String filesystemSafe(String text) {
-        return text.replaceAll("[^a-zA-Z0-9.-]", "_")
+        text = text.replaceAll("[^a-zA-Z0-9.-]", "_")
                    .replaceAll("__", "_")
                    .replaceAll("_+$", "");
+        if (text.length() > 255) {
+            text = text.substring(0, 254);
+        }
+        return text;
     }
 
     public static String bytesToHumanReadable(int bytes) {