X-Git-Url: https://gitweb.stoutner.com/?a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fcom%2Fstoutner%2Fprivacybrowser%2FDownloadFile.java;h=1a68f71fd82dd479365d7a91c82b2ba11434c5fa;hb=7d632afdd10cad5b4d62fce37707eaceebe260cb;hp=a35091fbab74e845bc894db237fa45e4410d923f;hpb=e8bcccda781e0aa65ee4cc11428f3e99dc400015;p=PrivacyBrowserAndroid.git diff --git a/app/src/main/java/com/stoutner/privacybrowser/DownloadFile.java b/app/src/main/java/com/stoutner/privacybrowser/DownloadFile.java index a35091fb..1a68f71f 100644 --- a/app/src/main/java/com/stoutner/privacybrowser/DownloadFile.java +++ b/app/src/main/java/com/stoutner/privacybrowser/DownloadFile.java @@ -19,14 +19,17 @@ package com.stoutner.privacybrowser; -import android.app.Activity; +import android.annotation.SuppressLint; import android.app.Dialog; -import android.app.DialogFragment; +import android.content.Context; import android.content.DialogInterface; import android.net.Uri; import android.os.Bundle; +import android.support.annotation.NonNull; // `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; +// We have to use `AppCompatDialogFragment` instead of `DialogFragment` or an error is produced on API <=22. +import android.support.v7.app.AppCompatDialogFragment; import android.view.KeyEvent; import android.view.LayoutInflater; import android.view.View; @@ -36,7 +39,7 @@ import android.widget.TextView; import java.util.Locale; -public class DownloadFile extends DialogFragment { +public class DownloadFile extends AppCompatDialogFragment { private String downloadUrl; private String downloadFileName; @@ -47,20 +50,23 @@ public class DownloadFile extends DialogFragment { Bundle argumentsBundle = new Bundle(); String fileNameString; - if (contentDisposition.isEmpty()) { // If `contentDisposition` is empty, use the last path segment of the URL as the file name. + + // Parse `filename` from `contentDisposition`. + if (contentDisposition.contains("filename=\"")) { // The file name is contained in a string surrounded by `""`. + fileNameString = contentDisposition.substring(contentDisposition.indexOf("filename=\"") + 10, contentDisposition.indexOf("\"", contentDisposition.indexOf("filename=\"") + 10)); + } else if (contentDisposition.contains("filename=") && ((contentDisposition.indexOf(";", contentDisposition.indexOf("filename=") + 9)) > 0 )) { // The file name is contained in a string beginning with `filename=` and ending with `;`. + fileNameString = contentDisposition.substring(contentDisposition.indexOf("filename=") + 9, contentDisposition.indexOf(";", contentDisposition.indexOf("filename=") + 9)); + } else if (contentDisposition.contains("filename=")) { // The file name is contained in a string beginning with `filename=` and proceeding to the end of `contentDisposition`. + fileNameString = contentDisposition.substring(contentDisposition.indexOf("filename=") + 9, contentDisposition.length()); + } else { // `contentDisposition` does not contain the filename, so use the last path segment of the URL. 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); } - // Convert `contentLength` to MB and store it in `fileSizeString`. `%.3g` displays the three most significant digits. - String fileSizeString = String.format(Locale.getDefault(), "%.3g", (float) contentLength / 1048576) + " MB"; - // Store the variables in the `Bundle`. argumentsBundle.putString("URL", urlString); argumentsBundle.putString("File_Name", fileNameString); - argumentsBundle.putString("File_Size", fileSizeString); + argumentsBundle.putLong("File_Size", contentLength); // Add `argumentsBundle` to this instance of `DownloadFile`. DownloadFile thisDownloadFileDialog = new DownloadFile(); @@ -75,43 +81,60 @@ public class DownloadFile extends DialogFragment { // Store the strings in the local class variables. downloadUrl = getArguments().getString("URL"); downloadFileName = getArguments().getString("File_Name"); - fileSize = getArguments().getString("File_Size"); + + // Get the `File_Size`. + long fileSizeLong = getArguments().getLong("File_Size"); + + // Convert `fileSizeLong` to a String. + if (fileSizeLong == -1) { // We don't know the file size. + fileSize = getString(R.string.unknown_size); + } else { // Convert `fileSize` to MB and store it in `fileSizeString`. `%.3g` displays the three most significant digits. + fileSize = String.format(Locale.getDefault(), "%.3g", (float) fileSizeLong / 1048576) + " MB"; + } } // The public interface is used to send information back to the parent activity. public interface DownloadFileListener { - void onDownloadFile(DialogFragment dialogFragment, String downloadUrl); + void onDownloadFile(AppCompatDialogFragment dialogFragment, String downloadUrl); } // `downloadFileListener` is used in `onAttach()` and `onCreateDialog()`. private DownloadFileListener downloadFileListener; - // Check to make sure tha the parent activity implements the listener. - public void onAttach(Activity parentActivity) { - super.onAttach(parentActivity); + @Override + public void onAttach(Context context) { + super.onAttach(context); + + // Check to make sure the parent activity implements the listener. try { - downloadFileListener = (DownloadFileListener) parentActivity; + downloadFileListener = (DownloadFileListener) context; } catch (ClassCastException exception) { - throw new ClassCastException(parentActivity.toString() + " must implement DownloadFileListener."); + throw new ClassCastException(context.toString() + " must implement DownloadFileListener."); } } @Override + @NonNull + // `@SuppressLing("InflateParams")` removes the warning about using `null` as the parent view group when inflating the `AlertDialog`. + @SuppressLint("InflateParams") public Dialog onCreateDialog(Bundle savedInstanceState) { // Get the activity's layout inflater. LayoutInflater layoutInflater = getActivity().getLayoutInflater(); // 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); + + // Set the title. dialogBuilder.setTitle(R.string.save_as); - // The parent view is `null` because it will be assigned by `AlertDialog`. + + // Set the view. The parent view is `null` because it will be assigned by `AlertDialog`. dialogBuilder.setView(layoutInflater.inflate(R.layout.download_file_dialog, null)); // Set an `onClick()` listener on the negative button. dialogBuilder.setNegativeButton(R.string.cancel, new DialogInterface.OnClickListener() { @Override public void onClick(DialogInterface dialog, int which) { - // Do nothing if `Cancel` is clicked. + // Do nothing if `Cancel` is clicked. The `Dialog` will automatically close. } }); @@ -124,10 +147,12 @@ public class DownloadFile extends DialogFragment { } }); - // Create an `AlertDialog` from the `AlertDialog.Builder`. final AlertDialog alertDialog = dialogBuilder.create(); + // Remove the warning below that `setSoftInputMode` might produce `java.lang.NullPointerException`. + assert alertDialog.getWindow() != null; + // Show the keyboard when `alertDialog` is displayed on the screen. alertDialog.getWindow().setSoftInputMode(WindowManager.LayoutParams.SOFT_INPUT_STATE_ALWAYS_VISIBLE); @@ -136,12 +161,12 @@ public class DownloadFile extends DialogFragment { // Set the text for `downloadFileSizeTextView`. TextView downloadFileSizeTextView = (TextView) alertDialog.findViewById(R.id.download_file_size); - assert downloadFileSizeTextView != null; // Remove the warning of the following line that `downloadFileSizeTextView` might be null. + assert downloadFileSizeTextView != null; // Remove the warning on the following line that `downloadFileSizeTextView` might be `null`. downloadFileSizeTextView.setText(fileSize); // Set the text for `downloadFileNameTextView`. EditText downloadFileNameTextView = (EditText) alertDialog.findViewById(R.id.download_file_name); - assert downloadFileNameTextView != null; // Remove the warning on the following line that `downloadFileNameTextView` might be null. + assert downloadFileNameTextView != null; // Remove the warning on the following line that `downloadFileNameTextView` might be `null`. downloadFileNameTextView.setText(downloadFileName); // Allow the `enter` key on the keyboard to save the file from `downloadFileNameTextView`. @@ -162,7 +187,6 @@ public class DownloadFile extends DialogFragment { } }); - // `onCreateDialog` requires the return of an `AlertDialog`. return alertDialog; }