From 372c0f3dbd395e93c89ac0730a06344e3c9d6376 Mon Sep 17 00:00:00 2001 From: Harsh Shandilya Date: Wed, 12 Aug 2020 13:04:47 +0530 Subject: Improve clone operation semantics (#1010) * Improve clone operation semantics Fixes #1003 Signed-off-by: Harsh Shandilya * Update changelog Signed-off-by: Harsh Shandilya --- .../java/com/zeapo/pwdstore/git/operation/CloneOperation.kt | 11 ++++++++++- .../java/com/zeapo/pwdstore/git/operation/GitOperation.kt | 3 ++- 2 files changed, 12 insertions(+), 2 deletions(-) (limited to 'app/src/main/java/com') diff --git a/app/src/main/java/com/zeapo/pwdstore/git/operation/CloneOperation.kt b/app/src/main/java/com/zeapo/pwdstore/git/operation/CloneOperation.kt index 8c516eac..a032a669 100644 --- a/app/src/main/java/com/zeapo/pwdstore/git/operation/CloneOperation.kt +++ b/app/src/main/java/com/zeapo/pwdstore/git/operation/CloneOperation.kt @@ -24,6 +24,15 @@ class CloneOperation(fileDir: File, uri: String, callingActivity: AppCompatActiv ) override suspend fun execute() { - GitCommandExecutor(callingActivity, this).execute() + GitCommandExecutor(callingActivity, this, finishActivityOnEnd = false).execute() + } + + override fun onSuccess() { + callingActivity.finish() + } + + override fun onError(err: Exception) { + finishFromErrorDialog = false + super.onError(err) } } diff --git a/app/src/main/java/com/zeapo/pwdstore/git/operation/GitOperation.kt b/app/src/main/java/com/zeapo/pwdstore/git/operation/GitOperation.kt index ad8d318a..e72c5743 100644 --- a/app/src/main/java/com/zeapo/pwdstore/git/operation/GitOperation.kt +++ b/app/src/main/java/com/zeapo/pwdstore/git/operation/GitOperation.kt @@ -46,6 +46,7 @@ abstract class GitOperation(gitDir: File, internal val callingActivity: Fragment private var provider: CredentialsProvider? = null private val sshKeyFile = callingActivity.filesDir.resolve(".ssh_key") private val hostKeyFile = callingActivity.filesDir.resolve(".host_key") + protected var finishFromErrorDialog = true protected val repository = PasswordRepository.getRepository(gitDir) protected val git = Git(repository) protected val remoteBranch = GitSettings.branch @@ -166,7 +167,7 @@ abstract class GitOperation(gitDir: File, internal val callingActivity: Fragment .setTitle(callingActivity.resources.getString(R.string.jgit_error_dialog_title)) .setMessage(ErrorMessages[err]) .setPositiveButton(callingActivity.resources.getString(R.string.dialog_ok)) { _, _ -> - callingActivity.finish() + if (finishFromErrorDialog) callingActivity.finish() }.show() } -- cgit v1.2.3