diff --git a/app/Services/OpenId/UserService.php b/app/Services/OpenId/UserService.php index 444570ed..9c60679f 100644 --- a/app/Services/OpenId/UserService.php +++ b/app/Services/OpenId/UserService.php @@ -265,7 +265,7 @@ final class UserService extends AbstractService implements IUserService } } - if(isset($payload["email"])){ + if(isset($payload["email"]) && !empty($payload["email"])){ $former_user = $this->repository->getByEmailOrName(trim($payload["email"])); if(!is_null($former_user) && $former_user->getId() != $id) throw new ValidationException(sprintf("email %s already belongs to another user", $payload["email"])); diff --git a/app/libs/Auth/Factories/UserFactory.php b/app/libs/Auth/Factories/UserFactory.php index 91217189..78c96bcf 100644 --- a/app/libs/Auth/Factories/UserFactory.php +++ b/app/libs/Auth/Factories/UserFactory.php @@ -48,11 +48,12 @@ final class UserFactory if(isset($payload['last_name'])) $user->setLastName(trim($payload['last_name'])); - if(isset($payload['email'])) + if(isset($payload['email']) && !empty($payload['email'])) $user->setEmail(strtolower(trim($payload['email']))); if(isset($payload['second_email'])) $user->setSecondEmail(strtolower(trim($payload['second_email']))); + if(isset($payload['third_email'])) $user->setThirdEmail(strtolower(trim($payload['third_email'])));