[issue 323] Modification pour le problème showintoc #325
Conversation
Est-ce que tu peux préciser où tu as vu ces caractères ? Afin de faciliter l'interprétation, l'analyse du problème il nous faut en général plus de détail dans ce type d'issues. |
lib/mv.js
Outdated
| @@ -1625,7 +1625,7 @@ var mv = (function () { | |||
| sld: $(l).attr("sld"), | |||
| legendurl: $(l).attr("legendurl"), | |||
| attributefilter: $(l).attr("attributefilter") === "true", | |||
| showintoc: $(l).attr("showintoc") === "true", | |||
There was a problem hiding this comment.
cf file change. En enlevant le == "true" ça passe
There was a problem hiding this comment.
pourquoi il y avait des === true
Les === "true" permettent de savoir quelle est la valeur par défaut du paramètre dans le XML.
Dans le XML, on aura showintoc="true" et donc c'est comme ca qu'on va transformer un string en booléen.
Après il faut voir dans le reste du code si c'est le string "true" qui est testé ou le booléen true car avec ce test, on attend du boolean.
There was a problem hiding this comment.
Alors il semble y avoir un léger effet de bord :
- je fais une appli avec une couche en shwointoc=true et une couche en shwointoc=false
- la preview fonctionne parfaitement
- je publie l'application
- la visualisation fonctionne comme attendue
- J'ai juste un souci : au niveau de la checkbox, elle revient à la valeur par défaut
Je comprends pas ce qui se passe à la publication qui entraîne ces changements ?
|
Là on est bien au niveau des tests |
|
Je viens de tester à froid ça fonctionne. |
Issue #323
J'ai testé, pas vu d'effet de bord. par contre, j'aimerai bien savoir pourquoi il y avait des === true. Merci