Changeset 108049 in spip-zone


Ignore:
Timestamp:
Dec 14, 2017, 11:33:07 AM (19 months ago)
Author:
marcimat@…
Message:

Suite de r102612 : une erreur s’était subtilement glissée dans la requête SQL car il manquait la fin du order by.

Location:
_plugins_/mailshot/trunk
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • _plugins_/mailshot/trunk/inc/mailshot.php

    r102612 r108049  
    228228 * @param $email
    229229 */
    230 function mailshot_verifier_email_fail($email)
    231 {
    232     if (_MAILSHOT_DESABONNER_FAILED != false) {
    233         if (!defined('_MAILSHOT_MAX_FAIL'))
    234             define('_MAILSHOT_MAX_FAIL', 3);
    235 
    236         $historique = sql_allfetsel('date, statut, try', 'spip_mailshots_destinataires', 'statut!=' . sql_quote('todo') . ' AND email=' . sql_quote($email), '', 'date DESC', "0,$nb_check");
    237         $nb_failed = 0;
    238         foreach ($historique as $h) {
    239             if ($h['statut'] == 'fail' AND $h['try'] > 1) {
    240                 $nb_failed++;
    241             }
    242         }
    243         if ($nb_failed >= _MAILSHOT_MAX_FAIL) {
    244             $unsubscribe = charger_fonction("unsubscribe", "newsletter");
    245             $unsubscribe($email, array('notify' => false));
    246         }
    247     }
    248 }
     230function mailshot_verifier_email_fail($email) {
     231        if (_MAILSHOT_DESABONNER_FAILED != false) {
     232                if (!defined('_MAILSHOT_MAX_FAIL')) {
     233                        define('_MAILSHOT_MAX_FAIL', 3);
     234                }
     235
     236                $historique = sql_allfetsel(
     237                        'date, statut, try',
     238                        'spip_mailshots_destinataires',
     239                        'statut!=' . sql_quote('todo') . ' AND email=' . sql_quote($email),
     240                        '',
     241                        'date DESC',
     242                        "0," . _MAILSHOT_MAX_FAIL
     243                );
     244
     245                $nb_failed = 0;
     246                foreach ($historique as $h) {
     247                        if ($h['statut'] == 'fail' AND $h['try'] > 1) {
     248                                $nb_failed++;
     249                        }
     250                }
     251                if ($nb_failed >= _MAILSHOT_MAX_FAIL) {
     252                        $unsubscribe = charger_fonction("unsubscribe", "newsletter");
     253                        $unsubscribe($email, array('notify' => false));
     254                }
     255        }
     256}
     257
    249258/**
    250259 * Initialiser les destinataires d'un envoi
  • _plugins_/mailshot/trunk/mailshot_options.php

    r102612 r108049  
    66 */
    77
    8 if (!defined('_ECRIRE_INC_VERSION')) return;
    9 if (!defined('_MAILSHOT_DESABONNER_FAILED')) define('_MAILSHOT_DESABONNER_FAILED',true) ;
     8if (!defined('_ECRIRE_INC_VERSION')) {
     9        return;
     10}
     11if (!defined('_MAILSHOT_DESABONNER_FAILED')) {
     12        define('_MAILSHOT_DESABONNER_FAILED', true) ;
     13}
    1014
  • _plugins_/mailshot/trunk/paquet.xml

    r107051 r108049  
    22        prefix="mailshot"
    33        categorie="communication"
    4         version="1.22.3"
     4        version="1.22.4"
    55        etat="stable"
    66        compatibilite="[3.0.5;3.2.*]"
Note: See TracChangeset for help on using the changeset viewer.