Erreur JS - icon_options is not defined #50

Uncaught ReferenceError: icon_options is not defined

Déclenchée ici :
da5b8db7e1/javascript/leaflet.gis.js (L198)

C'est suite à da5b8db7e1 visiblement, si je reviens sur le commit précédent je n'ai plus l'erreur.

`Uncaught ReferenceError: icon_options is not defined` Déclenchée ici : https://git.spip.net/spip-contrib-extensions/gis/src/commit/da5b8db7e143d33ba3e65a55478aea6768672b00/javascript/leaflet.gis.js#L198 C'est suite à da5b8db7e143d33ba3e65a55478aea6768672b00 visiblement, si je reviens sur le commit précédent je n'ai plus l'erreur. @rastapopoulos

Il suffit de l'initialiser avec var icon_options = {}; au tout début de la fonction.

Il suffit de l'initialiser avec `var icon_options = {};` au tout début de la fonction.

Initialiser la variable dès le début devrait le faire...

diff --git a/javascript/leaflet.gis.js b/javascript/leaflet.gis.js
index cfdf865..ad6b604 100644
--- a/javascript/leaflet.gis.js
+++ b/javascript/leaflet.gis.js
@@ -158,6 +158,7 @@ L.Map.Gis = L.Map.extend({
 	* }
 	*/ 
 	setGeoJsonFeatureIcon: function (feature, layer) {
+		var icon_options = {};
 		// Marqueur en HTML plutôt qu'en icone si défini 
 		if (feature.properties && feature.properties.icon_html) {
 			icon_options = {

Initialiser la variable dès le début devrait le faire... ```diff diff --git a/javascript/leaflet.gis.js b/javascript/leaflet.gis.js index cfdf865..ad6b604 100644 --- a/javascript/leaflet.gis.js +++ b/javascript/leaflet.gis.js @@ -158,6 +158,7 @@ L.Map.Gis = L.Map.extend({ * } */ setGeoJsonFeatureIcon: function (feature, layer) { + var icon_options = {}; // Marqueur en HTML plutôt qu'en icone si défini if (feature.properties && feature.properties.icon_html) { icon_options = { ```

/o\ @rastapopoulos tu as mal copié le fix qu'on te proposait, là sans le var ta varible est globale et donc accessible directement depuis la console par exemple.

Pour être complètement honnête @b_b moi j'ai aussi reçu le mail de notif de ton message qui proposait le patch sans le var :p

Initialiser la variable dès le début devrait le faire...

diff --git a/javascript/leaflet.gis.js b/javascript/leaflet.gis.js
index cfdf865..ad6b604 100644
--- a/javascript/leaflet.gis.js
+++ b/javascript/leaflet.gis.js
@@ -158,6 +158,7 @@ L.Map.Gis = L.Map.extend({
   * }
  */
  setGeoJsonFeatureIcon: function (feature, layer) {
+         icon_options = {};
           // Marqueur en HTML plutôt qu'en icone si défini
          if (feature.properties && feature.properties.icon_html) {
                  icon_options = {

Visiblement la notif notifie le premier message, même si tu corriges aussitôt après... (et il y a aucune notification de modif).

Pour être complètement honnête @b_b moi j'ai aussi reçu le mail de notif de ton message qui proposait le patch sans le `var` :p ``` Initialiser la variable dès le début devrait le faire... diff --git a/javascript/leaflet.gis.js b/javascript/leaflet.gis.js index cfdf865..ad6b604 100644 --- a/javascript/leaflet.gis.js +++ b/javascript/leaflet.gis.js @@ -158,6 +158,7 @@ L.Map.Gis = L.Map.extend({ * } */ setGeoJsonFeatureIcon: function (feature, layer) { + icon_options = {}; // Marqueur en HTML plutôt qu'en icone si défini if (feature.properties && feature.properties.icon_html) { icon_options = { ``` Visiblement la notif notifie le premier message, même si tu corriges aussitôt après... (et il y a aucune notification de modif).

Dans ces cas il y a un "edited" dans l'entête des messages, qui permet d'accéder aux révisions d'un commentaire (et les notifs de modifs sont bien envoyées sur IRC).

Oui mais pour ça faut aller dans l'interface web… ce que je ne fais presque jamais, 90% du temps je lis les contenus dans mes emails et je ne vais sur la page que si j'ai à répondre (et pour Discuss j'y réponds donc même pour écrire je vais jamais dans l'interface web). Donc effectivement moi je n'avais eu que le contenu de l'email. :)

Oui mais pour ça faut aller dans l'interface web… ce que je ne fais presque jamais, 90% du temps je lis les contenus dans mes emails et je ne vais sur la page que si j'ai à répondre (et pour Discuss j'y réponds donc même pour écrire je vais *jamais* dans l'interface web). Donc effectivement moi je n'avais eu que le contenu de l'email. :)

Oui mais pour ça faut aller dans l'interface web… ce que je ne fais presque jamais

Perso je le fais toujours, justement pour ne pas me faire avoir par l'édition de commentaire après coup :)

> Oui mais pour ça faut aller dans l'interface web… ce que je ne fais presque jamais Perso je le fais toujours, justement pour ne pas me faire avoir par l'édition de commentaire après coup :)

Labels Milestones

Erreur JS - icon_options is not defined #50