Skip to content

Feat/autres objets fiches detail - #687

Open
IdrissaD wants to merge 4 commits into
PnX-SI:developfrom
naturalsolutions:feat/autres-objets-fiches-detail
Open

Feat/autres objets fiches detail#687
IdrissaD wants to merge 4 commits into
PnX-SI:developfrom
naturalsolutions:feat/autres-objets-fiches-detail

Conversation

@IdrissaD

@IdrissaD IdrissaD commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Fixes #547

Deux couches Sites et Groupes de sites sont ajoutées au gestionnaire de couches Leaflet des fiches détail des groupes de sites, sites, visites, observations... Elles sont décochées par défaut. Lorsqu'elles sont cochées, les autres entités sont affichées sur la carte et leur popup permet de passer de l'une à l'autre sans repasser par la page de liste. L'état de cochage de chacune des couches est conservé quand on change de fiche.

Un quatrième DisplayMode, info_hidden, déclare une couche dans le gestionnaire sans l'afficher. clearExisting et testLayerControlExists sont remplacés par un suivi des couches par nom. Le service garde donc un handle sur chaque couche info, sans ça une couche décochée ne serait jamais retirée.

Les couches « info » sont restreintes aux types de site du sous-module, via un ConfigService.moduleTypesSite() car la route geometries ne l'applique pas d'elle-même.

Deux défauts préexistants que cette fonctionnalité révèle :

onCancelEdit appelait _location.back(), ce qui renvoyait vers la dernière page affichée, qui peut maintenant être une autre fiche détail. Comme l'édition depuis une fiche n'est qu'un changement d'état, et pas de page, le retour arrière n'est gardé que là où il est pertinent : en création, et à l'arrivée en mode édition depuis la liste (icône crayon). Maintenant, annuler une édition d'une fiche détail garde l'utilisateur sur la même fiche détail.

navigateToParent ne traitait que le parent « module ». Après la suppression d'un site sans groupe, on atterrissait sur la liste des groupes, maintenant on arrive sur la liste des sites.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant