Anomalie #998: Emailing - Sauvegarde avant envoi - Galette - Galette bugs & features

Actions

Copier le lien

Anomalie #998

fermé

Emailing - Sauvegarde avant envoi

Ajouté par Nicolas Béhier-Dévigne il y a environ 8 ans. Mis à jour il y a plus de 7 ans.

Statut:

Fermé

Priorité:

Bas

Assigné à:

Johan Cwiklinski

Catégorie:

Emails

Version cible:

Début:

10/01/2017

Echéance:

% réalisé:

100%

Temps estimé:

Version utilisée:

0.8.3

Vote:

Description

Sur la page /mailing_adherents.php?mailing_new=true, le bouton Sauvegarder soumet le formulaire en POST à l'URL /mailing_adherents.php?mailing_new=true ce qui ne permet pas de sauvegarder un brouillon email.

Actions

Copier le lien

Mis à jour par Michaël DEVILLET il y a presque 8 ans

When i would like to save a mail thefollowing page after clicking "Save" is :

Internal Server Error

The server encountered an internal error or misconfiguration and was unable to complete your request.

Please contact the server administrator at [no address given] to inform them of the time this error occurred, and the actions you performed just before this error.

More information about this error may be available in the server error log.

The page adress is : http://************/galette/mailing_adherents.php#mail_preview

Actions

Copier le lien

Mis à jour par Johan Cwiklinski il y a presque 8 ans

Statut changé de Nouveau à In Progress
Priorité changé de Normal à Bas

I do not know what is the root cause ot this issue; but this is not related to the URL the form points to. I can reproduce the error with a "clean" URL.

This has already been fixed for the next Galette release (0.9); I'll try to provide a quick fix you can apply on your instance, but since there will not be any new 0.8 release; I'm not going to spend much time on this.

Actions

Copier le lien

Mis à jour par Johan Cwiklinski il y a presque 8 ans

Catégorie mis à Emails
Assigné à mis à Johan Cwiklinski

Le correctif :

diff --git a/galette/lib/Galette/Core/MailingHistory.php b/galette/lib/Galette/Core/MailingHistory.php
index 5446663..57f40ea 100644
--- a/galette/lib/Galette/Core/MailingHistory.php
+++ b/galette/lib/Galette/Core/MailingHistory.php
@@ -286,7 +286,7 @@ class MailingHistory extends History
                 'mailing_recipients' => serialize($_recipients),
                 'mailing_sent' => ($this->_sent) ?
                     true :
-                    ($this->zdb->isPostgres() ? 'false' : 0)
+                    ($zdb->isPostgres() ? 'false' : 0)
             );

             $update = $zdb->update(self::TABLE);
@@ -335,7 +335,7 @@ class MailingHistory extends History
                 'mailing_recipients' => serialize($_recipients),
                 'mailing_sent' => ($this->_sent) ?
                     true :
-                    ($this->zdb->isPostgres() ? 'false' : 0)
+                    ($zdb->isPostgres() ? 'false' : 0)
             );

             $insert = $zdb->insert(self::TABLE);