Re: [Galette-discussion] Bug plugin FullCard

2012-06-19 Par sujet Johan Cwiklinski
Salut,

Le 20/06/2012 01:54, Jacques Le Sausse a écrit :
> Bonjour Johan,
> 
> Je me suis apparemment mal fait comprendre.
> J'ai bien installé le plugin dans galette/plugins/*FullCard* comme il était
> indiqué dans la documentation pour la version 0.7
> 
> Les templates adh_actions.tpl, adh_fiche_action.tpl et headers.tpl
> indiquent bien href="{$galette_base_path}{*$fullcard_dir*}fullcard.php" et
> j'accéde bien au PDF d'une fiche adhérent (menu de droite)
>-> monsite.super-h.fr/galette/*plugins/FullCard*/fullcard.php?id_adh=1
> ==> Ok !
> 
> Par contre, dans le fichier menu.tpl le href est :
> href="{$galette_base_path}{*$galette_galette_fullcard_path*}fullcard.php"
> (path modifié "as is" dans commit du sam. mars 03 00:12:48 2012)
> Cette variable n'étant pas connue et donc pas interprétée par smarty, le
> chemin d'accès pour une fiche vierge (menu de gauche : Carte vide) donnait :
> -> monsite.super-h.fr/galette*/*fullcard.php"  => NOK !!!
> 
> C'est donc bien un autre bug que celui de l'issue #203 que je signalais.
> C'est d'ailleurs en allant voir les 3 autres templates que j'ai vu l'erreur
> et que j'ai pu la corriger dans le fichier menu.tpl.

Ha mais tu n'a pas la bonne version du plugin ! Le correctif de cette
coquille de chemins passe par une modification côté Galette aussi, que
tu n'as donc pas en 0.7.

En gros : la version de développement des plugins est basée sur la
version de développement de Galette. Mais ce n'est pas tout à fait vrai,
c'est plus compliqué que ça en fait.

J'ai d'ailleurs implémenté un système de vérification de la
compatibilité des plugins pour la prochaine version
(http://redmine.ulysses.fr/issues/241), qui évitera ce genre de soucis
dans le futur.

Je vais voir pour taguer dans le dépôt la dernière version compatible
avec Galette 0.7.
Normalement, dans menu.tpl, si tu remets « $galette_fullcard_path »
comme c'était avant le correctif, ça devrait focntionner correctement.

++
-- 
Johan





signature.asc
Description: OpenPGP digital signature
___
Galette-discussion mailing list
Galette-discussion@gna.org
https://mail.gna.org/listinfo/galette-discussion


Re: [Galette-discussion] Bug plugin FullCard

2012-06-19 Par sujet Jacques Le Sausse
Bonjour Johan,

Je me suis apparemment mal fait comprendre.
J'ai bien installé le plugin dans galette/plugins/*FullCard* comme il était
indiqué dans la documentation pour la version 0.7

Les templates adh_actions.tpl, adh_fiche_action.tpl et headers.tpl
indiquent bien href="{$galette_base_path}{*$fullcard_dir*}fullcard.php" et
j'accéde bien au PDF d'une fiche adhérent (menu de droite)
   -> monsite.super-h.fr/galette/*plugins/FullCard*/fullcard.php?id_adh=1
==> Ok !

Par contre, dans le fichier menu.tpl le href est :
href="{$galette_base_path}{*$galette_galette_fullcard_path*}fullcard.php"
(path modifié "as is" dans commit du sam. mars 03 00:12:48 2012)
Cette variable n'étant pas connue et donc pas interprétée par smarty, le
chemin d'accès pour une fiche vierge (menu de gauche : Carte vide) donnait :
-> monsite.super-h.fr/galette*/*fullcard.php"  => NOK !!!

C'est donc bien un autre bug que celui de l'issue #203 que je signalais.
C'est d'ailleurs en allant voir les 3 autres templates que j'ai vu l'erreur
et que j'ai pu la corriger dans le fichier menu.tpl.
@+


-- Message transféré --
> From: Johan Cwiklinski 
> To: galette-discussion@gna.org
> Cc:
> Date: Tue, 19 Jun 2012 20:22:53 +0200
> Subject: Re: [Galette-discussion] Bug et correction proposée dans plugin
> FullCard
> Salut,
>
> Le 19/06/2012 13:52, Jacques Le Sausse a écrit :
> > Bonjour,
> > Je vous signale une erreur détectée dans le plugin FullCard, au niveau du
> > template "default/menu.tpl" (Fiche adhérent/Carte vide)
> >
> > Les balises smarty dans le href ne renvoient pas le bon chemin.
> > Il faut remplacer le href par :
> > href="{$galette_base_path}{$fullcard_dir}fullcard.php"
>
> C'est un problème connu et corrigé dans la prochaine version de Galette,
> et dans les versions de développement des plugins.
>
> http://redmine.ulysses.fr/issues/203
>
> Comme stipulé dans le second commentaire du bogue, la solution de
> contournement la plus simple consiste à rennomer le dossier du plugin en
> FullCard.
>
___
Galette-discussion mailing list
Galette-discussion@gna.org
https://mail.gna.org/listinfo/galette-discussion