problema con replace o update in php con variabile da funzione

<?php
//FUNZIONE
function mandamail($indmail, $pwutente) 
{
    $oksi = "si";
    $to = $indmail;
    $headers = "From: La Coccinella";
    // soggetto della mail
    $subject = "Credenziali d'accesso a www.lacoccinella.it";
    
    // corpo messaggio
    $body = "Mail generata automaticamente, si prega di NON rispondere.\n";
    
    $body .= "Per eventuali domande: xxxxx@xxx.it\n\n";
    
    $body .= "Credenziali d'accesso:\n";
    
    $body .= "\tNOME UTENTE: " . $indmail . "\n";
    $body .= "\tPASSWORD: " . $pwutente . "\n";
    $body .= "Buona giornata,\n\n\tLa Coccinella";
    
    $indmail=$row['mail'];
    $query = "UPDATE clienti SET invsino = 'si' WHERE mail = '$indmail'"; 
 
    $result = mysql_query($query);
    
    mail($to, $subject, $body, $headers,$message);
} 





$con=mysqli_connect("localhost","formcontatti","xxxxxxx","my_formcontatti");
// Check connection
if (mysqli_connect_errno())
  {
  echo "Failed to connect to MySQL: " . mysqli_connect_error();
  }

$result = mysqli_query($con,"SELECT * FROM clienti");


while($row = mysqli_fetch_array($result))
{
    if($row['invsino']=="no")
    {
        echo $row['mail'] . " | " . $row['object']. " | " . $row['invsino']. " | " . $row['passut'];
        echo "<input type=\"submit\" name=\"ciao\" value=\"INVIA\" onClick=\"".mandamail($row['mail'], $row['passut']) ."\" />"; 
        echo "<br>";
    }
}

mysqli_close($con);
?>

funziona tutto ma non mi cambia il campo invsino nel database
come mai secondo voi?

[alemoppo]

In questi casi, per prima cosa devi sempre guardare se e cosa dice la funzione mysql_error(). Comunque, le funzioni mysql_* sono deprecate. Andrebbero usate le mysqli_*. (comunque, anche se son deprecate, ancora dovrebbero funzionare).

Ciao!

In questi casi, per prima cosa devi sempre guardare se e cosa dice la funzione mysql_error(). Comunque, le funzioni mysql_* sono deprecate. Andrebbero usate le mysqli_*. (comunque, anche se son deprecate, ancora dovrebbero funzionare).

Ciao!

non riesco molto a raccapezzarmi, non è che potresti darmi qualche consiglio? anche su un'altra funzione in cui ho tentato l'update non ottengo risultati!

<?php
// PROVA CREAZIONE FILE PER GENERARE MAIL A CLIENTI GIA ISCRITTI

$con=mysqli_connect("localhost","***","*****","my_formcontatti");

$result = mysqli_query($con,"SELECT * FROM clienti WHERE LENGTH(passut) < 8");


while($row = mysqli_fetch_array($result))
{
        $password = '';
        
        $possibleChars = '0123456789'; 
        
        $i = 0; 
        $length = 8;
        while ($i < $length) 
        { 
          $char = substr($possibleChars, mt_rand(0, strlen($possibleChars)-1), 1);
          
          if (!strstr($password, $char)) 
          { 
            $password .= $char;
            $i++;
          }
        }

//questo update non mi funziona!
        $indmail=$row['mail'];
        $query = "UPDATE clienti SET passut = '$password' WHERE mail = '$indmail'"; 
        
}

mysqli_close($con);
?>

[alemoppo]

Nel primo codice, prima di invocare la mail(), controlla $result se è un errore, ad esempio:

if(!$result)
      echo 'Errore mysql:'.mysql_error(); 

In questo modo dovresti vedere cosa non va.

EDIT: adesso che guardo meglio, dove fai:

$indmail=$row['mail']; 

, $row non esiste dentro la funzione. $indmail esiste già invece, perché è il primo parametro della funzione.

Nel secondo caso, invece, metti la query dentro una stringa ma in realtà non la esegui mai!

Ciao!

Nel primo codice, prima di invocare la mail(), controlla $result se è un errore, ad esempio:

if(!$result)
      echo 'Errore mysql:'.mysql_error(); 

In questo modo dovresti vedere cosa non va.

EDIT: adesso che guardo meglio, dove fai:

$indmail=$row['mail']; 

, $row non esiste dentro la funzione. $indmail esiste già invece, perché è il primo parametro della funzione.

Nel secondo caso, invece, metti la query dentro una stringa ma in realtà non la esegui mai!

Ciao!

ho provato con l'error ma non mi da nessuna stringa quindi suppongo che il richiamo al database funzioni, infatti mi stampa la lista e mi fa inviare le mail quando clicco il pulsante, l'unica nota dolente è l'aggiornamento del campo invsino tramite UPDATE! come posso fare a far funzionare il controllo WHERE dell'UPDATE? passo anche la $row['mail'] come parametro della funzione?


EDIT: in pratica io vorrei fare in modo che scorra tutto l'elenco mail fino a trovare la corrispondenza, e in caso il pulsante INVIA venga schiacciato automaticamente lui imposti invsino = 'si' sulla riga corrispondente alla mail in questione!

[alemoppo]

Rileggi il mio "EDIT". Chiedi pure se non capisci.

Ciao!

Rileggi il mio "EDIT". Chiedi pure se non capisci.

Ciao!

ho provato levando la riassegnazione di row a $indmail ma comunque non mi modifica il campo invsino nel database, ti posto il codice modificato e commentato per maggior chiarezza

se per te è più comodo lavorare via PM scrivimi pure li!

grazie!

<?php
//FUNZIONE CHE AL CLICK DI INVIO MANDA LA MAIL ALL'INDIRIZZO 
//E DEVE INSERIRE 'si' ALL'INTERNO DEL CAMPO invsino DEL DATABASE

function mandamail($indmail, $pwutente) 
{
    $oksi = "si";
    $to = $indmail;
    $headers = "From: La Coccinella";
    // soggetto della mail
    $subject = "Credenziali d'accesso a www.lacoccinella.it";
    // corpo messaggio
    $body = "Mail generata automaticamente, si prega di NON rispondere.\n";
    $body .= "Per eventuali domande: riccardo@lacoccinella.it\n\n";
    $body .= "Credenziali d'accesso:\n";
    $body .= "\tNOME UTENTE: " . $indmail . "\n";
    $body .= "\tPASSWORD: " . $pwutente . "\n";
    $body .= "Buona giornata,\n\n\tLa Coccinella";
    
//QUERY PER CAMBIARE invsino IN 'si'
    $query = "UPDATE clienti SET invsino = 'si' WHERE mail = '$indmail'"; 
     $result = mysql_query($query);
    mail($to, $subject, $body, $headers,$message);
} 





$con=mysqli_connect("localhost","formcontatti","__________","my_formcontatti");
// CHECK CONNESSIONE
if (mysqli_connect_errno())
{
      echo "Failed to connect to MySQL: " . mysqli_connect_error();
}
//QUERY CHE SELEZIONA TUTTO DA DB PER VISUALIZZARE ELENCO NOMINATIVI
$result = mysqli_query($con,"SELECT * FROM clienti");
//CHECK
if(!$result)
      echo 'Errore mysql:'.mysql_error();  
//CICLO CHE VISUALIZZA ELENCO COMPLETO DEI NOMINATIVI CON PULSANTE INVIA A FIANCO DI OGNUNO
while($row = mysqli_fetch_array($result))
{
    if($row['invsino']=="no")
    {
        echo $row['mail'] . " | " . $row['object']. " | " . $row['invsino']. " | " . $row['passut'];
        echo "<input type=\"submit\" name=\"ciao\" value=\"INVIA\" onClick=\"".mandamail($row['mail'], $row['passut']) ."\" />"; 
        echo "<br>";
    }
}
//CHIUSURA DATABASE
mysqli_close($con);
?>

[alemoppo]

Mi è più comodo qui nel forum, anche perché così possono rispondere altri in caso io non ci fossi.
Comunque, fai stampare con una echo la variabile $query. Quindi vai su phpMyAdmin -> SQL e prova se funziona.

In ogni caso, aggiungi anche mysql_error() perché avere messaggi di errore se ci son degli errori, aiuta a risolvere i problemi. In generale, anche a inizio file un

error_reporting(E_ALL);

farebbe comodo per eventuali warning di PHP.

Ciao!

Comunque, fai stampare con una echo la variabile $query. Quindi vai su phpMyAdmin -> SQL e prova se funziona.

sostituisco $indmail con una delle mail presente del db e provo subito!


query stampata a video da echo:
UPDATE clienti SET invsino = 'si' WHERE mail = 'blabla@gmail.com'
cosi facendo mi inseriva effettivamente si nel campo invsino, e ogni riga aveva l'indirizzo mail corrispondente!

ho inserito i codici error che dicevi ma niente! come diavolo può non funzionare che la query è giusta >.<

EDIT: ho notato però che mettendo echo "\n".$query; nella funzione, viene stampato anche senza pressione del bottone che richiama la funzione, strano D: