Re: [Galette-devel] Rappel d'échéance adhérent enfant

[f.lassave@xxxxxxx]

Bonjour Johan,

----- Mail original -----
> De: "Johan Cwiklinski" <johan@xxxxxxxx>
> À: devel@xxxxxxxxxxxxxxxx
> Envoyé: Vendredi 14 Avril 2017 20:30:12
> Objet: Re: [Galette-devel] Rappel d'échéance adhérent enfant
> 
> Hello,
> 
> Le 02/04/2017 à 09:17, Johan Cwiklinski a écrit :
> > Salut,
> > 
> > Le 01/04/2017 à 00:34, f.lassave@xxxxxxx a écrit :
> >> Bonjour,
> >>
> >> Je n'avais pas encore testé la fonction de rappel d'échéance. Je
> >> rencontre un problème avec les adhérents "enfants" ... Lorsqu'un
> >> adhérent dépend d'un "parent", et ne possède donc pas d'adresse
> >> email, il m'est impossible d'envoyer un rappel d'échéance à
> >> l'adresse
> >> email de son "parent".
> >>
> >> En regardant le code, je n'arrive pas à voir où insérer la
> >> fonction
> >> "getEmail()" (qui retourne l'adresse du parent si l'adhérent n'a
> >> pas
> >> de mail et a un parent), de "Adherent.php" ?
> >>
> >> Pouvez vous me venir en aide, svp ?
> > 
> > Il faudrait tester ça :
> > https://github.com/galette/galette/compare/feature/parentmail-reminders
> > 
> > Attention : ça part de la branche develop, pas de la version stable
> > (0.9
> > donc, pas 0.8).
> 
> Bon. On prend les mêmes ; et on recommence :D
> 
> Je me suis rendu compte de pas mal de coquilles dans la proposition
> d'origine (si c'était simple, hein)...
> 
> Voici une nouvelle proposition :
> https://github.com/galette/galette/compare/feature/parentmail-reminders
> 

J'ai appliqué les modifications proposées sur les fichiers suivants :
 galette/lib/Galette/Entity/Reminder.php
 galette/lib/Galette/Repository/Members.php 
 galette/lib/Galette/Repository/Reminders.php 

> Par contre ; je n'ai aucune garantie que ça soit applicable en 0.8
> (et
> en toute honnêteté - c'est pas le but : la 0.9 a besoin d'un peu de
> tests aussi -- hey c'est la liste devel hein :D).
> 

J'ai fait les essais sur la version que j'ai sur mon serveur, soit la 0.8.2..3 ... 

> Dans ce qui a été modifié, normalement le nombre de rappels affichés
> par
> l'UI et le nombre de rappels effectivement envoyés doit être
> cohérent.
> 

Effectivement, l'UI indique : " En retard (prévisualiser) - 2 adhérents avec courriel 0 membres sans courriel", pour 1 enfant (sans mail) et son parent (avec mail) ! :)

Dans la table _reminders, on retrouve bien les deux envois ...
Et le parent reçoit bien deux mails de rappel (un pour lui et un pour son enfant) 
:)

J'ai aussi essayé avec un adhérent qui n'a pas de mail et dont son parent n'en a pas non plus :
l'UI indique : "  En retard (prévisualiser) - 2 adhérents avec courriel 1 membres sans courriel"

Et on envoie bien deux mails de rappels ... 

:D

> Il reste un "TODO", qui concerne l'affichage des membres concernés
> dans
> la liste des adhérents ; ça ne prend actuellement pas en compte le
> coup
> des parents (chaque chose en son temps -- une fois que les rappels
> seront OK, j'aviserai).
> 

Effectivement ... 
Pour le moment, ça ne m'a pas été gênant ... Mais c'est sûr que c'est mieux quand c'est cohérent ... ;)

Autre chose, la fonction " Génère des étiquettes pour les adhérents en retard sans adresse courriel" ne semble pas fonctionner chez moi ... :/
Ce n'est pas, je pense, une fonction que nous utiliserons, mais j'ai remarqué qu'elle ne fonctionnait pas en 0.8 ... 

Bien sûr, je n'ai pas testé toutes ces modifs en 0.9 ... j'en suis désolé ... Mais, hormis le " galette/includes/routes/members.routes.php " (qui n'est pas en 0.8), les modifs ont fonctionné en 0.8 ! Il me semble que ce devrait être de même en 0.9 !?...

Merci encore pour tout ce que tu fais ! ;)

@++
Fred

--
Galette development - http://galette.eu/

List documentation: https://listengine.tuxfamily.org/lists.galette.eu/devel/