]> gitweb.stoutner.com Git - PrivacyBrowserAndroid.git/blobdiff - app/src/main/java/com/stoutner/privacybrowser/dialogs/SaveWebpageDialog.java
Respect proxies when getting source and saving URLs. https://redmine.stoutner.com...
[PrivacyBrowserAndroid.git] / app / src / main / java / com / stoutner / privacybrowser / dialogs / SaveWebpageDialog.java
index 1c24e0bedc6ca95f4414c74ae3c92bd92001c205..c9e691f63200740d87a52726adf5a77d5557e349 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright © 2019 Soren Stoutner <soren@stoutner.com>.
+ * Copyright © 2019-2020 Soren Stoutner <soren@stoutner.com>.
  *
  * This file is part of Privacy Browser <https://www.stoutner.com/privacy-browser>.
  *
@@ -29,6 +29,7 @@ import android.content.DialogInterface;
 import android.content.Intent;
 import android.content.SharedPreferences;
 import android.content.pm.PackageManager;
+import android.net.Uri;
 import android.os.Build;
 import android.os.Bundle;
 import android.os.Environment;
@@ -49,11 +50,9 @@ import androidx.preference.PreferenceManager;
 import com.stoutner.privacybrowser.R;
 import com.stoutner.privacybrowser.activities.MainWebViewActivity;
 
-public class SaveWebpageDialog extends DialogFragment {
-    // Define the save type constants.
-    public static final int ARCHIVE = 0;
-    public static final int IMAGE = 1;
+import java.io.File;
 
+public class SaveWebpageDialog extends DialogFragment {
     // Define the save webpage listener.
     private SaveWebpageListener saveWebpageListener;
 
@@ -71,12 +70,13 @@ public class SaveWebpageDialog extends DialogFragment {
         saveWebpageListener = (SaveWebpageListener) context;
     }
 
-    public static SaveWebpageDialog saveWebpage(int saveType) {
+    public static SaveWebpageDialog saveUrl(int saveType, String url) {
         // Create an arguments bundle.
         Bundle argumentsBundle = new Bundle();
 
-        // Store the save type in the bundle.
+        // Store the arguments in the bundle.
         argumentsBundle.putInt("save_type", saveType);
+        argumentsBundle.putString("url", url);
 
         // Create a new instance of the save webpage dialog.
         SaveWebpageDialog saveWebpageDialog = new SaveWebpageDialog();
@@ -88,7 +88,7 @@ public class SaveWebpageDialog extends DialogFragment {
         return saveWebpageDialog;
     }
 
-    // `@SuppressLing("InflateParams")` removes the warning about using null as the parent view group when inflating the alert dialog.
+    // `@SuppressLint("InflateParams")` removes the warning about using null as the parent view group when inflating the alert dialog.
     @SuppressLint("InflateParams")
     @Override
     @NonNull
@@ -99,8 +99,9 @@ public class SaveWebpageDialog extends DialogFragment {
         // Remove the incorrect lint warning that the arguments might be null.
         assert arguments != null;
 
-        // Get the save type.
+        // Get the arguments from the bundle.
         int saveType = arguments.getInt("save_type");
+        String url = arguments.getString("url");
 
         // Get a handle for the activity and the context.
         Activity activity = getActivity();
@@ -127,11 +128,15 @@ public class SaveWebpageDialog extends DialogFragment {
 
             // Set the icon according to the save type.
             switch (saveType) {
-                case ARCHIVE:
+                case StoragePermissionDialog.SAVE:
+                    dialogBuilder.setIcon(R.drawable.copy_enabled_dark);
+                    break;
+
+                case StoragePermissionDialog.SAVE_AS_ARCHIVE:
                     dialogBuilder.setIcon(R.drawable.dom_storage_cleared_dark);
                     break;
 
-                case IMAGE:
+                case StoragePermissionDialog.SAVE_AS_IMAGE:
                     dialogBuilder.setIcon(R.drawable.images_enabled_dark);
                     break;
             }
@@ -141,11 +146,15 @@ public class SaveWebpageDialog extends DialogFragment {
 
             // Set the icon according to the save type.
             switch (saveType) {
-                case ARCHIVE:
+                case StoragePermissionDialog.SAVE:
+                    dialogBuilder.setIcon(R.drawable.copy_enabled_light);
+                    break;
+
+                case StoragePermissionDialog.SAVE_AS_ARCHIVE:
                     dialogBuilder.setIcon(R.drawable.dom_storage_cleared_light);
                     break;
 
-                case IMAGE:
+                case StoragePermissionDialog.SAVE_AS_IMAGE:
                     dialogBuilder.setIcon(R.drawable.images_enabled_light);
                     break;
             }
@@ -153,17 +162,21 @@ public class SaveWebpageDialog extends DialogFragment {
 
         // Set the title according to the type.
         switch (saveType) {
-            case ARCHIVE:
+            case StoragePermissionDialog.SAVE:
+                dialogBuilder.setTitle(R.string.save);
+                break;
+
+            case StoragePermissionDialog.SAVE_AS_ARCHIVE:
                 dialogBuilder.setTitle(R.string.save_archive);
                 break;
 
-            case IMAGE:
+            case StoragePermissionDialog.SAVE_AS_IMAGE:
                 dialogBuilder.setTitle(R.string.save_image);
                 break;
         }
 
         // Set the view.  The parent view is null because it will be assigned by the alert dialog.
-        dialogBuilder.setView(activity.getLayoutInflater().inflate(R.layout.save_dialog, null));
+        dialogBuilder.setView(activity.getLayoutInflater().inflate(R.layout.save_webpage_dialog, null));
 
         // Set the cancel button listener.  Using `null` as the listener closes the dialog without doing anything else.
         dialogBuilder.setNegativeButton(R.string.cancel, null);
@@ -177,7 +190,7 @@ public class SaveWebpageDialog extends DialogFragment {
         // Create an alert dialog from the builder.
         AlertDialog alertDialog = dialogBuilder.create();
 
-        // Remove the incorrect lint warning below that `getWindow()` might be null.
+        // Remove the incorrect lint warning below that the window might be null.
         assert alertDialog.getWindow() != null;
 
         // Disable screenshots if not allowed.
@@ -189,25 +202,99 @@ public class SaveWebpageDialog extends DialogFragment {
         alertDialog.show();
 
         // Get handles for the layout items.
+        EditText urlEditText = alertDialog.findViewById(R.id.url_edittext);
         EditText fileNameEditText = alertDialog.findViewById(R.id.file_name_edittext);
         Button browseButton = alertDialog.findViewById(R.id.browse_button);
+        TextView fileExistsWarningTextView = alertDialog.findViewById(R.id.file_exists_warning_textview);
         TextView storagePermissionTextView = alertDialog.findViewById(R.id.storage_permission_textview);
         Button saveButton = alertDialog.getButton(AlertDialog.BUTTON_POSITIVE);
 
-        // Create a default file name string.
-        String defaultFileName = "";
+        // Update the status of the save button whe the URL changes.
+        urlEditText.addTextChangedListener(new TextWatcher() {
+            @Override
+            public void beforeTextChanged(CharSequence charSequence, int i, int i1, int i2) {
+                // Do nothing.
+            }
+
+            @Override
+            public void onTextChanged(CharSequence charSequence, int i, int i1, int i2) {
+                // Do nothing.
+            }
 
-        // Set the default file name according to the type.
+            @Override
+            public void afterTextChanged(Editable editable) {
+                // Enable the save button if the URL and file name are populated.
+                saveButton.setEnabled(!urlEditText.getText().toString().isEmpty() && !fileNameEditText.getText().toString().isEmpty());
+            }
+        });
+
+        // Update the status of the save button when the file name changes.
+        fileNameEditText.addTextChangedListener(new TextWatcher() {
+            @Override
+            public void beforeTextChanged(CharSequence s, int start, int count, int after) {
+                // Do nothing.
+            }
+
+            @Override
+            public void onTextChanged(CharSequence s, int start, int before, int count) {
+                // Do nothing.
+            }
+
+            @Override
+            public void afterTextChanged(Editable s) {
+                // Get the current file name.
+                String fileNameString = fileNameEditText.getText().toString();
+
+                // Convert the file name string to a file.
+                File file = new File(fileNameString);
+
+                // Check to see if the file exists.
+                if (file.exists()) {
+                    // Show the file exists warning.
+                    fileExistsWarningTextView.setVisibility(View.VISIBLE);
+                } else {
+                    // Hide the file exists warning.
+                    fileExistsWarningTextView.setVisibility(View.GONE);
+                }
+
+                // Enable the save button if the file name is populated.
+                saveButton.setEnabled(!fileNameString.isEmpty() && !urlEditText.getText().toString().isEmpty());
+            }
+        });
+
+        // Create a file name string.
+        String fileName = "";
+
+        // Set the file name according to the type.
         switch (saveType) {
-            case ARCHIVE:
-                defaultFileName = getString(R.string.webpage_mht);
+            case StoragePermissionDialog.SAVE:
+                // Convert the URL to a URI.
+                Uri uri = Uri.parse(url);
+
+                // Get the last path segment.
+                String lastPathSegment = uri.getLastPathSegment();
+
+                // Use a default file name if the last path segment is null.
+                if (lastPathSegment == null) {
+                    lastPathSegment = getString(R.string.file);
+                }
+
+                // Use the last path segment as the file name.
+                fileName = lastPathSegment;
                 break;
 
-            case IMAGE:
-                defaultFileName = getString(R.string.webpage_png);
+            case StoragePermissionDialog.SAVE_AS_ARCHIVE:
+                fileName = getString(R.string.webpage_mht);
+                break;
+
+            case StoragePermissionDialog.SAVE_AS_IMAGE:
+                fileName = getString(R.string.webpage_png);
                 break;
         }
 
+        // Save the file name as the default file name.  This must be final to be used in the lambda below.
+        final String defaultFileName = fileName;
+
         // Create a string for the default file path.
         String defaultFilePath;
 
@@ -215,32 +302,20 @@ public class SaveWebpageDialog extends DialogFragment {
         if (ContextCompat.checkSelfPermission(context, Manifest.permission.WRITE_EXTERNAL_STORAGE) == PackageManager.PERMISSION_GRANTED) {  // The storage permission has been granted.
             // Set the default file path to use the external public directory.
             defaultFilePath = Environment.getExternalStorageDirectory() + "/" + defaultFileName;
+
+            // Hide the storage permission text view.
+            storagePermissionTextView.setVisibility(View.GONE);
         } else {  // The storage permission has not been granted.
             // Set the default file path to use the external private directory.
             defaultFilePath = context.getExternalFilesDir(null) + "/" + defaultFileName;
         }
 
-        // Display the default file path.
+        // Populate the edit texts.
+        urlEditText.setText(url);
         fileNameEditText.setText(defaultFilePath);
 
-        // Update the status of the save button when the file name changes.
-        fileNameEditText.addTextChangedListener(new TextWatcher() {
-            @Override
-            public void beforeTextChanged(CharSequence s, int start, int count, int after) {
-                // Do nothing.
-            }
-
-            @Override
-            public void onTextChanged(CharSequence s, int start, int before, int count) {
-                // Do nothing.
-            }
-
-            @Override
-            public void afterTextChanged(Editable s) {
-                // // Enable the save button if a file name exists.
-                saveButton.setEnabled(!fileNameEditText.getText().toString().isEmpty());
-            }
-        });
+        // Move the cursor to the end of the default file path.
+        fileNameEditText.setSelection(defaultFilePath.length());
 
         // Handle clicks on the browse button.
         browseButton.setOnClickListener((View view) -> {
@@ -251,15 +326,7 @@ public class SaveWebpageDialog extends DialogFragment {
             browseIntent.setType("*/*");
 
             // Set the initial file name according to the type.
-            switch (saveType) {
-                case ARCHIVE:
-                    browseIntent.putExtra(Intent.EXTRA_TITLE, getString(R.string.webpage_mht));
-                    break;
-
-                case IMAGE:
-                    browseIntent.putExtra(Intent.EXTRA_TITLE, getString(R.string.webpage_png));
-                    break;
-            }
+            browseIntent.putExtra(Intent.EXTRA_TITLE, defaultFileName);
 
             // Set the initial directory if the minimum API >= 26.
             if (Build.VERSION.SDK_INT >= 26) {
@@ -273,11 +340,6 @@ public class SaveWebpageDialog extends DialogFragment {
             activity.startActivityForResult(browseIntent, MainWebViewActivity.BROWSE_SAVE_WEBPAGE_REQUEST_CODE);
         });
 
-        // Hide the storage permission text view on API < 23 as permissions on older devices are automatically granted.
-        if (Build.VERSION.SDK_INT < 23) {
-            storagePermissionTextView.setVisibility(View.GONE);
-        }
-
         // Return the alert dialog.
         return alertDialog;
     }