From 6fc1fafe991d8c2a21340e41293fe130c88d48fa Mon Sep 17 00:00:00 2001
From: Harsh Shandilya <me@msfjarvis.dev>
Date: Sun, 13 Mar 2022 18:37:29 +0530
Subject: Misc PGP v2 fixes (#1784)

* Don't throw in PGPKeyImportActivity when no file is selected

* PGPSettings: mark import option as dependent on V2 backend pref
---
 .../dev/msfjarvis/aps/ui/pgp/PGPKeyImportActivity.kt    | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

(limited to 'app/src/main/java/dev/msfjarvis/aps/ui/pgp')

diff --git a/app/src/main/java/dev/msfjarvis/aps/ui/pgp/PGPKeyImportActivity.kt b/app/src/main/java/dev/msfjarvis/aps/ui/pgp/PGPKeyImportActivity.kt
index b97de5e0..02ea4fc3 100644
--- a/app/src/main/java/dev/msfjarvis/aps/ui/pgp/PGPKeyImportActivity.kt
+++ b/app/src/main/java/dev/msfjarvis/aps/ui/pgp/PGPKeyImportActivity.kt
@@ -29,7 +29,7 @@ class PGPKeyImportActivity : AppCompatActivity() {
     registerForActivityResult(OpenDocument()) { uri ->
       runCatching {
           if (uri == null) {
-            throw IllegalStateException("Selected URI was null")
+            return@runCatching null
           }
           val keyInputStream =
             contentResolver.openInputStream(uri)
@@ -41,13 +41,14 @@ class PGPKeyImportActivity : AppCompatActivity() {
         }
         .mapBoth(
           { key ->
-            require(key != null) { "Key cannot be null here" }
-            MaterialAlertDialogBuilder(this)
-              .setTitle(getString(R.string.pgp_key_import_succeeded))
-              .setMessage(getString(R.string.pgp_key_import_succeeded_message, tryGetId(key)))
-              .setPositiveButton(android.R.string.ok) { _, _ -> finish() }
-              .setOnCancelListener { finish() }
-              .show()
+            if (key != null) {
+              MaterialAlertDialogBuilder(this)
+                .setTitle(getString(R.string.pgp_key_import_succeeded))
+                .setMessage(getString(R.string.pgp_key_import_succeeded_message, tryGetId(key)))
+                .setPositiveButton(android.R.string.ok) { _, _ -> finish() }
+                .setOnCancelListener { finish() }
+                .show()
+            }
           },
           { throwable ->
             MaterialAlertDialogBuilder(this)
-- 
cgit v1.2.3