Candidats : limitation du nom à 70 caractères et correctif et sécurité#4320
Candidats : limitation du nom à 70 caractères et correctif et sécurité#4320celine-m-s merged 2 commits intomasterfrom
Conversation
celine-m-s
changed the title
celine-m-s
force-pushed
the
celinems/secu
branch
from
57a1936 to
9637fc3
Compare
celine-m-s
force-pushed
the
celinems/secu
branch
from
9637fc3 to
af856b6
Compare
| """ | ||
| full_name = f"{self.first_name.strip().title()} {self.last_name.upper()}" | ||
| return full_name.strip() | ||
| return full_name.strip()[:70] |
There was a problem hiding this comment.
Plus besoin de cela si ?
There was a problem hiding this comment.
C'est un ajout indépendant. En fait, j'aurais pu en faire une PR en soi.
Je trouve que c'est mieux de tronquer les noms trop longs. Certains utilisateurs se trompent de champ et indiquent des adresses postales.
There was a problem hiding this comment.
Je ne comprends pas ce changement. Les noms peuvent être très long, et il est toujours possible de rentrer 150 caractères pour le prénom et 150 pour le nom, c’est juste qu’on n’affichera juste les 70 premiers lorsqu’on utilise get_full_name().
De plus, rien n’indique la troncation (django.utils.text.Truncator est disponible si besoin).
| def get_kind_display(self): | ||
| return UserKind(self.kind).label | ||
|
|
||
| def autocomplete_display(self): |
There was a problem hiding this comment.
C'est la méthode par défaut pour RemoteAutocompleteSelect2Widget.
J'ai effectivement l'impression que ce n'est pas utilisé dans le formulaire GPS mais potentiellement dans d'autres ?
There was a problem hiding this comment.
La méthode a été introduite par Vincent quand l'app GPS a vu le jour. Tous les tests passent donc je dirais que c'est le seul endroit où elle est utilisée.
3 participants
🤔 Pourquoi ?
Corrections suite à la campagne de Bug bounty.
🚨 À vérifier
🏝️ Comment tester
💻 Captures d'écran