Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Composant carte, outil de dessin "rectangle" non fonctionnel #1802

Open
jpm-cbna opened this issue Mar 27, 2022 · 2 comments
Open

Composant carte, outil de dessin "rectangle" non fonctionnel #1802

jpm-cbna opened this issue Mar 27, 2022 · 2 comments
Labels

Comments

@jpm-cbna
Copy link
Contributor

Version
develop

Description du bug
Dans le module Synthese, sur la carte, l'outil de dessin "rectangle" fonctionne mal. Lorsqu'on sélectionne l'outil et que l'on commence à tracer un rectangle, celui-ci reste tout petit.
RAS pour polygone et cercle

A priori, le problème vient du passage à Angular 12 qui a activé le mode stricte de Javascript.
Le ticket #1005 sur Leaflet.draw indique que la PR #1002 résoud le problème.

En modifiant le code dans node_modules/dist/leaflet.draw-src.js puis en le minimifiant et collant le résultat dans node_modules/dist/leaflet.draw.js, le problème est bien réglé...
La question est de savoir comment récupérer cette correction pour GeoNature ?

Comportement attendue
Un rectangle devrait être dessiner sur la carte.

Comment reproduire
Installer la version "develop" de GeoNature, se rendre dans le module "Synthese" et essayer de dessiner un rectangle avec l'outil disponible sur la carte.

Logs
Extrait du fichier /var/log/geonature.log dans le cas d’une erreur 500.

@camillemonchicourt
Copy link
Member

Oui le soucis est que Leaflet.draw ne semble plus maintenu.
Il faudrait songer à la remplacer.
Peut-être par https://github.com/geoman-io/leaflet-geoman ?

@jpm-cbna
Copy link
Contributor Author

jpm-cbna commented Apr 6, 2022

Oui, leaflet-geoman a l'air d'avoir les mêmes fonctionnalités et elle est maintenu.
Par contre, pour la v2.10 de GN et résoudre le bug en question, je pencherai plutôt pour intégrer la bibliothèque leaflet.draw directement dans GeoNature comme l'a proposé @TheoLechemia . Nous corrigerons ainsi le bug (très simple) et pourrons continuer à l'utiliser le temps d'une migration vers leaflet-geoman.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants