- Belgium
- http://tcaresset.com
-
admin of herisson
- Joined on
2022-03-15
66282bce9f
Create new constructor for ReinscriptionRequest so you don't have to worry about the type of the request
Je suis d'accord, c'est pas nécéssaire mais on en avait parlé donc c'est pour voir. pour moi le fait qu'elles soients hard-codé c'est bon
Ca me semble bien, j'approuve parce que c'est bon mais j'avoue que l'exception me semble un peu bizzare
f.delete peut throw uniquement : SecurityException et je trouve (personnellement) que c'est assez clair comme ça. dire qu'il y a une érreur de permissions est plus clair que dire "j'ai pas réussi à supprimer lol" ^^
Autant ça me fait mal que ça soit un Object mais je comprends mais pourquoi une liste du coup ?
Si c'est du debug il faudrais l'enlever, si c'est du log c'est mieux d'utiliser Logger.info(msg) https://docs.oracle.com/en/java/javase/21/docs/api/java.logging/java/util/logging/Logger.html#info(java.lang.String)
@Maxime Le problème c'est que si tu me retourne l'url tu vas pas pouvoir le lier à la db faudrait du coup me retourner l'id que je te retourne. Ce qui n'est pas ouf non plus mais bon. en vrai…
c'est parce que j'avais fait ça avant qu'on parle des pp 👉👈