Skip to content

Commit

Permalink
refact: [admin/users] modifie la section utilisateur de l’espace admi…
Browse files Browse the repository at this point in the history
…nistration

* restructuration du module adminstration en fragments / widgets
  • Loading branch information
matthieuaudemard committed Apr 30, 2023
1 parent a5bcb4d commit a204595
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ public void signup(@Valid final RegisterRequest registerRequest) throws Implicac
final String activationKey = generateActivationKey();
final UserModel user = registerUser(registerRequest, activationKey);


if (user.isEnabled()) {
publisher.publishEvent(new UserEnabledEvent(this, user.getUsername()));
} else {
Expand All @@ -83,7 +83,7 @@ public void signup(@Valid final RegisterRequest registerRequest) throws Implicac
/**
* Vérifie la validité de la requête de sign-up
*
* @throws AuthenticationException si le nom d’utilisateur ou l’email est déjà utilisé
* @throws AuthenticationException si le nom d’utilisateur ou l’adresse mail est déjà utilisé
*/
private void validateRegisterRequest(@Valid final RegisterRequest registerRequest) throws ImplicactionException {
if (userRepository.existsByUsername(registerRequest.getUsername())) {
Expand All @@ -96,10 +96,10 @@ private void validateRegisterRequest(@Valid final RegisterRequest registerReques
}

/**
* Définit l’adresse email de l’utilisateur correspondant à la clé d’activation comme vérifié si elle ne l’est pas déjà
* Définit l’adresse mail de l’utilisateur correspondant à la clé d’activation comme vérifié si elle ne l’est pas déjà
*
* @throws EntityNotFoundException si aucun utilisateur n’est associé à cette clé d’activation
* @throws AuthenticationException si l’email de l’utilisateur est déjà vérifié
* @throws AuthenticationException si l’adresse mail de l’utilisateur est déjà vérifié
*/
@Transactional
public void verifyAccount(final String activationKey) throws ImplicactionException {
Expand All @@ -114,7 +114,7 @@ public void verifyAccount(final String activationKey) throws ImplicactionExcepti
userRepository.save(user);
}

// TODO: revoir potentiellement la logique des jetons (ex: suppression des refresh tokens associés précédemment)
// TODO : revoir potentiellement la logique des jetons (ex: suppression des refresh tokens associés précédemment)
@Transactional
public LoginResponse login(@Valid final LoginRequest loginRequest) throws ImplicactionException {
final String username = loginRequest.getUsername();
Expand Down Expand Up @@ -179,8 +179,7 @@ private UserModel registerUser(final RegisterRequest registerRequest, final Stri
.registeredAt(now())
.build();

// On est obligé de flush ici pour que l'objet soit modifié instantanément dans le cas
// où il faille créer le profil dans la foulée
// On est obligé de flush ici pour que l’objet soit modifié instantanément dans le cas où il faille créer le profil dans la foulée.
userRepository.saveAndFlush(user);

return user;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ <h2 class="card-title">{{title}}</h2>
<p-multiSelect
optionLabel="label"
optionValue="code"
[styleClass]="'col-10'"
[styleClass]="'col-8'"
[(ngModel)]="user.roles"
[showHeader]="false"
[options]="roles"
Expand Down

0 comments on commit a204595

Please sign in to comment.