Per prima cosa ti spiego alcune delle modifiche che ho apportato al tuo codice.
- I back-tick non sono obbligatori a meno che tu non stia utilizzando una parola riservata, quindi li ho rimossi
- Stai già definendo
$id
come$id = $_SESSION['memberID'];
quindi ho cambiato$stmt->bindParam(":id", $_SESSION['memberID'], PDO::PARAM_STR);
- Se stai vincolando i tuoi parametri, non è necessario eseguire con un array, quindi ho cambiato
$stmt->execute(array(':email' => $_POST['email'], ':location' => $_POST['location'], ':id' => $id));
a$stmt->execute();
- L'
action
nel tuo modulo deve essere ripreso.
Questo è il processo risultante
<?php
if(isset($_POST['submit'])) {
$email = $_POST['email'];
$location = $_POST['location'];
$id = $_SESSION['memberID'];
$sql = "UPDATE members SET email=:email, location=:location WHERE memberID=:id";
$stmt = $db->prepare($sql);
$stmt->bindValue(":email", $email, PDO::PARAM_STR);
$stmt->bindValue(":location", $location, PDO::PARAM_STR);
$stmt->bindValue(":id", $id, PDO::PARAM_STR);
$stmt->execute();
}
?>
Questo è il modulo risultante (più facile da leggere con i rientri)
<form role="form" method="POST" action="<?php echo $_PHP_SELF ?>">
<div class="form-group">
<label class="control-label">Email</label>
<input type="text" value="<?php echo $_SESSION['email'] ?>" name="email" id="email" class="form-control"/>
</div>
<div class="form-group">
<label class="control-label">Location</label>
<input type="text" value="<?php echo $_SESSION['location'] ?>" name="location" id="location" class="form-control"/>
</div>
<div class="margiv-top-10">
<input type="submit" name="submit" class="btn green" value="Update" >
<a href="profile.html" class="btn default">Annuller </a>
</div>
</form>