diff --git a/source/java/org/alfresco/rest/api/impl/PeopleImpl.java b/source/java/org/alfresco/rest/api/impl/PeopleImpl.java index b8d46875db..0c6ee11026 100644 --- a/source/java/org/alfresco/rest/api/impl/PeopleImpl.java +++ b/source/java/org/alfresco/rest/api/impl/PeopleImpl.java @@ -393,7 +393,14 @@ public class PeopleImpl implements People { throw new ConstraintViolatedException("Person '"+person.getUserName()+"' already exists."); } - Map props = person.toProperties(); + + // set enabled default value true + if (person.isEnabled() == null) + { + person.setEnabled(true); + } + + Map props = person.toProperties(); MutableAuthenticationService mas = (MutableAuthenticationService) authenticationService; mas.createAuthentication(person.getUserName(), person.getPassword().toCharArray()); @@ -437,7 +444,6 @@ public class PeopleImpl implements People checkRequiredField("id", person.getUserName()); checkRequiredField("firstName", person.getFirstName()); checkRequiredField("email", person.getEmail()); - checkRequiredField("enabled", person.isEnabled()); checkRequiredField("password", person.getPassword()); } diff --git a/source/test-java/org/alfresco/rest/api/tests/TestPeople.java b/source/test-java/org/alfresco/rest/api/tests/TestPeople.java index 0fbc0b65c3..2630ab782e 100644 --- a/source/test-java/org/alfresco/rest/api/tests/TestPeople.java +++ b/source/test-java/org/alfresco/rest/api/tests/TestPeople.java @@ -318,7 +318,6 @@ public class TestPeople extends EnterpriseTestApi person.setUserName("joe.bloggs.2@"+account1.getId()); person.setFirstName("Joe"); person.setEmail("joe.bloggs.2@example.com"); - person.setEnabled(true); person.setPassword("password-is-secret"); Person p = people.create(person);