search

InseeFr / Bauhaus-Back-Office

Back-office services for Bauhaus
MIT License
3 stars 8 forks source link

refactoring controller : retourner des ResponseEntity<TypeObjet> #657

Open GtanSndil opened 1 week ago

GtanSndil commented 1 week ago

Dans les classes controller, actuellement on retourne le plus souvent une ResponseEntity (notamment cas où la classe service peut renvoyer une/des RmesException) Le mieux c'est de retourner ResponseEntity, ReponseEntity, ResponseEntity, ... si possible ça serait bien pour les GET : normalement on laisse Spring gérer la sérialisation Dans le même temps, supprimer la méthode GenericResources#returnRmesException (et en fait, la classe GenericResources) pour utiliser un @ControllerAdvice : c'est plutôt la bonne pratique

GtanSndil commented 3 days ago

Notamment régression pour CodeListsResources.deleteCodeList (pas d'affichage du message en cas d'exception) : ajouter CodeListsResources.class dans RmesExceptionHandler + supprimer le bloc catch

  • © Githubissues.
  • Githubissues is a development platform for aggregating issues.