From 91e00d897f190ec6ab5330f7b3f43d6d7f3e57fb Mon Sep 17 00:00:00 2001
From: Harsh Shandilya <me@msfjarvis.dev>
Date: Fri, 8 Jan 2021 19:43:52 +0530
Subject: app: cleanup how we handle menu item presses (#1275)

Fixes #1274

Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
---
 .../msfjarvis/aps/ui/passwords/PasswordStore.kt    | 29 +++++++---------------
 1 file changed, 9 insertions(+), 20 deletions(-)

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

diff --git a/app/src/main/java/dev/msfjarvis/aps/ui/passwords/PasswordStore.kt b/app/src/main/java/dev/msfjarvis/aps/ui/passwords/PasswordStore.kt
index c7ed5ee9..49620ac8 100644
--- a/app/src/main/java/dev/msfjarvis/aps/ui/passwords/PasswordStore.kt
+++ b/app/src/main/java/dev/msfjarvis/aps/ui/passwords/PasswordStore.kt
@@ -292,9 +292,6 @@ class PasswordStore : BaseGitActivity() {
         return super.onPrepareOptionsMenu(menu)
     }
 
-    // Handle action bar item clicks here. The action bar will
-    // automatically handle clicks on the Home/Up button, so long
-    // as you specify a parent activity in AndroidManifest.xml.
     override fun onOptionsItemSelected(item: MenuItem): Boolean {
         val id = item.itemId
         val initBefore = MaterialAlertDialogBuilder(this)
@@ -307,41 +304,33 @@ class PasswordStore : BaseGitActivity() {
                 }.onFailure { e ->
                     e.printStackTrace()
                 }
-                return true
             }
             R.id.git_push -> {
                 if (!PasswordRepository.isInitialized) {
                     initBefore.show()
-                    return false
+                } else {
+                    runGitOperation(GitOp.PUSH)
                 }
-                runGitOperation(GitOp.PUSH)
-                return true
             }
             R.id.git_pull -> {
                 if (!PasswordRepository.isInitialized) {
                     initBefore.show()
-                    return false
+                } else {
+                    runGitOperation(GitOp.PULL)
                 }
-                runGitOperation(GitOp.PULL)
-                return true
             }
             R.id.git_sync -> {
                 if (!PasswordRepository.isInitialized) {
                     initBefore.show()
-                    return false
+                } else {
+                    runGitOperation(GitOp.SYNC)
                 }
-                runGitOperation(GitOp.SYNC)
-                return true
-            }
-            R.id.refresh -> {
-                refreshPasswordList()
-                return true
             }
+            R.id.refresh -> refreshPasswordList()
             android.R.id.home -> onBackPressed()
-            else -> {
-            }
+            else -> return super.onOptionsItemSelected(item)
         }
-        return super.onOptionsItemSelected(item)
+        return true
     }
 
     override fun onBackPressed() {
-- 
cgit v1.2.3