bug fix for video containing '/'; changed msg on download

tastelessjolt · Sep 15, 2018 · 6c9ce02 · 6c9ce02
1 parent c80fee0
commit 6c9ce02
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/app/src/main/java/me/harshithgoka/youtubedl/Format.java b/app/src/main/java/me/harshithgoka/youtubedl/Format.java
@@ -91,10 +91,14 @@ public void writeToParcel(Parcel parcel, int flags) {
         parcel.writeString(type);
     }
 
+    public String sanitizeFilename() {
+        return title.replaceAll("/", "_");
+    }
+
     public void download (Context context) {
         String extension = Utils.getExtension(this);
-        String name = title;
-        Log.d("Filename", title + "." + extension);
+        String name = sanitizeFilename();
+        Log.d("Filename", name + "." + extension);
 
         DownloadManager.Request req = new DownloadManager.Request(Uri.parse(url));
         req.setTitle(name)

diff --git a/app/src/main/java/me/harshithgoka/youtubedl/FormatAdapter.java b/app/src/main/java/me/harshithgoka/youtubedl/FormatAdapter.java
@@ -50,8 +50,8 @@ public void onClick(View view) {
             assert clipboard != null;
             ClipData clip = ClipData.newRawUri("DownloadURL", Uri.parse(finalurl));
             clipboard.setPrimaryClip(clip);
-
-            Toast.makeText(context, String.format("(%s) Quality link copied to Clipboard", ((TextView) view.findViewById(R.id.format_quality)).getText().toString()), Toast.LENGTH_SHORT).show();
+            Toast.makeText(context, String.format("Your video \"%s\" is now downloading. Check the notification area.", formats.get(getLayoutPosition()).title), Toast.LENGTH_SHORT).show();
+//            Toast.makeText(context, String.format("(%s) Quality link copied to Clipboard", ((TextView) view.findViewById(R.id.format_quality)).getText().toString()), Toast.LENGTH_SHORT).show();
 
             formats.get(getLayoutPosition()).download(context);
         }