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

[eguilley]

👉 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 🤷‍♀️) :

00_init_db.sql

Manque le nom du schéma src_lpodatas :

gn_vn2synthese/00_init_db.sql

Line 6 in d8fe7b9

CREATE SCHEMA IF NOT EXISTS AUTHORIZATION geonatadmin;

07_update_synthese_extended.sql

Il y a un gnadm qui traîne alors que partout ailleurs c'est geonatadmin :

gn_vn2synthese/07_update_synthese_extended.sql

Lines 488 to 489 in d8fe7b9

	ALTER FUNCTION src_lpodatas.fct_tri_c_delete_vn_observation_from_geonature () OWNER TO gnadm
	;

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 :

gn_vn2synthese/08a_init_data_nomenclatures.sql

Lines 744 to 759 in d8fe7b9

	WITH selection AS (
	SELECT
	site,
	id
	FROM
	src_vn_json.observations_json
	LIMIT 5)
	UPDATE
	src_vn_json.observations_json
	SET
	site = observations_json.site
	FROM
	selection
	WHERE
	observations_json.site = selection.site
	AND observations_json.id = selection.id

[eguilley]

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 :

gn_vn2synthese/07_update_synthese_extended.sql

Lines 443 to 451 in d8fe7b9

	DROP TRIGGER IF EXISTS fct_tri_c_upsert_vn_observation_to_geonature ON src_vn_json.observations_json
	;
	CREATE TRIGGER fct_tri_c_upsert_vn_observation_to_geonature
	AFTER INSERT OR UPDATE
	ON src_vn_json.observations_json
	FOR EACH ROW
	EXECUTE PROCEDURE src_lpodatas.fct_tri_c_upsert_vn_observation_to_geonature ()
	;