Galette

Je ne sais pas trop quel est le bug ici à vrai dire... :p

J'avais prévu initialement que tout mail qui servirait de modèle récupèrerai l'ensemble des propriétés du modèles, incluant les pièces jointes. Du coup, que faire ? Conserver mon idée initiale, ou faire en sorte qu'un mailing à partir d'un modèle ne tienne pas compte des pièces jointes ?

Attention : cela vaut uniquement pour les mailings déjà envoyés. En 0.7.7, les mailings qui n'ont pas été envoyés et qui sont enregistrés dans l'historique ne donnent pas lieu à un nouveau mailing, et ils récupèreront donc dans tous les cas l'ensemble des propriétés (pièces-jointes inclues).

Johan Cwiklinski a écrit :

Je ne sais pas trop quel est le bug ici à vrai dire... :p

Un dessin vaut mieux qu'un long discours ;-)
Je veux utiliser comme modèle un courriel déjà diffusé avec une pièce jointe. Cette pièce jointe apparaît comme déjà jointe (flèche rouge) mais je n'en veux pas.

Donc je clique sur la croix rouge pour virer cette pièce jointe du courriel que je veux envoyer et paf ! j'ai l'anomalie décrite ci-dessus (= page blanche ...). Car en fait, cette pièce jointe ne l'est pas. Pour preuve, si je laisse cette pj quand j'envoie le nouveau courriel, celui-ci est diffusé ... sans pj !

J'avais prévu initialement que tout mail qui servirait de modèle récupèrerai l'ensemble des propriétés du modèles, incluant les pièces jointes. Du coup, que faire ? Conserver mon idée initiale, ou faire en sorte qu'un mailing à partir d'un modèle ne tienne pas compte des pièces jointes ?

Je ne discute pas du tout le principe de conserver les pièces jointes dans la copie du courriel déjà expédié !

Attention : cela vaut uniquement pour les mailings déjà envoyés. En 0.7.7, les mailings qui n'ont pas été envoyés et qui sont enregistrés dans l'historique ne donnent pas lieu à un nouveau mailing, et ils récupèreront donc dans tous les cas l'ensemble des propriétés (pièces-jointes inclues).

C'est clair et c'est parfait ;-)

Ai-je été plus clair ?

L'anomalie est en effet corrigée ; en revanche, le nom du fichier n'apparaît plus dans la consultation de l'e-mailing d'origine (après que ladite pj ait été détruite dans la copie d'e-mailing, je ne sais pas s'il y a un lien de cause à effet). La création de copies d'e-mailing ayant une pj mais qui ne sont ni sauvegardées, ni expédiées génère dans le répertoire attachments des sous-répertoires qui y restent sine die semble-t-il :

Johan Cwiklinski a écrit :

Roland Telle a écrit :

L'anomalie est en effet corrigée ; en revanche, le nom du fichier n'apparaît plus dans la consultation de l'e-mailing d'origine (après que ladite pj ait été détruite dans la copie d'e-mailing, je ne sais pas s'il y a un lien de cause à effet).

Hum... Il y a des chances que ce soit relaltif aux récentes modifications que j'ai apportées.

Corrigé dans 3eed586ee001b458839b35bf6feaf09d84bca8a9

Tout fonctionne parfaitement. Merci.

Roland Telle a écrit :

Tout fonctionne parfaitement. Merci.

... hormis un effet collatéral non désiré : lors de la création d'un e-mailing, le nom du fichier joint apparaît deux fois :

Je pense que ça devrait corriger c problème, en ajoutant pas d'effets de bord :

diff --git a/galette/lib/Galette/Core/Mailing.php b/galette/lib/Galette/Core/Mailing.php
index 0e4a5d3..5dfb5fd 100644
--- a/galette/lib/Galette/Core/Mailing.php
+++ b/galette/lib/Galette/Core/Mailing.php
@@ -163,7 +163,9 @@ class Mailing extends GaletteMail
         } else {
             $this->_tmp_path = null;
             $this->_id = $rs->mailing_id;
-            $this->_loadAttachments();
+            if ( !$this->attachments ) {
+                $this->_loadAttachments();
+            }
             $this->_history_id = $rs->mailing_id;
         }
     }

Pourrais-tu tester et me confirmer que c'est OK ?

Johan Cwiklinski a écrit :

Je pense que ça devrait corriger c problème, en ajoutant pas d'effets de bord : [...]
Pourrais-tu tester et me confirmer que c'est OK ?

C'est Ok ! Merci.