From bb1d5fc16967d81aea549bb2a01051f7430cda36 Mon Sep 17 00:00:00 2001
From: ByteHamster <info@bytehamster.com>
Date: Fri, 22 Jan 2021 11:59:32 +0100
Subject: [PATCH] Use authentication dialog instead of dedicated activity

---
 app/src/main/AndroidManifest.xml              |  1 +
 .../DownloadAuthenticationActivity.java       | 96 +++++++------------
 .../config/DownloadServiceCallbacksImpl.java  |  1 +
 .../dialog/AuthenticationDialog.java          |  1 +
 .../download_authentication_activity.xml      | 64 -------------
 5 files changed, 38 insertions(+), 125 deletions(-)
 delete mode 100644 app/src/main/res/layout/download_authentication_activity.xml

diff --git a/app/src/main/AndroidManifest.xml b/app/src/main/AndroidManifest.xml
index b4a2c52a3..2d22c7815 100644
--- a/app/src/main/AndroidManifest.xml
+++ b/app/src/main/AndroidManifest.xml
@@ -88,6 +88,7 @@
 
         <activity
             android:name=".activity.DownloadAuthenticationActivity"
+            android:theme="@style/Theme.AntennaPod.Dark.Translucent"
             android:launchMode="singleInstance"/>
 
         <activity
diff --git a/app/src/main/java/de/danoeh/antennapod/activity/DownloadAuthenticationActivity.java b/app/src/main/java/de/danoeh/antennapod/activity/DownloadAuthenticationActivity.java
index 4900f0e27..0f1d38db6 100644
--- a/app/src/main/java/de/danoeh/antennapod/activity/DownloadAuthenticationActivity.java
+++ b/app/src/main/java/de/danoeh/antennapod/activity/DownloadAuthenticationActivity.java
@@ -1,11 +1,7 @@
 package de.danoeh.antennapod.activity;
 
-import android.app.Activity;
 import android.os.Bundle;
 import android.text.TextUtils;
-import android.widget.EditText;
-import android.widget.TextView;
-import androidx.annotation.NonNull;
 import androidx.appcompat.app.AppCompatActivity;
 import de.danoeh.antennapod.R;
 import de.danoeh.antennapod.core.feed.FeedMedia;
@@ -15,6 +11,7 @@ import de.danoeh.antennapod.core.service.download.DownloadRequest;
 import de.danoeh.antennapod.core.storage.DBReader;
 import de.danoeh.antennapod.core.storage.DBWriter;
 import de.danoeh.antennapod.core.storage.DownloadRequester;
+import de.danoeh.antennapod.dialog.AuthenticationDialog;
 import io.reactivex.Completable;
 import io.reactivex.android.schedulers.AndroidSchedulers;
 import io.reactivex.schedulers.Schedulers;
@@ -32,71 +29,48 @@ public class DownloadAuthenticationActivity extends AppCompatActivity {
      */
     public static final String ARG_DOWNLOAD_REQUEST = "request";
 
-    private EditText etxtUsername;
-    private EditText etxtPassword;
-    private DownloadRequest request;
-
     @Override
     protected void onCreate(Bundle savedInstanceState) {
-        setTheme(UserPreferences.getNoTitleTheme());
+        setTheme(UserPreferences.getTranslucentTheme());
         super.onCreate(savedInstanceState);
-        setContentView(R.layout.download_authentication_activity);
-
-        etxtUsername = findViewById(R.id.etxtUsername);
-        etxtPassword = findViewById(R.id.etxtPassword);
 
         Validate.isTrue(getIntent().hasExtra(ARG_DOWNLOAD_REQUEST), "Download request missing");
-        request = getIntent().getParcelableExtra(ARG_DOWNLOAD_REQUEST);
+        DownloadRequest request = getIntent().getParcelableExtra(ARG_DOWNLOAD_REQUEST);
 
-        TextView txtvDescription = findViewById(R.id.txtvDescription);
-        String newDescription = txtvDescription.getText() + ":\n\n" + request.getTitle();
-        txtvDescription.setText(newDescription);
+        new AuthenticationDialog(this, R.string.authentication_label, true, "", "") {
+            @Override
+            protected void onConfirmed(String username, String password) {
+                Completable.fromAction(
+                        () -> {
+                            request.setUsername(username);
+                            request.setPassword(password);
 
-        if (savedInstanceState != null) {
-            etxtUsername.setText(savedInstanceState.getString("username"));
-            etxtPassword.setText(savedInstanceState.getString("password"));
-        }
-
-        findViewById(R.id.butConfirm).setOnClickListener(v ->
-                Completable.fromAction(this::updatePassword)
-                .subscribeOn(Schedulers.io())
-                .observeOn(AndroidSchedulers.mainThread())
-                .subscribe(() -> {
-                    DownloadRequester.getInstance().download(DownloadAuthenticationActivity.this, request);
-                    finish();
-                }));
-
-        findViewById(R.id.butCancel).setOnClickListener(v -> {
-            setResult(Activity.RESULT_CANCELED);
-            finish();
-        });
-
-    }
-
-    private void updatePassword() {
-        String username = etxtUsername.getText().toString();
-        String password = etxtPassword.getText().toString();
-        request.setUsername(username);
-        request.setPassword(password);
-
-        if (request.getFeedfileType() == FeedMedia.FEEDFILETYPE_FEEDMEDIA) {
-            long mediaId = request.getFeedfileId();
-            FeedMedia media = DBReader.getFeedMedia(mediaId);
-            if (media != null) {
-                FeedPreferences preferences = media.getItem().getFeed().getPreferences();
-                if (TextUtils.isEmpty(preferences.getPassword()) || TextUtils.isEmpty(preferences.getUsername())) {
-                    preferences.setUsername(username);
-                    preferences.setPassword(password);
-                    DBWriter.setFeedPreferences(preferences);
-                }
+                            if (request.getFeedfileType() == FeedMedia.FEEDFILETYPE_FEEDMEDIA) {
+                                long mediaId = request.getFeedfileId();
+                                FeedMedia media = DBReader.getFeedMedia(mediaId);
+                                if (media != null) {
+                                    FeedPreferences preferences = media.getItem().getFeed().getPreferences();
+                                    if (TextUtils.isEmpty(preferences.getPassword())
+                                            || TextUtils.isEmpty(preferences.getUsername())) {
+                                        preferences.setUsername(username);
+                                        preferences.setPassword(password);
+                                        DBWriter.setFeedPreferences(preferences);
+                                    }
+                                }
+                            }
+                        })
+                        .subscribeOn(Schedulers.io())
+                        .observeOn(AndroidSchedulers.mainThread())
+                        .subscribe(() -> {
+                            DownloadRequester.getInstance().download(DownloadAuthenticationActivity.this, request);
+                            finish();
+                        });
             }
-        }
-    }
 
-    @Override
-    protected void onSaveInstanceState(@NonNull Bundle outState) {
-        super.onSaveInstanceState(outState);
-        outState.putString("username", etxtUsername.getText().toString());
-        outState.putString("password", etxtPassword.getText().toString());
+            @Override
+            protected void onCancelled() {
+                finish();
+            }
+        }.show();
     }
 }
diff --git a/app/src/main/java/de/danoeh/antennapod/config/DownloadServiceCallbacksImpl.java b/app/src/main/java/de/danoeh/antennapod/config/DownloadServiceCallbacksImpl.java
index f509f6dd4..f782308d1 100644
--- a/app/src/main/java/de/danoeh/antennapod/config/DownloadServiceCallbacksImpl.java
+++ b/app/src/main/java/de/danoeh/antennapod/config/DownloadServiceCallbacksImpl.java
@@ -30,6 +30,7 @@ public class DownloadServiceCallbacksImpl implements DownloadServiceCallbacks {
     @Override
     public PendingIntent getAuthentificationNotificationContentIntent(Context context, DownloadRequest request) {
         final Intent activityIntent = new Intent(context.getApplicationContext(), DownloadAuthenticationActivity.class);
+        activityIntent.setAction("request" + request.getFeedfileId());
         activityIntent.putExtra(DownloadAuthenticationActivity.ARG_DOWNLOAD_REQUEST, request);
         return PendingIntent.getActivity(context.getApplicationContext(),
                 R.id.pending_intent_download_service_auth, activityIntent, PendingIntent.FLAG_ONE_SHOT);
diff --git a/app/src/main/java/de/danoeh/antennapod/dialog/AuthenticationDialog.java b/app/src/main/java/de/danoeh/antennapod/dialog/AuthenticationDialog.java
index 39d321f18..d89cc41ca 100644
--- a/app/src/main/java/de/danoeh/antennapod/dialog/AuthenticationDialog.java
+++ b/app/src/main/java/de/danoeh/antennapod/dialog/AuthenticationDialog.java
@@ -29,6 +29,7 @@ public abstract class AuthenticationDialog extends AlertDialog.Builder {
             etxtPassword.setText(passwordInitialValue);
         }
         setOnCancelListener(dialog -> onCancelled());
+        setOnDismissListener(dialog -> onCancelled());
         setNegativeButton(R.string.cancel_label, null);
         setPositiveButton(R.string.confirm_label, (dialog, which)
                 -> onConfirmed(etxtUsername.getText().toString(), etxtPassword.getText().toString()));
diff --git a/app/src/main/res/layout/download_authentication_activity.xml b/app/src/main/res/layout/download_authentication_activity.xml
deleted file mode 100644
index e16a8b3a8..000000000
--- a/app/src/main/res/layout/download_authentication_activity.xml
+++ /dev/null
@@ -1,64 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<LinearLayout
-        xmlns:android="http://schemas.android.com/apk/res/android"
-        android:layout_width="match_parent"
-        android:layout_height="match_parent"
-        android:orientation="vertical"
-        android:padding="16dp">
-
-    <TextView
-            android:id="@+id/txtvTitle"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:text="@string/authentication_notification_title"
-            android:textSize="@dimen/text_size_large"
-            android:textColor="?android:attr/textColorPrimary"/>
-
-    <TextView
-            android:id="@+id/txtvDescription"
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:text="@string/authentication_notification_msg"
-            android:textColor="?android:attr/textColorSecondary"/>
-
-    <EditText
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:id="@+id/etxtUsername"
-            android:hint="@string/username_label"
-            android:focusable="true"
-            android:focusableInTouchMode="true"
-            android:cursorVisible="true"/>
-
-    <EditText
-            android:layout_width="match_parent"
-            android:layout_height="wrap_content"
-            android:id="@+id/etxtPassword"
-            android:hint="@string/password_label"
-            android:inputType="textPassword"
-            android:focusable="true"
-            android:focusableInTouchMode="true"
-            android:cursorVisible="true"/>
-
-    <LinearLayout
-            android:layout_width="fill_parent"
-            android:layout_height="48dp"
-            android:orientation="horizontal"
-            android:gravity="end">
-
-        <Button
-                android:id="@+id/butCancel"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:text="@string/cancel_label"
-                style="@style/Widget.MaterialComponents.Button.TextButton"/>
-
-        <Button
-                android:id="@+id/butConfirm"
-                android:layout_width="wrap_content"
-                android:layout_height="wrap_content"
-                android:text="@string/confirm_label"
-                style="@style/Widget.MaterialComponents.Button.TextButton"/>
-    </LinearLayout>
-
-</LinearLayout>
\ No newline at end of file