2015-12-14 11 views
6

Vorrei eliminare una riga dalla tabella dei miei utenti quando l'utente fa clic su un pulsante, l'utente deve essere connesso a eliminare il proprio account.Eliminazione di righe dalla tabella degli utenti PHP MYSQL

ho echo'd il $ user_id che mostra '4', che è l'id corretto per l'utente connesso, in modo da user_id = $ user_id

Questa è la pagina che ho che detiene il pulsante che voglio eliminare la riga utenti nel database

<?php 
include_once 'dbconfig.php'; 
if(!$user->is_loggedin()) 
{ 
$user->redirect('index.php'); 
} 
$user_id = $_SESSION['user_session']; 

    if(isset($_POST['leave'])){ 
    $stmt = $DB_con->prepare("DELETE FROM users WHERE user_id = $user_id "); 
     $stmt->execute(); 
    } 
$stmt = $DB_con->prepare("SELECT * FROM users WHERE user_id=:user_id"); 
$stmt->execute(array(":user_id"=>$user_id)); 
$userRow=$stmt->fetch(PDO::FETCH_ASSOC); 
?> 

<head> 
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> 
<link rel="stylesheet" href="bootstrap/css/bootstrap.min.css" type="text/css" /> 
<link rel="stylesheet" href="style.css" type="text/css" /> 
<title>Welcome - <?php print($userRow['user_email']); ?></title> 
</head> 

<body> 

<div class="header"> 

    <div class="right"> 
    <label><a href="logout.php?logout=true"><i class="glyphicon glyphicon-log-out"></i> logout</a></label> 
    </div> 
</div> 
<div class="content"> 

Welcome <?php print($userRow['user_name']); ?> <br> 
<?php print($userRow['team_name']);?><br> 
Rank <?php print($userRow['user_rank']); ?> <br> 
<a href="players.php">Players</a> 
<a href="teams.php">Teams</a> 

<form action='teams.php' method='post'> 
<input type='submit' name='leave' value='Delete Profile'/> </form> 

<?php echo $user_id?> 

</div> 
</body> 
</html> 
+0

Perché non parametrizzare il più pericoloso della query; 'ELIMINA DAGLI utenti DOVE user_id = $ user_id'? – chris85

+0

Non sembri impostare $ _SESSION ['user_session'] ovunque, a meno che non manchi qualcosa. Inoltre, questa non è una produzione sicura. – markdwhite

+0

Ho impostato user_session su una pagina diversa, e sì è un po 'dappertutto, molto nuovo per php. – Dannad

risposta

5

credo che il problema è il vostro form action (teams.php) che riceverà il palo data.Your cancellare codice è sullo stesso file e logicamente $ _POST [ 'leave'] non sarà mai impostato in questa pagina.

Basta provare a rimuovere il tuo teams.php nell'attributo azione moduli.

<form action='' method='post'> 
<input type='submit' name='leave' value='Delete Profile'/> </form> 

o nel file teams.php aggiungere il codice di eliminazione

//Make sure you have started the session before using it 
$user_id = $_SESSION['user_session']; 

if(isset($_POST['leave'])){ 
    $stmt = $DB_con->prepare("DELETE FROM users WHERE user_id = $user_id "); 
     $stmt->execute(); 
} 

Un altro pezzo di consiglio è di query uso Parametrizzazione. Esempio:

if(isset($_POST['leave'])){ 
    $stmt = $DB_con->prepare("DELETE FROM users WHERE user_id = ? "); 
     $stmt-> bindParam(1,$user_id); 
     $stmt->execute(); 

} 
+0

Questo era il problema esatto, l'ho appena capito prima di leggere la tua risposta: P. Grazie mille per il tuo aiuto – Dannad

+0

Sono contento che tu lo risolvi. : D –

0
<?php $servername = "localhost"; 
$username = "username"; 
$password = "password"; 
$dbname = "myDB"; 

// Create connection 
$conn = new mysqli($servername, $username, $password, $dbname); 
// Check connection 
if ($conn->connect_error) { 
    die("Connection failed: " . $conn->connect_error); 
} ?>  

<a href="?id=<?php echo $id;?>" onclick="return confirm('Are you sure?')">Delete</a> 

    <?php if(isset($_GET['id'])){ 
     $user_id = $_SESSION['user_session']; 
     $id=$_GET['id']; 
     if($id==$user_id){ 
      $sql = "DELETE FROM Tablename WHERE id='$id'"; 
      if ($conn->query($sql) === TRUE) { 
       echo "Record deleted successfully"; 
      } else { 
       echo "Error deleting record: " . $conn->error; 
      } 

     } 
    }?> 
Problemi correlati