Skip to content
This repository has been archived by the owner on Feb 10, 2022. It is now read-only.

Ne pas utiliser les identifiants nomenclature dans les fichiers de conf mais les codes #30

Closed
amandine-sahl opened this issue Jun 6, 2019 · 2 comments
Labels
bug Something isn't working

Comments

@amandine-sahl
Copy link

Dans le fichier conf_schema_toml.py les identifiants nomenclatures sont en dur (tableau STATUS_INFO) ce qui pose des soucis de compatibilité avec la nomenclature. Il faudrait mieux utiliser les codes nomenclatures (cd_nomenclature) qui eux sont censés être persistants entre les instances.

Par ailleurs il manque un test pour vérifier que l'identifiant retourné dans le jeux de données est bien dans la liste VALIDATION_CONFIG.STATUS_INFO

@amandine-sahl amandine-sahl added the bug Something isn't working label Jun 6, 2019
@camillemonchicourt
Copy link
Member

Ah oui bien vu, surtout que ça a posé des problèmes à certains.
Ça serait encore plus simple et souple d'utiliser les CODES.
Dans ce cas il faudrait ainsi un paramètre pour l'id ou le code du type de nomenclature, pour ne pas non plus le mettre en dur dans le code.

@camillemonchicourt
Copy link
Member

camillemonchicourt commented Jun 13, 2019

Fait ici en même temps que de la refactorisation du code : PnX-SI/GeoNature@a9491f9

On ne liste plus les id_nomenclatures à utiliser dans la conf du module. Mais on utilise toutes les nomenclatures activées du type de nomenclature STATUT_VALID.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants