libre.sh/keycloak-scim
3
0
Fork 0
Code Issues 15 Pull requests Releases Packages Activity Actions

KEYCLOAK-74 name to 1st/last name in registration form

Browse source
This commit is contained in:
vrockai 2013-09-27 15:55:13 +02:00
parent 8663c9f026
commit 32779d956a
9 changed files with 39 additions and 43 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
forms/src/main/resources/META-INF/resources/forms/theme/default/register.ftl
View file

@ -14,8 +14,12 @@
<form action="${url.registrationAction}" method="post"> <form action="${url.registrationAction}" method="post">
<p class="subtitle">${rb.getString('allRequired')}</p> <p class="subtitle">${rb.getString('allRequired')}</p>
<div> <div>
<label for="name">${rb.getString('fullName')}</label> <label for="firstName">${rb.getString('firstName')}</label>
<input type="text" id="name" name="name" value="${register.formData.name!''}" /> <input type="text" id="firstName" name="firstName" value="${register.formData.firstName!''}" />
</div>
<div>
<label for="lastName">${rb.getString('lastName')}</label>
<input type="text" id="lastName" name="lastName" value="${register.formData.lastName!''}" />
</div> </div>
<div> <div>
<label for="email">${rb.getString('email')}</label> <label for="email">${rb.getString('email')}</label>

3
forms/src/main/resources/org/keycloak/forms/messages.properties
View file

@ -25,7 +25,8 @@ invalidUser=Invalid username or password
invalidPassword=Invalid username or password invalidPassword=Invalid username or password
accountDisabled=Account is disabled, contact admin accountDisabled=Account is disabled, contact admin
missingName=Please specify full name missingFirstName=Please specify first name
missingLastName=Please specify last name
missingEmail=Please specify email missingEmail=Please specify email
missingUsername=Please specify username missingUsername=Please specify username
missingPassword=Please specify password missingPassword=Please specify password

4
services/src/main/java/org/keycloak/services/messages/Messages.java
View file

@ -38,7 +38,9 @@ public class Messages {
public static final String MISSING_EMAIL = "missingEmail"; public static final String MISSING_EMAIL = "missingEmail";
public static final String MISSING_NAME = "missingName"; public static final String MISSING_FIRST_NAME = "missingFirstName";
public static final String MISSING_LAST_NAME = "missingLastName";
public static final String MISSING_PASSWORD = "missingPassword"; public static final String MISSING_PASSWORD = "missingPassword";

26
services/src/main/java/org/keycloak/services/resources/TokenService.java
View file

@ -286,30 +286,8 @@ public class TokenService {
} }
user = realm.addUser(username); user = realm.addUser(username);
user.setFirstName(formData.getFirst("firstName"));
String fullname = formData.getFirst("name"); user.setLastName(formData.getFirst("lastName"));
if (fullname != null) {
StringTokenizer tokenizer = new StringTokenizer(fullname, " ");
StringBuffer first = null;
String last = "";
while (tokenizer.hasMoreTokens()) {
String token = tokenizer.nextToken();
if (tokenizer.hasMoreTokens()) {
if (first == null) {
first = new StringBuffer();
} else {
first.append(" ");
}
first.append(token);
} else {
last = token;
}
}
if (first == null)
first = new StringBuffer();
user.setFirstName(first.toString());
user.setLastName(last);
}
user.setEmail(formData.getFirst("email")); user.setEmail(formData.getFirst("email"));

8
services/src/main/java/org/keycloak/services/validation/Validation.java
View file

@ -9,8 +9,12 @@ import java.util.List;
public class Validation { public class Validation {
public static String validateRegistrationForm(MultivaluedMap<String, String> formData, List<String> requiredCredentialTypes) { public static String validateRegistrationForm(MultivaluedMap<String, String> formData, List<String> requiredCredentialTypes) {
if (isEmpty(formData.getFirst("name"))) { if (isEmpty(formData.getFirst("firstName"))) {
return Messages.MISSING_NAME; return Messages.MISSING_FIRST_NAME;
}
if (isEmpty(formData.getFirst("lastName"))) {
return Messages.MISSING_LAST_NAME;
} }
if (isEmpty(formData.getFirst("email"))) { if (isEmpty(formData.getFirst("email"))) {

2
testsuite/src/test/java/org/keycloak/testsuite/actions/RequiredActionEmailVerificationTest.java
View file

@ -109,7 +109,7 @@ public class RequiredActionEmailVerificationTest {
public void verifyEmailRegister() throws IOException, MessagingException { public void verifyEmailRegister() throws IOException, MessagingException {
loginPage.open(); loginPage.open();
loginPage.clickRegister(); loginPage.clickRegister();
registerPage.register("name", "email", "verifyEmail", "password", "password"); registerPage.register("firstName", "lastName", "email", "verifyEmail", "password", "password");
Assert.assertTrue(driver.getPageSource().contains("Verify email")); Assert.assertTrue(driver.getPageSource().contains("Verify email"));

2
testsuite/src/test/java/org/keycloak/testsuite/actions/RequiredActionTotpSetupTest.java
View file

@ -85,7 +85,7 @@ public class RequiredActionTotpSetupTest {
public void setupTotpRegister() { public void setupTotpRegister() {
loginPage.open(); loginPage.open();
loginPage.clickRegister(); loginPage.clickRegister();
registerPage.register("name", "email", "setupTotp", "password", "password"); registerPage.register("firstName", "lastName", "email", "setupTotp", "password", "password");
totpPage.assertCurrent(); totpPage.assertCurrent();

10
testsuite/src/test/java/org/keycloak/testsuite/forms/RegisterTest.java
View file

@ -63,7 +63,7 @@ public class RegisterTest {
loginPage.clickRegister(); loginPage.clickRegister();
registerPage.assertCurrent(); registerPage.assertCurrent();
registerPage.register("name", "email", "test-user@localhost", "password", "password"); registerPage.register("firstName", "lastName", "email", "test-user@localhost", "password", "password");
registerPage.assertCurrent(); registerPage.assertCurrent();
Assert.assertEquals("Username already exists", registerPage.getError()); Assert.assertEquals("Username already exists", registerPage.getError());
@ -75,7 +75,7 @@ public class RegisterTest {
loginPage.clickRegister(); loginPage.clickRegister();
registerPage.assertCurrent(); registerPage.assertCurrent();
registerPage.register("name", "email", "registerUserInvalidPasswordConfirm", "password", "invalid"); registerPage.register("firstName", "lastName", "email", "registerUserInvalidPasswordConfirm", "password", "invalid");
registerPage.assertCurrent(); registerPage.assertCurrent();
Assert.assertEquals("Password confirmation doesn't match", registerPage.getError()); Assert.assertEquals("Password confirmation doesn't match", registerPage.getError());
@ -87,7 +87,7 @@ public class RegisterTest {
loginPage.clickRegister(); loginPage.clickRegister();
registerPage.assertCurrent(); registerPage.assertCurrent();
registerPage.register("name", "email", "registerUserMissingPassword", null, null); registerPage.register("firstName", "lastName", "email", "registerUserMissingPassword", null, null);
registerPage.assertCurrent(); registerPage.assertCurrent();
Assert.assertEquals("Please specify password", registerPage.getError()); Assert.assertEquals("Please specify password", registerPage.getError());
@ -99,7 +99,7 @@ public class RegisterTest {
loginPage.clickRegister(); loginPage.clickRegister();
registerPage.assertCurrent(); registerPage.assertCurrent();
registerPage.register("name", "email", null, "password", "password"); registerPage.register("firstName", "lastName", "email", null, "password", "password");
registerPage.assertCurrent(); registerPage.assertCurrent();
Assert.assertEquals("Please specify username", registerPage.getError()); Assert.assertEquals("Please specify username", registerPage.getError());
@ -111,7 +111,7 @@ public class RegisterTest {
loginPage.clickRegister(); loginPage.clickRegister();
registerPage.assertCurrent(); registerPage.assertCurrent();
registerPage.register("name", "email", "registerUserSuccess", "password", "password"); registerPage.register("firstName", "lastName", "email", "registerUserSuccess", "password", "password");
Assert.assertEquals(RequestType.AUTH_RESPONSE, appPage.getRequestType()); Assert.assertEquals(RequestType.AUTH_RESPONSE, appPage.getRequestType());
} }

17
testsuite/src/test/java/org/keycloak/testsuite/pages/RegisterPage.java
View file

@ -29,8 +29,11 @@ import org.openqa.selenium.support.FindBy;
*/ */
public class RegisterPage extends Page { public class RegisterPage extends Page {
@FindBy(id = "name") @FindBy(id = "firstName")
private WebElement nameInput; private WebElement firstNameInput;
@FindBy(id = "lastName")
private WebElement lastNameInput;
@FindBy(id = "email") @FindBy(id = "email")
private WebElement emailInput; private WebElement emailInput;
@ -50,9 +53,13 @@ public class RegisterPage extends Page {
@FindBy(id = "loginError") @FindBy(id = "loginError")
private WebElement loginErrorMessage; private WebElement loginErrorMessage;
public void register(String name, String email, String username, String password, String passwordConfirm) { public void register(String firstName, String lastName, String email, String username, String password, String passwordConfirm) {
if (name != null) { if (firstName != null) {
nameInput.sendKeys(name); firstNameInput.sendKeys(firstName);
}
if (lastName != null) {
lastNameInput.sendKeys(lastName);
} }
if (email != null) { if (email != null) {