search

demarches-simplifiees / demarches-simplifiees.fr

Dématérialiser et simplifier les démarches administratives
https://www.demarches-simplifiees.fr
GNU Affero General Public License v3.0
189 stars 89 forks source link

Valider la formule de Luhn sur les SIRET #1716

Closed gregoirenovel closed 4 years ago

gregoirenovel commented 6 years ago

Cf. https://fr.wikipedia.org/wiki/Syst%C3%A8me_d%27identification_du_r%C3%A9pertoire_des_%C3%A9tablissements#Calcul_et_validit%C3%A9_d'un_num%C3%A9ro_SIRET

Afficher une erreur précise en cas de non validation

kemenaran commented 6 years ago

@gregoirenovel C'est déjà le cas, non ?

gregoirenovel commented 6 years ago

Yes, mais pas sur le champ SIRET, juste sur l'identification entreprise. Et faudrait donc porter le validateur direct dans le modèle du type de champ je pense, ping @tchak (cf. models/siret.rb et SiretFormatValidator).

D'ailleurs je ferais 2 validateurs plus disctincts qu'actuellement :

gregoirenovel commented 6 years ago

(et l'issue portait surtout sur le fait qu'actuellement les messages d'erreur sont cryptique : "Erreur SIRET" dans tous les cas (problème d'API, de format, etc.), sans plus de précision)