mailRe: [Galette-devel] bug 10550


Others Months | Index by Date | Thread Index
>>   [Date Prev] [Date Next] [Thread Prev] [Thread Next]

Header


Content

Posted by Johan Cwiklinski on March 06, 2008 - 15:29:

Bonjour,



On Thu, 06 Mar 2008 15:13:07 +0100, Aurélie <aurelie@xxxxxxxxxxxxxx>

wrote:

Aurélie a écrit :

2/ Pour l'erreur de type 'manque </p>'

Dans mon cas, j'ai deux champs input de type text. La fermeture du 

paragraphe est absente pour le premier.



vers les lignes 56-62



On a:

{if $field.field_pos != 1 || $field.field_repeat != 1}

     {if $is_form eq false}

             </tr>

     {else}

             </p>

     {/if}

{/if}



Il n'y a pas de {else}

J'ai modifié en:



{if $field.field_pos != 1 || $field.field_repeat != 1}

     {if $is_form eq false}

             </tr>

     {else}

             </p>

     {/if}

{else}

     </p>

{/if}



Et ça marche pour mon essai (deux champs input text), mais je n'ai pas 

testé si c'est généralisable. Faut-il un nouveau {if $is_form eq

false}?



Pour la version trunk du svn, j'avais viré ce test pour corriger le

problème. Je ne me souviens plus pourquoi, il faudra que je regarde cela

de plus près (du boulot je n'ai pas accès au svn, ce n'est pas possible

dans l'immédiat).





La réponse est oui (constat au moment où l'on valide le formulaire,

donc



dans voir_adherent.php), j'ai mis un nouveau fichier dans le bugtracker.



Mais du coup, ça n'a pas beaucoup de sens:



              {if $field.field_pos != 1 || $field.field_repeat != 1}

                      {if $is_form eq false}

                              </tr>

                      {else}

                              </p>

                      {/if}

              {else}

                      {if $is_form eq false}

                              </tr>

                      {else}

                              </p>

                      {/if}

              {/if}

Autant supprimer la condition...

Mais comme je ne sais pas à quoi ça sert...



La variable $is_form vaut true lorsque l'on affiche le formulaire

d'ajout/modification (mis en page par des fieldset et des p), false dans le

cas de l'affichage d'un compte utilisateur (mis en page par un tableau).



La condition sur field_pos et field_repeat est censée détecter les champs

répétables et fermer correctement la balise préalablement ouverte.



Je regarde de plus près ce week end probablement, merci pour les rapports

et patches en tous cas :-)





Aurélie



Johan




Related Messages


Powered by MHonArc, Updated Thu Mar 06 15:41:44 2008