Merge pull request #4855 from ByteHamster/simplify-password

Remember entered password

2 files changed, 46 insertions, 40 deletions

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 912038e4c..4900f0e27 100644
--- a/app/src/main/java/de/danoeh/antennapod/activity/DownloadAuthenticationActivity.java
+++ b/app/src/main/java/de/danoeh/antennapod/activity/DownloadAuthenticationActivity.java
@@ -1,61 +1,54 @@
 package de.danoeh.antennapod.activity;
 
 import android.app.Activity;
-import android.content.Intent;
 import android.os.Bundle;
-
-import androidx.annotation.NonNull;
-import androidx.appcompat.app.AppCompatActivity;
-import android.widget.Button;
+import android.text.TextUtils;
 import android.widget.EditText;
 import android.widget.TextView;
-
-import org.apache.commons.lang3.Validate;
-
+import androidx.annotation.NonNull;
+import androidx.appcompat.app.AppCompatActivity;
 import de.danoeh.antennapod.R;
+import de.danoeh.antennapod.core.feed.FeedMedia;
+import de.danoeh.antennapod.core.feed.FeedPreferences;
 import de.danoeh.antennapod.core.preferences.UserPreferences;
 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 io.reactivex.Completable;
+import io.reactivex.android.schedulers.AndroidSchedulers;
+import io.reactivex.schedulers.Schedulers;
+import org.apache.commons.lang3.Validate;
+
 
 /**
  * Shows a username and a password text field.
  * The activity MUST be started with the ARG_DOWNlOAD_REQUEST argument set to a non-null value.
- * Other arguments are optional.
- * The activity's result will be the same DownloadRequest with the entered username and password.
  */
 public class DownloadAuthenticationActivity extends AppCompatActivity {
 
     /**
-     * The download request object that contains information about the resource that requires a username and a password
+     * The download request object that contains information about the resource that requires a username and a password.
      */
     public static final String ARG_DOWNLOAD_REQUEST = "request";
-    /**
-     * True if the request should be sent to the DownloadRequester when this activity is finished, false otherwise.
-     * The default value is false.
-     */
-    public static final String ARG_SEND_TO_DOWNLOAD_REQUESTER_BOOL = "send_to_downloadrequester";
-
-    private static final String RESULT_REQUEST = "request";
 
     private EditText etxtUsername;
     private EditText etxtPassword;
+    private DownloadRequest request;
 
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         setTheme(UserPreferences.getNoTitleTheme());
         super.onCreate(savedInstanceState);
-
         setContentView(R.layout.download_authentication_activity);
-        TextView txtvDescription = findViewById(R.id.txtvDescription);
+
         etxtUsername = findViewById(R.id.etxtUsername);
         etxtPassword = findViewById(R.id.etxtPassword);
-        Button butConfirm = findViewById(R.id.butConfirm);
-        Button butCancel = findViewById(R.id.butCancel);
 
         Validate.isTrue(getIntent().hasExtra(ARG_DOWNLOAD_REQUEST), "Download request missing");
-        DownloadRequest request = getIntent().getParcelableExtra(ARG_DOWNLOAD_REQUEST);
-        boolean sendToDownloadRequester = getIntent().getBooleanExtra(ARG_SEND_TO_DOWNLOAD_REQUESTER_BOOL, false);
+        request = getIntent().getParcelableExtra(ARG_DOWNLOAD_REQUEST);
 
+        TextView txtvDescription = findViewById(R.id.txtvDescription);
         String newDescription = txtvDescription.getText() + ":\n\n" + request.getTitle();
         txtvDescription.setText(newDescription);
 
@@ -64,28 +57,42 @@ public class DownloadAuthenticationActivity extends AppCompatActivity {
             etxtPassword.setText(savedInstanceState.getString("password"));
         }
 
-        butConfirm.setOnClickListener(v -> {
-            String username = etxtUsername.getText().toString();
-            String password = etxtPassword.getText().toString();
-            request.setUsername(username);
-            request.setPassword(password);
-            Intent result = new Intent();
-            result.putExtra(RESULT_REQUEST, request);
-            setResult(Activity.RESULT_OK, result);
-
-            if (sendToDownloadRequester) {
-                DownloadRequester.getInstance().download(DownloadAuthenticationActivity.this, request);
-            }
-            finish();
-        });
+        findViewById(R.id.butConfirm).setOnClickListener(v ->
+                Completable.fromAction(this::updatePassword)
+                .subscribeOn(Schedulers.io())
+                .observeOn(AndroidSchedulers.mainThread())
+                .subscribe(() -> {
+                    DownloadRequester.getInstance().download(DownloadAuthenticationActivity.this, request);
+                    finish();
+                }));
 
-        butCancel.setOnClickListener(v -> {
+        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);
+                }
+            }
+        }
+    }
+
     @Override
     protected void onSaveInstanceState(@NonNull Bundle outState) {
         super.onSaveInstanceState(outState);
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 55bf05e43..f509f6dd4 100644
--- a/app/src/main/java/de/danoeh/antennapod/config/DownloadServiceCallbacksImpl.java
+++ b/app/src/main/java/de/danoeh/antennapod/config/DownloadServiceCallbacksImpl.java
@@ -31,7 +31,6 @@ public class DownloadServiceCallbacksImpl implements DownloadServiceCallbacks {
     public PendingIntent getAuthentificationNotificationContentIntent(Context context, DownloadRequest request) {
         final Intent activityIntent = new Intent(context.getApplicationContext(), DownloadAuthenticationActivity.class);
         activityIntent.putExtra(DownloadAuthenticationActivity.ARG_DOWNLOAD_REQUEST, request);
-        activityIntent.putExtra(DownloadAuthenticationActivity.ARG_SEND_TO_DOWNLOAD_REQUESTER_BOOL, true);
         return PendingIntent.getActivity(context.getApplicationContext(),
                 R.id.pending_intent_download_service_auth, activityIntent, PendingIntent.FLAG_ONE_SHOT);
     }