summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author João Victor Mendes Freire <jvfreire@google.com> 2022-10-13 17:26:55 +0000
committer João Victor Mendes Freire <jvfreire@google.com> 2022-10-14 12:05:51 +0000
commit2ded76c29ffe375c726b6da10c0d6f9acec5425b (patch)
treef80032588ed22cac6462726bc6561d07b1e5d55d
parentb5a160241b6f22bf09c0583d554602e8e371e67b (diff)
Fix creating multiple users from same alert dialog
Disable the button listeners after the first click to avoid the creation of more than one user per dialog. Bug: 252831574 Test: Manually tested by calling the API from a regular app Change-Id: I6d343ae0a2a4d81d35a85f51a60c88c0c18dba21
-rw-r--r--core/java/com/android/internal/app/ConfirmUserCreationActivity.java5
1 files changed, 4 insertions, 1 deletions
diff --git a/core/java/com/android/internal/app/ConfirmUserCreationActivity.java b/core/java/com/android/internal/app/ConfirmUserCreationActivity.java
index 8833512c2217..0a28997885ff 100644
--- a/core/java/com/android/internal/app/ConfirmUserCreationActivity.java
+++ b/core/java/com/android/internal/app/ConfirmUserCreationActivity.java
@@ -49,6 +49,7 @@ public class ConfirmUserCreationActivity extends AlertActivity
private String mAccountType;
private PersistableBundle mAccountOptions;
private boolean mCanProceed;
+ private boolean mIsFirstClick;
private UserManager mUserManager;
@Override
@@ -82,6 +83,7 @@ public class ConfirmUserCreationActivity extends AlertActivity
ap.mNegativeButtonText = getString(android.R.string.cancel);
ap.mNegativeButtonListener = this;
}
+ mIsFirstClick = true;
setupAlert();
}
@@ -128,7 +130,8 @@ public class ConfirmUserCreationActivity extends AlertActivity
@Override
public void onClick(DialogInterface dialog, int which) {
setResult(RESULT_CANCELED);
- if (which == BUTTON_POSITIVE && mCanProceed) {
+ if (which == BUTTON_POSITIVE && mCanProceed && mIsFirstClick) {
+ mIsFirstClick = false;
Log.i(TAG, "Ok, creating user");
UserInfo user = mUserManager.createUser(mUserName, USER_TYPE, 0);
if (user == null) {