refactor: Rename shouldEnableConfirmButton with `isConfirmButtonEna…

…bled`
Infomaniak · Dec 3, 2024 · c78ac8b · c78ac8b
1 parent a15b681
commit c78ac8b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/app/src/main/java/com/infomaniak/swisstransfer/ui/components/SwissTransferAlertDialog.kt b/app/src/main/java/com/infomaniak/swisstransfer/ui/components/SwissTransferAlertDialog.kt
@@ -39,7 +39,7 @@ fun SwissTransferAlertDialog(
     @StringRes descriptionRes: Int,
     onDismiss: () -> Unit,
     onConfirmation: () -> Unit,
-    shouldEnableConfirmButton: () -> Boolean = { true },
+    isConfirmButtonEnabled: () -> Boolean = { true },
     content: @Composable (ColumnScope.() -> Unit)? = null,
 ) {
     BasicAlertDialog(
@@ -56,7 +56,7 @@ fun SwissTransferAlertDialog(
                 additionalContent = content,
                 onDismiss = onDismiss,
                 onConfirmation = onConfirmation,
-                shouldEnableConfirmButton = shouldEnableConfirmButton,
+                isConfirmButtonEnabled = isConfirmButtonEnabled,
             )
         }
     }
@@ -70,7 +70,7 @@ private fun BasicAlertDialogContent(
     additionalContent: @Composable (ColumnScope.() -> Unit)? = null,
     onDismiss: () -> Unit,
     onConfirmation: () -> Unit,
-    shouldEnableConfirmButton: () -> Boolean = { true },
+    isConfirmButtonEnabled: () -> Boolean = { true },
 ) {
     Column(modifier.padding(Margin.Large)) {
         TitleAndDescription(titleRes, descriptionRes)
@@ -79,7 +79,7 @@ private fun BasicAlertDialogContent(
             it()
             Spacer(Modifier.height(Margin.Mini))
         }
-        ActionButtons(onDismiss, onConfirmation, shouldEnableConfirmButton)
+        ActionButtons(onDismiss, onConfirmation, isConfirmButtonEnabled)
     }
 }
 
@@ -99,7 +99,7 @@ private fun TitleAndDescription(titleRes: Int, descriptionRes: Int) {
 }
 
 @Composable
-private fun ActionButtons(onDismissRequest: () -> Unit, onConfirmation: () -> Unit, shouldEnable: () -> Boolean) {
+private fun ActionButtons(onDismissRequest: () -> Unit, onConfirmation: () -> Unit, isEnabled: () -> Boolean) {
     Row(
         modifier = Modifier.fillMaxWidth(),
         horizontalArrangement = Arrangement.End,
@@ -114,7 +114,7 @@ private fun ActionButtons(onDismissRequest: () -> Unit, onConfirmation: () -> Un
         SmallButton(
             titleRes = R.string.buttonConfirm,
             onClick = onConfirmation,
-            enabled = shouldEnable
+            enabled = isEnabled
         )
     }
 }

diff --git a/...k/swisstransfer/ui/screen/newtransfer/importfiles/components/PasswordOptionAlertDialog.kt b/...k/swisstransfer/ui/screen/newtransfer/importfiles/components/PasswordOptionAlertDialog.kt
@@ -72,7 +72,7 @@ fun PasswordOptionAlertDialog(
         descriptionRes = R.string.settingsPasswordDescription,
         onDismiss = ::onDismiss,
         onConfirmation = ::onConfirmButtonClicked,
-        shouldEnableConfirmButton = { !isPasswordActivated || isPasswordValid() },
+        isConfirmButtonEnabled = { !isPasswordActivated || isPasswordValid() },
     ) {
         ActivatePasswordSwitch(isChecked = isPasswordActivated, onCheckedChange = { isPasswordActivated = it })
         Spacer(Modifier.height(Margin.Medium))