Better check and handling of user creation
This commit is contained in:
parent
4eff27ed42
commit
59627367ae
2 changed files with 6 additions and 3 deletions
|
@ -182,7 +182,7 @@ class OC_User {
|
|||
$backend->createUser($uid, $password);
|
||||
OC_Hook::emit( "OC_User", "post_createUser", array( "uid" => $uid, "password" => $password ));
|
||||
|
||||
return true;
|
||||
return self::userExists($uid);
|
||||
}
|
||||
}
|
||||
return false;
|
||||
|
|
|
@ -29,14 +29,17 @@ $username = $_POST["username"];
|
|||
$password = $_POST["password"];
|
||||
|
||||
// Does the group exist?
|
||||
if( in_array( $username, OC_User::getUsers())) {
|
||||
if(OC_User::userExists($username)) {
|
||||
OC_JSON::error(array("data" => array( "message" => "User already exists" )));
|
||||
exit();
|
||||
}
|
||||
|
||||
// Return Success story
|
||||
try {
|
||||
OC_User::createUser($username, $password);
|
||||
if (!OC_User::createUser($username, $password)) {
|
||||
OC_JSON::error(array('data' => array( 'message' => 'User creation failed for '.$username )));
|
||||
exit();
|
||||
}
|
||||
foreach( $groups as $i ) {
|
||||
if(!OC_Group::groupExists($i)) {
|
||||
OC_Group::createGroup($i);
|
||||
|
|
Loading…
Reference in a new issue