betagouv/pop

Contrôle de l'identifiant Muséofile

Opened this issue · 3 comments

Manu1400 commented

La ligne suivante utilise une expression rationnelle pour l'identifiant (français) Muséofile trop restrictive.

} else if (!req.body.museofile.match(/^M[0-9]+$/)) {

https://github.com/betagouv/pop/blob/master/apps/api/src/controllers/auth.js#L31

Je recommande d'utiliser plutôt M\d{4}(-\d{1,2})?

Exemple réel : musée des Tissus et des Arts décoratifs | M1037-38

Manu1400 commented

Documentation : Le format avec un tiret existe depuis au moins début juin 2013

goffle commented

Bonjour !
Merci pour le retour. Je suis sur le sujet d'harmonisation du code MUSEO ! Le timing est parfait