Feature/frontend/admin user - #64
Conversation
… and dynamic action visibility
…e_frontback into feature/frontend/item-management
… v1.1.0 and enhance item fetching with soft-delete functionality
…logic to use isDeleted flag
…nd hard deletion functionalities
…ling and localization support
… localization, and data fetching
…issions | feature backend : adapt backend for the frontend by adding some endpoints
Co-authored-by: Copilot <copilot@github.com>
…Informatique/template_frontback into feature/frontend/admin-user
…er roles instead of just spring-auth roles
…Informatique/template_frontback into feature/frontend/admin-user
KarolJar
left a comment
There was a problem hiding this comment.
Il y a deux problèmes:
On peut pas supprimer un utilisateur globalement: quand on fait une requete delete "http://localhost:8086/users/10/true",
Ca nous donne un "Failed to delete user: Method 'DELETE' is not supported."
Quand on veut mettre un utilisateur à jour ca donne un Internal Server Error.
DidierViret
left a comment
There was a problem hiding this comment.
Erreurs signalées par l'IA dans le backend :
UserController.java — updateUser() commit la mise à jour en local avant d'appeler spring-auth. Si l'appel distant échoue, l'utilisateur local reste modifié alors que spring-auth ne l'est pas → désynchronisation du mainRole entre les deux systèmes.
UserService.java — contrairement aux autres méthodes de la classe, updateUser() n'est pas dans un try/catch, et RoleEnum.valueOf(newUser.getMainRole()) n'a pas de garde contre un mainRole null → NullPointerException non gérée, 500 brut au lieu d'une erreur i18n propre.
UserService.java — register() peut désormais lever RoleNotFoundException, mais le catch ne liste toujours que DefaultRoleNotFoundException. Elle tombe donc dans le catch (Exception e) générique et est ré-enveloppée en UserCreationException, perdant le statut 404 et la clé de message i18n (et e.getMessage() est probablement null → message d'erreur vide/confus).
DidierViret
left a comment
There was a problem hiding this comment.
Erreurs signalées par l'IA dans le frontend :
users/api/api.js — getUsers()/getUserWithDeleted() appellent GET /users/all et /users/all-with-deleted, mais UserController n'expose que GET /users (avec un paramètre ?deleted=). Ces routes n'existent pas côté backend → 404 avalé silencieusement → la liste des utilisateurs s'affiche toujours vide.
users/api/api.js — Les fonctions de suppression appellent DELETE /users/{id}/false ou /{id}/true/permanent en segments d'URL, mais UserController.deleteUser attend DELETE /{userLogin} avec global/hard en query params, et un login (pas un id numérique). Suppression/hard-delete depuis l'UI admin échoue silencieusement (404 avalé → aucun retour visible).
…Add redirection to the last page after a successfull login.
GET /users/all a déja été fixé dans la branche feature/frontend/administration-page |
…ser ID, cleanup of mock
Add a list to manage the users from the frontend