Use the last path segment in the URLs as the default download file name if `Content...
authorSoren Stoutner <soren@stoutner.com>
Thu, 15 Sep 2016 06:55:33 +0000 (23:55 -0700)
committerSoren Stoutner <soren@stoutner.com>
Thu, 15 Sep 2016 06:55:33 +0000 (23:55 -0700)
app/src/main/assets/en/guide_tor.html
app/src/main/java/com/stoutner/privacybrowser/DownloadFile.java
app/src/main/java/com/stoutner/privacybrowser/MainWebViewActivity.java
app/src/main/res/values/strings.xml

index 1ce0d59dfd4b813aceb477f327da483373f10626..bc56b69ca701d9054da82d76023e3fe299ff67ff 100644 (file)
@@ -70,7 +70,7 @@
     make a purchase, every address that items are shipped to, and the GPS metadata of every picture that is
     uploaded to the internet.  They build a profile of a user's age, gender, marital status, address, political affiliations,
     religious affiliations, family circumstance, number of pets, and everything else they can get their hands on.
-    They even buy up databases of credit card usage at stores, so they can track the off-line purchasing patterns of the users
+    They even buy up databases of credit card usage at local stores, so they can track the off-line purchasing patterns of the users
     in their profiles. Because they already have much more accurate address information about a user than an IP address discloses,
     Tor provides no real privacy protection against mega corporations.</p>
 
index 4bdc879abb113e08064898a44e442799129925ad..a35091fbab74e845bc894db237fa45e4410d923f 100644 (file)
@@ -23,6 +23,7 @@ import android.app.Activity;
 import android.app.Dialog;
 import android.app.DialogFragment;
 import android.content.DialogInterface;
+import android.net.Uri;
 import android.os.Bundle;
 // `android.support.v7.app.AlertDialog` uses more of the horizontal screen real estate versus `android.app.AlertDialog's` smaller width.
 import android.support.v7.app.AlertDialog;
@@ -45,12 +46,11 @@ public class DownloadFile extends DialogFragment {
         // Create `argumentsBundle`.
         Bundle argumentsBundle = new Bundle();
 
-        // If `contentDisposition` is empty, use Android's standard string of `downloadfile.bin`.
         String fileNameString;
-        if (contentDisposition.isEmpty()) {
-            fileNameString = "downloadfile.bin";
-        } else {
-            // Extract `fileNameString` from `contentDisposition` using the substring beginning after `filename="` and ending one character before the end of `contentDisposition`.
+        if (contentDisposition.isEmpty()) {  // If `contentDisposition` is empty, use the last path segment of the URL as the file name.
+            Uri downloadUri = Uri.parse(urlString);
+            fileNameString = downloadUri.getLastPathSegment();
+        } else {  // Extract `fileNameString` from `contentDisposition` using the substring beginning after `filename="` and ending one character before the end of `contentDisposition`.
             fileNameString = contentDisposition.substring(contentDisposition.indexOf("filename=\"") + 10, contentDisposition.length() - 1);
         }
 
@@ -103,7 +103,7 @@ public class DownloadFile extends DialogFragment {
 
         // Use `AlertDialog.Builder` to create the `AlertDialog`.  `R.style.lightAlertDialog` formats the color of the button text.
         AlertDialog.Builder dialogBuilder = new AlertDialog.Builder(getActivity(), R.style.LightAlertDialog);
-        dialogBuilder.setTitle(R.string.file_download);
+        dialogBuilder.setTitle(R.string.save_as);
         // The parent view is `null` because it will be assigned by `AlertDialog`.
         dialogBuilder.setView(layoutInflater.inflate(R.layout.download_file_dialog, null));
 
index 54f6d8ff1aa3cd2025f7dd3cf6e9903dac10c9c5..87f30afe125afe43b6fc8e7f818223050fd730c6 100644 (file)
@@ -361,7 +361,7 @@ public class MainWebViewActivity extends AppCompatActivity implements Navigation
             public void onDownloadStart(String url, String userAgent, String contentDisposition, String mimetype, long contentLength) {
                 // Show the `DownloadFile` `AlertDialog` and name this instance `@string/download`.
                 DialogFragment downloadFileDialogFragment = DownloadFile.fromUrl(url, contentDisposition, contentLength);
-                downloadFileDialogFragment.show(getFragmentManager(), getResources().getString(R.string.file_download));
+                downloadFileDialogFragment.show(getFragmentManager(), getResources().getString(R.string.download));
             }
         });
 
index 9b6a969d939b1d85ec15347dcb11d4bebde9d319..51886aada597f7d9dd3bb10be1991034548f6cb8 100644 (file)
@@ -45,7 +45,7 @@
     <string name="dom_storage_deleted">DOM Storage deleted</string>
     <string name="open_navigation">Open Navigation Drawer</string>
     <string name="close_navigation">Close Navigation Drawer</string>
-    <string name="file_download">File Download</string>
+    <string name="save_as">Save as</string>
     <string name="file_name">File name</string>
     <string name="download">Download</string>