Merge pull request #5 from omg-xtao/fix

Fix the existing users can't register
This commit is contained in:
Muhammad Eko Prasetyo 2022-05-06 07:08:21 +07:00 committed by GitHub
commit 517a8ea5b3
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -30,15 +30,24 @@ public class RegisterHandler implements HttpContextHandler {
if (registerAccount.password.equals(registerAccount.password_confirmation)) { if (registerAccount.password.equals(registerAccount.password_confirmation)) {
if (registerAccount.password.length() >= 8) { if (registerAccount.password.length() >= 8) {
String password = Authentication.generateHash(registerAccount.password); String password = Authentication.generateHash(registerAccount.password);
Account account = DatabaseHelper.createAccountWithPassword(registerAccount.username, password); Account account = Authentication.getAccountByUsernameAndPassword(registerAccount.username, "");
if (account == null) { if (account != null) {
authResponse.success = false; account.setPassword(password);
authResponse.message = "USERNAME_TAKEN"; // ENG = "Username has already been taken by another user." account.save();
authResponse.jwt = "";
} else {
authResponse.success = true; authResponse.success = true;
authResponse.message = ""; authResponse.message = "";
authResponse.jwt = ""; authResponse.jwt = "";
} else {
account = DatabaseHelper.createAccountWithPassword(registerAccount.username, password);
if (account == null) {
authResponse.success = false;
authResponse.message = "USERNAME_TAKEN"; // ENG = "Username has already been taken by another user."
authResponse.jwt = "";
} else {
authResponse.success = true;
authResponse.message = "";
authResponse.jwt = "";
}
} }
} else { } else {
authResponse.success = false; authResponse.success = false;