fix: disallow adding repo with same name

ViscousPot · Jan 16, 2025 · 43a3e5b · 43a3e5b
1 parent 0ed927c
commit 43a3e5b
Showing 1 changed file with 21 additions and 10 deletions.
diff --git a/app/src/main/java/com/viscouspot/gitsync/MainActivity.kt b/app/src/main/java/com/viscouspot/gitsync/MainActivity.kt
@@ -499,17 +499,28 @@ class MainActivity : AppCompatActivity() {
                     .setTitle(getString(R.string.add_repository))
                     .setCancelable(1)
                     .setView(keyInput)
-                    .setPositiveButton(R.string.add) { _, _ ->
-                        val repoNames = repoManager.getRepoNames().toMutableList()
-                        repoNames.add(input.text.toString())
-
-                        repoManager.setRepoNames(repoNames)
-                        repoManager.setRepoIndex(repoNames.indexOf(input.text.toString()))
-
-                        updateRepoButtons()
-                        refreshAll()
-                    }
+                    .setPositiveButton(R.string.add) { _, _ -> }
                     .setNegativeButton(android.R.string.cancel) { _, _ -> }
+                    .apply {
+                        setOnShowListener {
+                            getButton(AlertDialog.BUTTON_POSITIVE).setOnClickListener listener@{
+                                val repoNames = repoManager.getRepoNames().toMutableList()
+                                if (repoNames.contains(input.text.toString())) {
+                                    input.setText("${input.text}2")
+                                    return@listener
+                                }
+
+                                repoNames.add(input.text.toString())
+
+                                repoManager.setRepoNames(repoNames)
+                                repoManager.setRepoIndex(repoNames.indexOf(input.text.toString()))
+
+                                updateRepoButtons()
+                                refreshAll()
+                                dismiss()
+                            }
+                        }
+                    }
                     .show()
             }
         }