[corrigé] OL ne demande plus de bbox

[sly]

Si je vais par exemple là :
http://dev.refuges.info/nav/11/massif/Beaufortain

L'appel envoyé est :
GET /exportations/massifs-gml.php?massif=11

Il manque le paramètre bbox

[Dominique]

Vu (dans ma version de dev de la lib aussi)
Reste à comprendre pourquoi

[sly]

Je viens de jouer à l'instant avec la commande "bisect" de git qui procède par dicotomie afin de trouver le commit responsable du bug (c'est marrant pour jouer ;-) et ça semble marcher)

Il me dit que c'est :

Code : Tout sélectionner

$ git bisect good
72c5311afe86d911a101e572df9a8d39eeb4f26c is the first bad commit
commit 72c5311afe86d911a101e572df9a8d39eeb4f26c
Author&#58; Dominique92 <protec@cavailhez.fr>
Date&#58;   Sat Mar 23 09&#58;50&#58;57 2013 -0700

    Correction du bug GMLSLD
    Simplification de GMLSLD
    Refonte du builder qui inclue dans la lib uniquement les classes des objets Openlayers et proj4js utilisées dans ../vues/*.js,

&#58;040000 040000 de917aa0a18f024ca4e02ddba24693f27963b661 5f9303ecb7228c966e7251e40d15d2dd5bdca295 M      ol2.12.4
&#58;040000 040000 0e2eb90d8cb7fcb4158547419dd46a14048f9593 d0a361176873bc533bd8c5401bb50ce098670b55 M      vues

Qui est responsable du bug

En version plus lisible, ça donne lui :
https://github.com/sletuffe/www.refuges ... 39eeb4f26c

Chouette plus que 1500 lignes à analyser et on trouve le coupable ;-(

[Dominique]

C'était aussi ma piste : le bug est apparu lors de la refonte du builder
Celui ci recompose maintenant automatiquement la liste des classes à incorporer en analysant le contenu de vues/*.js
Il doit manquer une classe.
Si c'est ça, ce sera simple (sauf qu'il faudra attendre demain soir )


Attention: c'est un bug bloquant pour le passage en prod

[Dominique]

Corrigé dans dev

Il manquait OpenLayers/Format/QueryStringFilter.js qui n'est pas déclaré en @requires mais signalé dans le commentaire
* if application uses the query string, for example, for BBOX parameters,
* OpenLayers/Format/QueryStringFilter.js should be included in the build config file

off course, mon builder ne lit pas les commentaires