fix: display mailles option (AFFICHAGE_MAILLE = True)

[juggler31]

No description provided.

[camillemonchicourt]

C'était quoi le soucis ?
Et ça corrige donc quoi ?

[camillemonchicourt]

Si on modifie data/atlas/13.atlas.vm_observations_mailles.sql la correction sera OK pour les nouvelles installations mais pas les mises à jour.
C'est pas idéal du tout, mais jusqu'à présent on se fait des fichiers SQL à la main pour bien appliquer les modifications de la BDD à chaque MAJ, ici : https://github.com/PnX-SI/GeoNature-atlas/tree/master/data/update

On a souvent évoqué le fait de plutôt supprimer et recréer toutes la BDD, notamment les vues matérialisées à chaque mise à jour pour ne pas avoir à maintenir les fichiers de création de la BDD et des fichiers de mises à jour, car c'est lourd, risqué de diverger et compliqué à maintenir, mais pour le moment, on n'est pas allé au bout de cette approche.

[juggler31]

@camillemonchicourt Ça corrige l'option dans le fichier config.py AFFICHAGE_MAILLE = True .

Le bug a été introduit entre la branche master et develop (de ce fait, il n'y a pas eu de release entre temps) !!
Est ce que pour toi un script de migration est quand même nécessaire ?

[camillemonchicourt]

OK donc si je comprends bien, le soucis aurait été intégré avec #519.
@jpm-cbna, peux-tu jeter un œil au soucis et à ce fix, car c'est toi qui avait bossé sur cette PR ? Merci.

[jpm-cbna]

OK donc si je comprends bien, le soucis aurait été intégré avec #519. @jpm-cbna, peux-tu jeter un œil au soucis et à ce fix, car c'est toi qui avait bossé sur cette PR ? Merci.

@camillemonchicourt La suppression de l'id_observation suite à la PR #519 était bien voulu. Il me semblait que ce n'était pas utilisé. Du coup, pour gagner en performance, on stocke directement le résultat du nombre d'observations par année.

Si les id_observations sont nécessaires, la solution retenu ici de stocker les id_observation dans un champ de type tableau est la bonne... @juggler31 C'est à quel niveau que ces id_observation sont utilisés ?

[juggler31]

@jpm-cbna Oui les 'id_observation' sont utiles dans la home page. Lorsqu'on clique sur les espèces observées a droites de la carte, ça la popup au dessus de la bonne maille.

[jpm-cbna]

@jpm-cbna Oui les 'id_observation' sont utiles dans la home page. Lorsqu'on clique sur les espèces observées a droites de la carte, ça la popup au dessus de la bonne maille.

Ok, merci pour le retour. C'est effectivement une fonctionnalité qui n'est pas active sur nos installations: https://atlas.biodiversite-auvergne-rhone-alpes.fr/
J'avais fait une recherche globale dans le code mais j'ai du louper cette utilisation...

[juggler31]

Ça marche, merci pour le retour,

@camillemonchicourt est ce que ce serait possible de merger rapidement tu penses ?
On est un peu bloqué pour tester !