search

betagouv / pop

Plateforme Ouverte du Patrimoine
https://pop.culture.gouv.fr/
MIT License
17 stars 9 forks source link

Contrôle de l'identifiant Muséofile #171

Open Manu1400 opened 5 years ago

Manu1400 commented 5 years ago

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 5 years ago

La partie routage n'est pas affectée : 👍 https://www.pop.culture.gouv.fr/museo/M5031 OK https://www.pop.culture.gouv.fr/museo/M1037-38 OK

Manu1400 commented 5 years ago

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

goffle commented 5 years ago

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