Primero explicaré algunos de los cambios que hice en su código.
- Las tildes de retroceso no son necesarias a menos que esté usando una palabra reservada, así que las eliminé
- Ya estás definiendo
$id
como$id = $_SESSION['memberID'];
así que cambié$stmt->bindParam(":id", $_SESSION['memberID'], PDO::PARAM_STR);
- Si está enlazando sus parámetros, no necesita ejecutar con una matriz, así que cambié
$stmt->execute(array(':email' => $_POST['email'], ':location' => $_POST['location'], ':id' => $id));
a$stmt->execute();
- La
action
en su formulario debe repetirse.
Este es el proceso resultante
<?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();
}
?>
Esta es la forma resultante (más fácil de leer con sangrías)
<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>