Projet

Général

Profil

Development #42765

carto, passer les paramètres de la couche dans la cellule plutôt qu'au niveau du geojson

Ajouté par Frédéric Péters il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
12 mai 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Aujourd'hui on ajoute au geojson une propriété layer avec les infos de la couche (slug, couleurs, icône, etc.); maintenant qu'on a un geojson par couche, on pourrait plutôt passer cette info au niveau de la cellule, plutôt qu'avoir à la dupliquer pour tous les marqueurs.


Fichiers

Révisions associées

Révision 179f77f4 (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

maps: give common layer properties to map cell (#42765)

Révision 6d332a9c (diff)
Ajouté par Frédéric Péters il y a presque 4 ans

tests: update test_download_geojson after #42765

Historique

#1

Mis à jour par Frédéric Péters il y a presque 4 ans

À noter le passage :

-                      $(cell).trigger('combo:map-feature-prepare', {'feature': feature, 'layer': layer});
+                      $(cell).trigger('combo:map-feature-prepare',
+                              {'feature': feature, 'layer': layer, 'geojson_layer': geojson_layer});

C'est pour Toodego, qui exploitait de sa manière les infos passées dans properties.layer, et qui n'y aurait plus accès sans ça.

Le reste est plutôt basique, et simplifie/nettoie à la suite de #42761, en utilisant le filtre |as_json.

#2

Mis à jour par Christophe Siraut il y a presque 4 ans

  • Statut changé de Solution proposée à Solution validée
#3

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 179f77f4bc0f81d64fed59e9020c505b9f67dcf3
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue May 12 13:37:41 2020 +0200

    maps: give common layer properties to map cell (#42765)
#4

Mis à jour par Frédéric Péters il y a presque 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF