Newsletter : interdire l'inscription

[Invité]

Bienvenue, Invité. Veuillez vous connecter ou vous inscrire.

Connexion avec identifiant et mot de passe

[HollowMac]

Bonjour,

J'aimerai envoyer une newsletter, mais pour certains de mes membres seulement. J'aimerai donc interdire l'inscription à la newsletter, de sorte que c'est moi qui choisit d'inscrire les membres qui la recevront. J'ai alors décidé d'interdire l'inscription à la newsletter. Mais lorsqu'un nouveau membre s'enregistre, il a toujours la possibilité d'y souscrire (sauf que la case n'est pas cochée par défaut)

Une idée du pourquoi de ce bug ?

Sinon, pour les utilisateurs du module Fancy Newsletter, est-ce que ce que je veux faire marche avec ce module, car je ne suis pas certain que ça marchera avec le module Newsletter par défaut de PMx.

Cordialement,
Merci.

[stefvar]

Bonjour,

Edite le fichier inclludes/mx_userfunctions.php, cherche la ligne :

Code:

$out .= "<tr valign=\"top\"><td bgcolor=\"$bgcolor2\"><b>" . _NEWSLETTER . ":</b></td><td colspan=\"3\" bgcolor=\"$bgcolor3\"><input type=\"checkbox\" name=\"newsletter\" value=\"1\" " . $cnewsletter . " /> " . _RECEIVENEWSLETTER . "</td></tr>\n";

et remplace là par :

Code:

require(PMX_MODULES_DIR . DS . "Your_Account/config.php");
if($shownewsletter=='1'){
$out .= "<tr valign=\"top\"><td bgcolor=\"$bgcolor2\"><b>" . _NEWSLETTER . ":</b></td><td colspan=\"3\" bgcolor=\"$bgcolor3\"><input type=\"checkbox\" name=\"newsletter\" value=\"1\" " . $cnewsletter . " /> " . _RECEIVENEWSLETTER . "</td></tr>\n";
}

[HollowMac]

C'est génial !!!!! Merci beaucoup

@team PMx : Par contre, s'il s'agit bien d'un bug (Impossible d'interdire l'inscription à la Newsletter), il faudrait le corriger

[stefvar]

Re,

@team PMx : Par contre, s'il s'agit bien d'un bug (Impossible d'interdire l'inscription à la Newsletter), il faudrait le corriger

Cette modification corrige ce "bug" 

[Diabolo]

Slt,
Bonne remontée avec des précisions qui me sont personnelles:

Dans le cas présent, il faut imaginer plusieurs possibilités:

1- Le module newsleter est désactivé = il doit apparaitre nul part sur le compte user
2- Le module newsletter est activé mais la case  -Autoriser les utilisateurs à souscrire à la newsletter- est indiquée "non" (cf paramètres utilisateurs. Par conte je vois pas trop l'intérêt de mettre cette option si le module est activé (peut être pour l'envoi admin seulement )
3- Et finalement le module est activé et la case  -Autoriser les utilisateurs à souscrire à la newsletter- est indiquée "oui"

Donc pour le code a modifier dans includes/mx_userfunctions.php, j'aurais plutot mis:

Code:

 
        /* Newsletter oui/non */
        if (mxModuleAllowed('Newsletter') && !empty($GLOBALS['shownewsletter'])) {
         $out .= "<tr valign=\"top\"><td bgcolor=\"$bgcolor2\"><b>" . _NEWSLETTER . ":</b></td><td colspan=\"3\" bgcolor=\"$bgcolor3\"><input type=\"checkbox\" name=\"newsletter\" value=\"1\" " . $cnewsletter . " /> " . _RECEIVENEWSLETTER . "</td></tr>\n";
        }

Ainsi sans require du fichier complet du config mais juste la variable qui nous intéresse avec une verif empty ;-)


et aussi une modif dans l'accueil du compte qui permet cas 2, de ne pas voir la ligne newsletter

dans modules/unserinfo/view.php
on cherche

Code:

   // Newsletter
    if (mxModuleAllowed("Newsletter")) {
        $resultx = sql_query("SELECT status FROM ${prefix}_newsletter WHERE email='" . mxAddSlashesForSQL($uinfo['email']) . "' OR email='" . mxAddSlashesForSQL($uinfo['femail']) . "'");
        list($status) = sql_fetch_row($resultx);
        $uinfo['newsletter'] = (empty($status) || $status == 3) ? _NOTSUBSCRIBED : _SUBSCRIBED;
        $img = (empty($status) || $status == 3) ? "images/td.gif" : "images/tu.gif";
        $uinfo['newsletter'] = mxCreateImage($img, $uinfo['newsletter']) . "&nbsp;" . $uinfo['newsletter'] . "\n";
    }

a remplacer par

Code:

    // Newsletter
    if (mxModuleAllowed('Newsletter') && !empty($GLOBALS['shownewsletter'])) {
        $resultx = sql_query("SELECT status FROM ${prefix}_newsletter WHERE email='" . mxAddSlashesForSQL($uinfo['email']) . "' OR email='" . mxAddSlashesForSQL($uinfo['femail']) . "'");
        list($status) = sql_fetch_row($resultx);
        $uinfo['newsletter'] = (empty($status) || $status == 3) ? _NOTSUBSCRIBED : _SUBSCRIBED;
        $img = (empty($status) || $status == 3) ? "images/td.gif" : "images/tu.gif";
        $uinfo['newsletter'] = mxCreateImage($img, $uinfo['newsletter']) . "&nbsp;" . $uinfo['newsletter'] . "\n";
    }

Voilà c'est a réflechir et tester, sinon dans tous les cas je remonte le post dans le bugmanager de pmx pour correction

 

[HollowMac]

Merci pour le code

Dans mon cas, je cherche à envoyer la newsletter à certains membres que je choisis, et il ne faut donc pas qu'il puisse cocher de case "S'inscrire à la newsletter" (et donc  -Autoriser les utilisateurs à souscrire à la newsletter- est indiquée "non"), mais il faut que moi je peux via la Gestion des Utilisateurs.

[stefvar]

Re,

Intéressant le "$GLOBALS['shownewsletter']", je n'avais pas vu encore cette possibilité 

Par contre, pour le cas deux, je n'ai pas vu la nécessité de modifier le fichier view.php car si la newsletter n'est pas activé dans l'admin, la ligne n'apparait pas lorsque l'on visualise son compte.