Modificare codice Newsletter [NO AV]

[miki92]

Salve mi sapreste modificare questo codice affinchè un utente appena iscritto riceva un messaggio di benvenuto tramite mail semmai questo messaggio è scritto in file di testo? Inoltre al posto di $googbye e $welcome posso inserire un alert?

<?
error_reporting(0);
include("config.php");

function checkmail ($email)
{ 
    global $listfile;
    $fd = fopen ($listfile, "r");
    $contents = fread ($fd, filesize ($listfile));
    fclose ($fd);
    
    if (@stristr($contents,$email))
        $valid = "invalid";
    if (eregi("^[_\.0-9a-z-]+@([0-9a-z][-0-9a-z\.]+)\.([a-z]{2,3}$)", $email) && !isset($valid))
        $valid = "valid";
    else
        $valid = "invalid";
    
    return $valid;
}

if ($action == "subscribe" && isset($email))
{
    $valid = checkmail($email);
    if ($valid == "valid")
    {
        $fp = fopen($listfile, 'a+');
            flock($fp,2); 
            fwrite($fp,"".strtolower($email).","); 
        fclose($fp);
        header ("Location: ".$welcome."");
        exit;                 

    }
    else {
        header ("Location: ".$error."");
        exit;                 
    }
}    
elseif ($action == "unsubscribe" && isset($email))
{
    $fd = fopen ($listfile, "r");
    $contents = fread ($fd, filesize ($listfile));
    fclose ($fd);
    $allpeople = "";
    $entries = explode(",",$contents);
    for($i=0;$i<count($entries);$i++)
    {
        if (strtolower($email) == strtolower($entries[$i]))
            $found = "yes";
        elseif ($entries[$i] != "")
            $allpeople .= "".$entries[$i].",";
    }
    if ($found == "yes")
    {
        $fp = fopen($listfile, 'w'); 
            fwrite($fp,$allpeople); 
        fclose($fp);
    }
    header ("Location: ".$goodbye."");
    exit;                 
    
}
else
    header ("Location: ".$error."");
    exit;                 
?>

[funcool]

header ("Location: ".$goodbye."");
exit; 

Questo serve a fare il redirect verso la pagina che c'è come valore della variabile $goodbye (lo stesso vale per $error e $welcome) e a bloccare l'esecuzione dello script.
Se vuoi mettere un alert, basta che al posto di quelle righe metti:

echo "<script type=\"text/javascript\">
alert('messaggio');
</script>"; 

Per mandare l'e-mail puoi aggiungere la funzione mail prima di:

header ("Location: ".$welcome.""); 

[Ospite]

Per prima cosa,leva quell' error_reporting(0) all'inizio che nasconde gli errori.
POI lo rimetterai (se vorrai) una volta terminata la modifica dello script.
Metti dopo:

       $fp = fopen($listfile, 'a+');
            flock($fp,2); 
            fwrite($fp,"".strtolower($email).","); 
        fclose($fp); 

mail($email,"Messaggio di benvenuto",file_get_contents("messaggio_benvenuto.txt")); 

Il messaggio di benvenuto devi inserirlo nel file messaggio_benvenuto.txt

Per l'alert, non puoi metterlo nelle variabili indicate, perchè queste vengono utilizzate come argomento della funzione header() per il redirect.
Però potresti togliere quella funzione e inserire l'alert in output,magari con echo o print:

echo "<script type=\"text/javascript\">alert('benvenuto');</script>"; 

[dreadnaut]

no, $goodbye ed $error sono indirizzi di pagine a cui vieni redirezionato. Se ci vuoi mettere un messaggio, non lo metti nelle variabili. Togli la chiamata ad header() e cambi il programma.

Per inviare mail, prova a cercare sul forum, o a leggere una delle tante guide sull'invio mail che puoi trovare.

Inoltre forse dovresti leggerti qualche guida sul php, in modo da non dover fare domande come questa. Sebbene molto disponibili, gli utenti di questo forum non sono qua per scrivere programmi agli altri. Correggerli è già più probabile

[miki92]

Per prima cosa,leva quell' error_reporting(0) all'inizio che nasconde gli errori.
POI lo rimetterai (se vorrai) una volta terminata la modifica dello script.
Metti dopo:

       $fp = fopen($listfile, 'a+');
            flock($fp,2); 
            fwrite($fp,"".strtolower($email).","); 
        fclose($fp); 

mail($email,"Messaggio di benvenuto",file_get_contents("messaggio_benvenuto.txt")); 

Il messaggio di benvenuto devi inserirlo nel file messaggio_benvenuto.txt

Per l'alert, non puoi metterlo nelle variabili indicate, perchè queste vengono utilizzate come argomento della funzione header() per il redirect.
Però potresti togliere quella funzione e inserire l'alert in output,magari con echo o print:

echo "<script type=\"text/javascript\">alert('benvenuto');</script>"; 

Aspetta probid non ho capito, se non sbaglio il codice per la mail che hai messo tu invia a tutti gli iscritti la mail di benvenuto no?

[Ospite]

Aspetta probid non ho capito, se non sbaglio il codice per la mail che hai messo tu invia a tutti gli iscritti la mail di benvenuto no?

Quando si iscrive un nuovo utente, gli dovrebbe arrivare l'email.

[miki92]

Scusami ma non mi trovo, come verrebbe il codice? Cosi:

<? 
include("config.php");

function checkmail ($email) 
{  
    global $listfile; 
    $fd = fopen ($listfile, "r"); 
    $contents = fread ($fd, filesize ($listfile)); 
    fclose ($fd); 
     
    if (@stristr($contents,$email)) 
        $valid = "invalid"; 
    if (eregi("^[_\.0-9a-z-]+@([0-9a-z][-0-9a-z\.]+)\.([a-z]{2,3}$)", $email) && !isset($valid)) 
        $valid = "valid"; 
    else 
        $valid = "invalid"; 
     
    return $valid; 
} 

if ($action == "subscribe" && isset($email)) 
{ 
    $valid = checkmail($email); 
    if ($valid == "valid") 
    { 
        $fp = fopen($listfile, 'a+'); 
            flock($fp,2);  
            fwrite($fp,"".strtolower($email).",");  
        fclose($fp);
mail($email,"Messaggio di benvenuto",file_get_contents("messaggio_benvenuto.txt"));  

echo "<script type=\"text/javascript\">
alert('Benvenuto.');
</script>" 
        exit;                  

    } 
    else { 
        header ("Location: ".$error.""); 
        exit;                  
    } 
}     
elseif ($action == "unsubscribe" && isset($email)) 
{ 
    $fd = fopen ($listfile, "r"); 
    $contents = fread ($fd, filesize ($listfile)); 
    fclose ($fd); 
    $allpeople = ""; 
    $entries = explode(",",$contents); 
    for($i=0;$i<count($entries);$i++) 
    { 
        if (strtolower($email) == strtolower($entries[$i])) 
            $found = "yes"; 
        elseif ($entries[$i] != "") 
            $allpeople .= "".$entries[$i].","; 
    } 
    if ($found == "yes") 
    { 
        $fp = fopen($listfile, 'w');  
            fwrite($fp,$allpeople);  
        fclose($fp); 
    } 
echo "<script type=\"text/javascript\">
alert('Arrivederci.');
</script>"
    exit;                  
     
} 
else 
echo "<script type=\"text/javascript\">
alert('Errore.');
</script>"
    exit;                  
?>

[Ospite]

Sì.
Occhio però, che hai lasciato un punto è virgola per ben tre volte :)

echo "<script type=\"text/javascript\">
alert('Arrivederci.');
</script>"; 

Inoltre metti:

echo "<script type=\"text/javascript\">
alert('Errore.');
</script>"; 

anche al posto di:

header ("Location: ".$error.""); 

Ciao!

[miki92]

Sisi avevo notato...grazie a tutti! Funziona tutto per il meglio.