Erreur JQuery et nouvelle version 1.0

**tmtisfree** · 17/04/2010, 01h18

Bonjour,

Depuis la récente version 1.0, Firebug reporte le problème suivant : elem.body is null au chargement de mon appli web.

L'API est chargée de même que GeoportalExtended.js mais l'exécution s'arrête à la première option (container), donc pas d'affichage de carte.

La ligne concernée dans JQuery 1.4.2 est soulignée dans le fragment de JQuery suivant :

return ("scrollTo" in elem && elem.document) ? // does it walk and quack like a window?
// Everyone else use document.documentElement or document.body depending on Quirks vs Standards mode
elem.document.compatMode === "CSS1Compat" && elem.document.documentElement[ "client" + name ] ||
elem.document.body[ "client" + name ] :

// Get document width or height
(elem.nodeType === 9) ? // is it a document
// Either scroll[Width/Height] or offset[Width/Height], whichever is greater
Math.max(
elem.documentElement["client" + name],
elem.body["scroll" + name], elem.documentElement["scroll" + name],
elem.body["offset" + name], elem.documentElement["offset" + name]
) :

// Get or set width or height on the element
size === undefined ?
// Get width or height on the element
jQuery.css( elem, type ) :

// Set the width or height on the element (default to pixels if value is unitless)
this.css( type, typeof size === "string" ? size : size + "px" );

Une erreur similaire survient avec la version de JQuery 1.3.2.

Le déboggeur de Chrome renvoie l'erreur suivante à la même ligne de JQuery : Uncaught TypeError: Cannot read property 'scrollHeight' of null

L'erreur ne semble pas survenir avec la version Geoportal.js.

Il serait opportun que les changements de code fassent l'objet de versions spécifiques afin d'offrir au développeur un peu de stabilité.

**mga_geo** · 17/04/2010, 08h41

Je pense qu'il faut mieux avoir la version de l'API en local pour s'affranchir des changements de versions du site Geoportail.
Tant que les sources de données (cartes/photos/...) restent avec le même protocole (wms-c/...) et que la gestion de la clé reste identique, l'application continue de fonctionner.

**tmtisfree** · 17/04/2010, 15h09

Envoyé par mga_geo

Je pense qu'il faut mieux avoir la version de l'API en local pour s'affranchir des changements de versions du site Geoportail.
Tant que les sources de données (cartes/photos/...) restent avec le même protocole (wms-c/...) et que la gestion de la clé reste identique, l'application continue de fonctionner.

La même erreur apparaît en local avec la source 1.0 Geoportail-API-1.0-src.zip actuellement proposée. Comme elle n'est pas taggée avec une date, impossible de savoir si c'est la version initiale de mars ou la nouvelle.

Pour contourner le problème temporairement, j'ai rapatrié localement JQuery et commenté les 2 parties incriminées :

return ("scrollTo" in elem && elem.document) ? // does it walk and quack like a window?
// Everyone else use document.documentElement or document.body depending on Quirks vs Standards mode
elem.document.compatMode === "CSS1Compat" && elem.document.documentElement[ "client" + name ] ||
elem.document.body[ "client" + name ] :

// Get document width or height
(elem.nodeType === 9) ? // is it a document
// Either scroll[Width/Height] or offset[Width/Height], whichever is greater
Math.max(
elem.documentElement["client" + name],
/* elem.body["scroll" + name],*/ elem.documentElement["scroll" + name],
/* elem.body["offset" + name],*/ elem.documentElement["offset" + name]
) :

// Get or set width or height on the element
size === undefined ?
// Get width or height on the element
jQuery.css( elem, type ) :

// Set the width or height on the element (default to pixels if value is unitless)
this.css( type, typeof size === "string" ? size : size + "px" );

C'est un peu plus long à charger maintenant, mais au moins la carte apparaît.

**mga_geo** · 18/04/2010, 08h09

Les sources (https://api.ign.fr/geoportail/api/do...PI-1.0-src.zip) sont ceux de la version actuelle.
Un passage dans un 'packer' permet de gagner en temps de télé-chargement.

**dgrichard** · 18/04/2010, 17h18

Ce qui serait vraiment sympa serait d'avoir un code montrant comment tu utilises jQuery avec l'API.

Le seul vrai changement entre la version 1.0bêta5 que j'envoyais au forum et la version 1.0 est la façon dont l'API est chargée (via api?) ... et là, il est possible qu'il y ait un conflit entre les API.

Pour les changements de versions, je rejouterai la date de publication dans l'API.
Je conseille très largement (comme l'indique mga_geo) de développer :

* avec une version locale de l'API;
* de tester avec les différentes corrections de production en ne montant que les versions qui permettent à l'application de continuer à fonctionner.

Pour les versions API qui ne fonctionnent pas ou plus, la bonne pratique est :
* toujours récupérer le code source complet lors d'une publication;
* le déployer en local sans fusion/compression des javascripts;
* partager les erreurs sur le forum.

**tmtisfree** · 18/04/2010, 18h25

Envoyé par mga_geo

Les sources (https://api.ign.fr/geoportail/api/do...PI-1.0-src.zip) sont ceux de la version actuelle.

J'avais deviné.

Envoyé par mga_geo

Un passage dans un 'packer' permet de gagner en temps de télé-chargement.

J'ai "re-minifié" le JQuery modifié mais je suis obligé de le charger depuis mon propre serveur maintenant (de googleapis avant). Je n'utilise pas les versions 'packers' (trop de problèmes).

**tmtisfree** · 18/04/2010, 18h59

Envoyé par dgrichard

Ce qui serait vraiment sympa serait d'avoir un code montrant comment tu utilises jQuery avec l'API.

J'utilise JQuery pour la gestion de l'interface principalement, rien de bien transcendant.

Envoyé par dgrichard

Le seul vrai changement entre la version 1.0bêta5 que j'envoyais au forum et la version 1.0 est la façon dont l'API est chargée (via api?) ... et là, il est possible qu'il y ait un conflit entre les API.

Je n'ai pas testé la 1.0bêta5, j'ai tout re-codé de la bêta4 à la v1.0 de mars. Fin mars tout fonctionne correctement, je passe à autre chose. Le 15 avril je teste l'appli, ayant eu de nouvelles idées, et là erreur de JQuery. Donc le problème est survenu entre la version 1.0 de mars et la 1.0 actuelle. Maintenant il est aussi fort possible que ce soit un faux positif : j'ai conservé une version de mon appli datée du 3 mars qui passe sans problème. Je suis en train de réduire le code pour trouver le coupable mais c'est long (2000 lignes truffées de php).

Envoyé par dgrichard

Pour les changements de versions, je rejouterai la date de publication dans l'API.

C'est apprécié, merci. De même que les conseils.

**tmtisfree** · 18/04/2010, 19h47

Bonne nouvelle, après 3 minutes de deboggage intensif (!) l'erreur a disparu : le même code inchangé ne renvoie plus l'erreur. Incompréhensible mais inespéré. Jamais déboggé si vite...

**dgrichard** · 18/04/2010, 21h55

Envoyé par tmtisfree

Maintenant il est aussi fort possible que ce soit un faux positif : j'ai conservé une version de mon appli datée du 3 mars qui passe sans problème.

3 mars, c'était encore la 1.0bêta4 ...

D'ailleurs, tu peux faire le test suivant en mettant v=1.0beta4 au lieu de 1.0, tu auras la version juste avant la bascule en 1.0