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

[Johan Cwiklinski <johan@xxxxxxxx>]

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/