Eliminare una riga da una pagina php

Allora, io ho cercato di fare in modo che un utente, dopo che si registra, riceve una mail con la conferma della registrazione e un link per annullare la registrazione (in caso qualcuno usa un email altrui) con questo codice

<?php
$testo = "Ciao, " . $username . ", grazie per esserti registrato su ClashClan, il nome del tuo clan è " . $nome_clan . ", ed è di livello " . $livello_clan . ".

Le informazioni del tuo clan:
Nome= " . $nome_clan . "
Tag= " . $clanTag . "
Livello= " . $livello_clan . "
Membri= " . $membri_clan . "
Trofei Minimi Richiesti= " . $trofei_clan . "

Se non sei stato tu a registrarti con questa email ( " . $email . " ) vai su theclashclan.altervista.org/ErrorMail/?email=" . $email . "&tipo_clan=" . $tipo_clan;

$nome_mittente = "ClashClan";
$mittente_mail = "registrazioni@clashclan.it";
$mail_reply = "theclashclan@altervista.org";

$mail_headers = "From: " .  $nome_mittente . " <" .  $mittente_mail . ">\r\n";
$mail_headers .= "Reply-To: " .  $mail_reply . "\r\n";
$mail_headers .= "X-Mailer: Mail " . $nome_mittente;

mail($email, 'Registrato su ClashClan', $testo, $mail_headers);

?>

e il codice PHP della pagina ErrorMail/index.php è

<?
$tipo_clan=$_GET[tipo_clan];
$localhost='localhost';
$username='theclashclan';
$password='';
$db = mysql_connect($localhost,$username,$password);
$result = mysql_query("DELETE FROM `my_theclashclan`.`$tipo_clan` WHERE `$tipo_clan`.`email` = $email;",$db) ;
if (!$result) {
   echo "ERROR: " . mysql_error();
} 
?>
Grazie per la tua segnalazione, l'account e' stato eliminato.

Ma mi da l'errore: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1 .
Grazie a tutti.

[mzanella]

La variabile $email non è definita in ErrorMail/index.php.

Ora mi dice: Unknown column 'theclashclan@altervista.org' in 'where clause' .
Ma il campo non è l'indirizzo mail ma 'mail'

[mzanella]

Mostra il codice aggiornato.

<?
$tipo_clan=$_GET[tipo_clan];
$email=$_GET[email];
$localhost='localhost';
$username='theclashclan';
$password='';
$db = mysql_connect($localhost,$username,$password);
$result = mysql_query("DELETE FROM `my_theclashclan`.`$tipo_clan` WHERE `$tipo_clan`.`email` = `$email`",$db) ;
if (!$result) {
   echo "ERROR: " . mysql_error();
} 
?>
Grazie per la tua segnalazione, l'account e' stato eliminato.

[chack1172]

@theclashclan non hai selezionato alcun database
Usa questo codice:

<?
$tipo_clan=$_GET[tipo_clan];
$email=$_GET[email];
$localhost='localhost';
$username='theclashclan';
$password='';
$db = mysql_connect($localhost,$username,$password);
mysql_select_db("my_theclashclan", $db);
$result = mysql_query("DELETE FROM `my_theclashclan`.`$tipo_clan` WHERE `$tipo_clan`.`email` = `$email`",$db) ;
if (!$result) {
   echo "ERROR: " . mysql_error();
} 
?>
Grazie per la tua segnalazione, l'account e' stato eliminato.

mi da sempre lo stesso errore

[mzanella]

Probabilmente perché stai usando i backquotes ` per racchiudere una stringa (`$email` che dovrebbe essere '$email').

Inoltre non convalidi l'input in $_GET, al quale accedi con tipo_clan anziché 'tipo_clan' e usi il deprecato mysql_*.

Dunque:

<?php
if (!isset($_GET['tipo_clan'], $_GET['email'])) {
    echo "ERROR: Indicare tipo di clan ed email";
}

$db = mysqli_connect("localhost", "theclashclan", "", "my_theclashclan");
if (mysqli_connect_errno($mysqli)) {
    echo "Failed to connect to MySQL: " . mysqli_connect_error();
}

$email = mysqli_real_escape_string($db, $_GET['email']);
$table = mysqli_real_escape_string($db, $_GET['tipo_clan']);
$query = "DELETE FROM `$table` WHERE `email` = '$email'";
$result = mysqli_query($db, "SELECT 'Possible but bad style.' AS _msg FROM DUAL");
if ($result === false) {
    echo "Failed to run query: (" . $mysqli->errno . ") " . $mysqli->error;
} else {
    echo "Grazie per la tua segnalazione, l'account e' stato eliminato.";
}
?>

Nella pagina non mostra errori ma nel database resta la riga!!!???

[alemoppo]

In questa riga:

$query = "DELETE FROM `$table` WHERE `email` = '$email'"; 

Assegni a $query la stringa della query, ma non la esegui mai.

Ciao!

[mzanella]

Chiedo scusa, errore di copia-e-incolla dalla documentazione ufficiale.

$result = mysqli_query($db, $query); 

Quello me n'ero accorto e l'ho modificato ma lo stesso non mostra errori ma la riga resta

Scusate, ora funziona