RE: [Galette-devel] Galettenightly comportement création email

["alain paris" <alain.paris77@xxxxxxx>]

Bonjour, J'ai fait quelques essais rapides pas approfondis.

Le comportement correspond à cette galette avait anciennement. (URL de sorties) avec les 2 commits.

Note : le dernier commit
(https://github.com/galette/galette/pull/75/commits/3d03fa8779dbece88ab41d06c2df07d36d64f86c)
Je ne vois pas la nécessité du troisième commit (url page origine au lieu de l'Url mailings), cela montre qu'il y a eu une action (enregistrement, modification du mail). A la rigueur il serait presque normal qu'après un envoi, on soit dirigé également sur url mailings.
C'est sûr après cela impose un clic ou 2 de plus pour revenir sur la page url d'origine peut être que cela peut gêner certaines personnes (actions supplémentaires, rapidité ...).

Par contre j'ai donc trouvé une anomalie avec le plugin événement :(problème général avec les plugins ?)
Si je créé et j'enregistre un mail à partir de réservations(all) avec un destinataire A, et qu'ensuite je créer un mail à partir de la liste des adhérents(members) avec un destinataire B : dans envoi(mailing) la prévisualisation ou dans gestion des adhérents sélectionnés/membres sélectionnés j'ai toujours le destinataire A.
(Il n'y a pas prise en compte du choix des destinataires cochés sur la liste des adhérents(members). Il faut se déconnecter de galette pour revenir à un comportement normal pour la liste des adhérents.  
(J'ai même à un moment plus de destinataire du tout après avoir coché pourtant un adhérent dans members).


J'ai trouvé un autre comportement qui est antérieur à Galette 0.9.4:
Lorsque l'on sort d'envoi(mailing) sans utiliser aucun des boutons spécifiques à cette page (envoyer, enregistrer, annuler), lors de la création d'un mail à partir de la liste des membres(members), une pop-up "Envois existants" apparait avec 2 boutons "Reprendre" et "nouveau":
Si l'on appuis sur reprendre :
1 nous sommes sorti de mailing sans avoir pré-visionné auparavant le mail, alors les zones texte et sujet ne sont pas préremplis (non stockés ?).
2 nous avons prévisionné le mail avant de sortir de mailing, alors le texte et le sujet sont préremplis.

J'en ai mis encore une tartine !!
Bon week-end

Alain


-----Message d'origine-----
De : Johan Cwiklinski <johan@xxxxxxxx> 
Envoyé : samedi 24 octobre 2020 00:06
À : devel@xxxxxxxxxxxxxxxx
Objet : Re: [Galette-devel] Galettenightly comportement création email

Salut,

Le 23/10/2020 à 13:48, alain paris a écrit :
> [...]
> Sur la Galette v0.9.4-git-1819b30285 (2020-10-16 12:21:51 GMT+0200) 
> plus le même comportement au déroulement des actions:
> Lors de l'enregistrement d'un message je retombe sur la page Envoi 
> (mailing) au lieu de (gestion des envois) Envois (mailings) comme sur 
> les galettes précédentes.

J'ai regardé les différences dans le code entres les différentes version "incriminées" ; et j'ai pu corriger 2 soucis distincts :
- j'ai ajouté une redirection vers le mailing s'il n'est pas envoyé
(https://github.com/galette/galette/commit/c5b1dc7622b6ace78b2c5389bb7418b5299de9b7#diff-3d3b5c1744cc328c79b9cbf227322c7a1966e5c0153b23c92628955ac5d70351R376)
- mais du coup à l'enregistrement l'url était incorrecte,
- lorsque l'on rédige un courriel depuis un plugin, une URL de redirection est stockée ; il est prévu qu'on y retourne une fois le mailing "terminé" ; mais la valeur stockée n'était pas réinitialisée. Le coeur n'utilise pas cette possibilité de redirection (pas besoin), l'URL était donc figée (tout plugin officiel utilisant l'envoi de mailing poserait sûrement le même problème).

Pour le second point, je suis à peu près certain de mon coup ; pour le premier un peu moins (il n'est pas impossible que j'aie zappé un cas, il en existe que pas grand monde ne teste :D).

> Si cela peut aider.

Oui, merci beaucoup ; l'inspection des différences du code sur des périodes restreintes m'a permis de cibler mes investigations.

Les correctifs ont été versés sur la branche
(https://github.com/galette/galette/pull/75) .

Note : le dernier commit
(https://github.com/galette/galette/pull/75/commits/3d03fa8779dbece88ab41d06c2df07d36d64f86c)
modifie le comportement de la redirection à l'enregistrement, pour utiliser l'url de redirection également (comportement similaire entre envoyer/annuler et enregistrer).
Je ne sais pas trop si c'est mieux ou moins bien ; c'est fait à part pour pouvoir être viré ; dis moi ce que tu préfères ;-)

Bon week-end,
--
Johan

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

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


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

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