Project

General

Profile

Anomalie #998

Emailing - Sauvegarde avant envoi

Added by Nicolas Béhier-Dévigne almost 3 years ago. Updated over 2 years ago.

Status:
Fermé
Priority:
Bas
Category:
Emails
Target version:
-
Start date:
01/10/2017
Due date:
% Done:

100%

Estimated time:
Version utilisée:
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.

Associated revisions

Revision f75f8a61 (diff)
Added by Johan Cwiklinski over 2 years ago

Fix variable; fixes #998

History

#1

Updated by Michaël DEVILLET almost 3 years ago

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

#2

Updated by Johan Cwiklinski over 2 years ago

  • Status changed from Nouveau to In Progress
  • Priority changed from Normal to 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.

#3

Updated by Johan Cwiklinski over 2 years ago

  • Category set to Emails
  • Assignee set to 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);

#4

Updated by Johan Cwiklinski over 2 years ago

  • Status changed from In Progress to Résolu
#5

Updated by Johan Cwiklinski over 2 years ago

  • % Done changed from 0 to 100
#6

Updated by Johan Cwiklinski over 2 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF