https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342021-11-04T17:03:52ZRedmine Entr’ouvertCombo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3360202021-11-04T17:03:52ZValentin Deniaud
<ul><li><strong>Assigné à</strong> mis à <i>Valentin Deniaud</i></li></ul> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3360222021-11-04T17:06:00ZValentin Deniaud
<ul><li><strong>Fichier</strong> <a href="/attachments/59033">0001-maps-move-properties-from-layer-to-cell-57760.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/59033/0001-maps-move-properties-from-layer-to-cell-57760.patch">0001-maps-move-properties-from-layer-to-cell-57760.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>En espérant avoir bien compris le ticket.</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3360452021-11-05T07:27:41ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Il y a différentes choses, en fait les propriétés elles sont avant tout là pour déterminer ce qui apparaitra en popup. Donc ça serait plutôt à afficher uniquement selon la valeur posée dans marker_behaviour_onclick. Surtout ça doit quand même rester associé à la couche (on peut avoir une couche "parking voiture" et "parking vélo" avec des propriétés différentes à afficher), ça fait que ça devrait plutôt aller dans MapLayerOptions.</p>
<p>Aussi, c'est éventuellement à faire en rapport avec <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a>, en plus des propriétés définies ici qui déterminent ce qu'il y aura dans la popup, il y a marker_colour à également inclure,</p>
<pre>
- 'properties': [x.strip() for x in l.properties.split(',')],
+ 'properties': [x.strip() for x in self.properties.split(',')],
</pre>
<p>mais ça ne fera à mon avis que ramener vers <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a>, parce que le js il va juste afficher tout ce qui est dispo dans la popup.</p>
<p>Bref tu dois inclure ici marker_colour pour ne pas casser <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: carto, permettre de définir une propriété (du geojson) comme étant la couleur (Fermé)" href="https://dev.entrouvert.org/issues/57296">#57296</a> mais ça ne résoud pas <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a>.</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3367222021-11-09T11:23:42ZValentin Deniaud
<ul></ul><p>Frédéric Péters a écrit :</p>
<blockquote>
<p>ça fait que ça devrait plutôt aller dans MapLayerOptions.</p>
</blockquote>
<p>Yep je vais faire ça.</p>
<blockquote>
<p>Aussi, c'est éventuellement à faire en rapport avec <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a>, en plus des propriétés définies ici qui déterminent ce qu'il y aura dans la popup, il y a marker_colour à également inclure,</p>
<p>[...]</p>
<p>mais ça ne fera à mon avis que ramener vers <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a>, parce que le js il va juste afficher tout ce qui est dispo dans la popup.</p>
<p>Bref tu dois inclure ici marker_colour pour ne pas casser <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: carto, permettre de définir une propriété (du geojson) comme étant la couleur (Fermé)" href="https://dev.entrouvert.org/issues/57296">#57296</a> mais ça ne résoud pas <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a>.</p>
</blockquote>
<p>J'ai peu envie de toucher au js ici donc j'ai envie de laisser <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a> tranquille, par contre je n'ai pas envie de casser quelque chose : et là je ne comprends pas du tout ce qui casse ici, le diff que tu as cité me laisse perplexe, je dois manquer de compréhension du code ?</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3367292021-11-09T11:33:19ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>S'il y a une option "attributs à inclure dans la popup" et qu'il y a dedans comme valeur ["titre", "description"] et que ça fait que dans le geojson il y a comme propriétés "titre" et "description", le js qui s'attend à trouver aussi la propriété définie dans marker_colour ne va pas la trouver.</p>
<p>Si alors tu inclus dans le geojson les propriétés "titre" et "description" + la propriété "marker_colour", l'utilisation de celle-ci ne sera pas cassée.</p>
<p>Mais dans la popup il y aura l'info marker_colour également, pas juste "titre" et "description". <-- le statu quo, <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a>.</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3368522021-11-09T16:32:02ZValentin Deniaud
<ul><li><strong>Fichier</strong> <a href="/attachments/59123">0002-maps-duplicate-map-layer-options-57760.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/59123/0002-maps-duplicate-map-layer-options-57760.patch">0002-maps-duplicate-map-layer-options-57760.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/59124">0001-maps-move-properties-from-layer-to-cell-57760.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/59124/0001-maps-move-properties-from-layer-to-cell-57760.patch">0001-maps-move-properties-from-layer-to-cell-57760.patch</a> ajouté</li></ul><p>En fait dans<br /><pre>
- 'properties': [x.strip() for x in l.properties.split(',')],
+ 'properties': [x.strip() for x in self.properties.split(',')],
</pre><br />je ne vois vraiment pas ce qui n'est pas équivalent puisque les propriétés sont copiées d'un objet à l'autre par la migration. J'aimerais bien comprendre ce qui « casse » pour pouvoir écrire un test de non régression.</p>
<p>D'ailleurs à tester cette fonctionnalité en local, je n'arrive à rien casser avec mon (nouveau) patch.</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3368562021-11-09T16:43:26ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>L'option dans la carto c'était "propriétés à inclure" (générique, pas précis), ici ça devrait être "propriétés à afficher dans la popup" (via mon commentaire "Donc ça serait plutôt à afficher uniquement selon la valeur posée dans marker_behaviour_onclick.").</p>
<p>Copier les données lors de la migration ça va aller, puis quelqu'un verra "couleur" dans les "propriétés à afficher dans la popup" et ira la retirer et à ce moment le rendu des marqueurs sera cassé, parce que "couleur" était la propriété précisée dans "marker colour".</p>
<p>On a :</p>
<ul>
<li>1/ propriétés dans le geojson</li>
<li>2/ propriété "marker colour" </li>
<li>3/ propriétés affichées dans la popup</li>
</ul>
<p>Aujourd'hui on définit 1/, on pense à inclure la propriété du point 2 dedans et les propriétés du point 3, et on a tout ça qui s'affiche dans la popup 3/, qui n'est jamais définie explicitement.</p>
<p>Ici on va définir explicitement le point 3, mais dans le geojson il ne faudrait pas avoir exclusivement les propriétés du point 3, il faudra aussi celle du point 2.</p>
<p>et faire ça fera que dans la popup il y aura les propriétés de 2 et de 3 et c'est le bug et le statu quo <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a>.</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3369112021-11-10T09:47:24ZValentin Deniaud
<ul></ul><p>Frédéric Péters a écrit :</p>
<blockquote>
<p>L'option dans la carto c'était "propriétés à inclure" (générique, pas précis), ici ça devrait être "propriétés à afficher dans la popup"</p>
</blockquote>
<p>Dac</p>
<blockquote>
<p>(via mon commentaire "Donc ça serait plutôt à afficher uniquement selon la valeur posée dans marker_behaviour_onclick.").</p>
</blockquote>
<p>C'était compréhensible dans la première version du patch où le champ propriétés s'affichait dans la cellule, là je trouverais ça très peu clair d'avoir le bouton d'édition de la couche geojson qui apparaît/disparaît en fonction de la valeur de marker_behaviour_onclick. Et quand bien même on ferait ça il faudra trouver une autre idée le jour où il y aura d'autres choses éditables que les propriétés. Donc autant trouver l'idée maintenant : ça pourrait le faire de désactiver le champ si marker_behaviour_onclick est « Aucun », et dans ce cas modifier le help_text pour dire d'aller changer ça avant toute chose ?</p>
<blockquote>
<p>Copier les données lors de la migration ça va aller, puis quelqu'un verra "couleur" dans les "propriétés à afficher dans la popup" et ira la retirer et à ce moment le rendu des marqueurs sera cassé, parce que "couleur" était la propriété précisée dans "marker colour".</p>
</blockquote>
<p>OK c'est plus clair, j'aurais eu tendance à faire l'hypothèse que <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a> pourrait être traité assez vite après pour que ce genre de malheur n'arrive pas.</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3369172021-11-10T10:04:11ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Ah oui j'étais sur l'idée que le bouton d'édition était déjà tout le temps présente mais c'est juste là pour le paramètre opacité uniquement pour les couches "tuiles".</p>
<p>Donc si je comprends bien ta proposition elle est de tout le temps l'afficher, mais que dedans il y ait une note sur l'help_text du champ "propriétés pour la popup" rappelant à l'usager que le comportement actuel n'est pas d'afficher une popup. Ça m'irait de s'arrêter à ça et de laisser quand même le champ éditable.</p>
<blockquote>
<p>que <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: map: le champ MapLayer.properties sert à la fois de filtre pour le GeoJSON et le contenu des popu... (Fermé)" href="https://dev.entrouvert.org/issues/58072">#58072</a> pourrait être traité assez vite après</p>
</blockquote>
<p>oui c'était mon idée, tu auras vu maintenant un patch de Benjamin dedans, je vous laisse voir entre vous.</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3369532021-11-10T11:31:51ZValentin Deniaud
<ul><li><strong>Fichier</strong> <a href="/attachments/59143">1636542918.png</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/59143/1636542918.png">1636542918.png</a> ajouté</li></ul><p>Branche à jour avec ça.</p> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3370742021-11-12T08:24:05ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3374762021-11-16T16:43:57ZValentin Deniaud
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>commit efa733b2cf3256f8890955cefa9f1b52d347b6ee
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Tue Nov 9 17:17:42 2021 +0100
maps: duplicate map layer options (#57760)
commit 04a4ff60d9d63795fb1912418bd2c2af3bad28a9
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Thu Nov 4 16:32:47 2021 +0100
maps: move properties from layer to cell (#57760)</pre> Combo - Development #57760: Couche cartographique : enlever les "propriétés" pour les mettre dans la cellule cartehttps://dev.entrouvert.org/issues/57760?journal_id=3375002021-11-16T23:17:01ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>