From 159206d52ce08d35ce903cd25fe54db745b479a0 Mon Sep 17 00:00:00 2001 From: Regis Houssin Date: Thu, 14 Jun 2012 09:12:52 +0200 Subject: [PATCH] Fix: $id and $rowid conflict --- htdocs/comm/mailing/cibles.php | 36 ++++++++++++++++++++-------------- 1 file changed, 21 insertions(+), 15 deletions(-) diff --git a/htdocs/comm/mailing/cibles.php b/htdocs/comm/mailing/cibles.php index df3b4725bf9..88c532f4791 100644 --- a/htdocs/comm/mailing/cibles.php +++ b/htdocs/comm/mailing/cibles.php @@ -48,7 +48,8 @@ $pagenext = $page + 1; if (! $sortorder) $sortorder="ASC"; if (! $sortfield) $sortfield="email"; -$id=GETPOST('rowid')?GETPOST('rowid'):GETPOST('id','int'); +$id=GETPOST('id','int'); +$rowid=GETPOST('rowid','int'); $action=GETPOST("action"); $search_nom=GETPOST("search_nom"); $search_prenom=GETPOST("search_prenom"); @@ -126,23 +127,28 @@ if ($action == 'clear') if ($action == 'delete') { // Ici, rowid indique le destinataire et id le mailing - $sql="DELETE FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE rowid=".$id; + $sql="DELETE FROM ".MAIN_DB_PREFIX."mailing_cibles WHERE rowid=".$rowid; $resql=$db->query($sql); if ($resql) - { //on récurpére l'id du mailing - $id = GETPOST('id','int'); - + { if (!empty($id)) { - $file = $dirmod."/modules_mailings.php"; - $classname = "MailingTargets"; - require_once($file); - - $obj = new $classname($db); - $obj->update_nb($id); - - Header("Location: ".$_SERVER['PHP_SELF']."?id=".$id); - exit; + foreach ($modulesdir as $dir) + { + $file = $modulesdir."/modules_mailings.php"; + + if (file_exists($file)) + { + $classname = "MailingTargets"; + require_once($file); + + $obj = new $classname($db); + $obj->update_nb($id); + + Header("Location: ".$_SERVER['PHP_SELF']."?id=".$id); + exit; + } + } } else { @@ -295,7 +301,7 @@ if ($object->fetch($id) >= 0) if ($object->statut == 0) { - print '
'; + print ''; print ''; }