search

lpoaura / gn_vn2synthese

SQL Scripts to automatically populate GeoNature synthesis from VisioNature datas
3 stars 2 forks source link

Branche 'next' : quelques potentielles coquilles dans les scripts SQL #3

Open eguilley opened 3 years ago

eguilley commented 3 years ago

:point_right: En utilisant les scripts SQL de la branche next pour la BDD GeoNature Pays de la Loire, j'ai repéré ce qui me semble être quelques coquilles (ou peut-être pas :woman_shrugging:) :

00_init_db.sql

Manque le nom du schéma src_lpodatas : https://github.com/lpoaura/gn_vn2synthese/blob/d8fe7b90ded76e7ee7e1b3f54863e94ad569b43b/00_init_db.sql#L6

07_update_synthese_extended.sql

Il y a un gnadm qui traîne alors que partout ailleurs c'est geonatadmin : https://github.com/lpoaura/gn_vn2synthese/blob/d8fe7b90ded76e7ee7e1b3f54863e94ad569b43b/07_update_synthese_extended.sql#L488-L489

08a_init_data_nomenclatures.sql

J'ai l'impression que ce bloc est un test ? Je me demandais s'il n'était pas nécessaire de le commenter : https://github.com/lpoaura/gn_vn2synthese/blob/d8fe7b90ded76e7ee7e1b3f54863e94ad569b43b/08a_init_data_nomenclatures.sql#L744-L759

eguilley commented 3 years ago

07_update_synthese_extended.sql

D'après les conventions de nommage, le trigger devrait s'appeler tri_c_upsert_vn_observation_to_geonature et non fct_tri_c_upsert_vn_observation_to_geonature :

https://github.com/lpoaura/gn_vn2synthese/blob/d8fe7b90ded76e7ee7e1b3f54863e94ad569b43b/07_update_synthese_extended.sql#L443-L451