1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33
| <?php
require "ConnectPDO.php";
echo "<select name='livre'>"; // Normalement, c'est plutôt comme ça : echo '<select name="livre">';
if(isset($_POST["idAuteur"]))
{
// Parenthèses autour de la requête inutiles
// Requête indentée (c'est plus facile à lire donc à déboguer)
// Aération du code de la requête (espace après la virgule et autour de = )
// Remplacement de $_POST["idAuteur"] par le paramètre nommé.
$sql = "
SELECT id, titre
FROM livre
WHERE idAuteur = :idAuteur
ORDER BY titre
";
$stmt = $bdd->prepare($sql);
$stmt->bindParam(':idAuteur', $_POST["idAuteur"], PDO::PARAM_INT); // Valorisation du paramètre nommé
$stmt->execute();
$result = $stmt->fetchAll();
foreach ($result as $row)
{
echo "<option value='".$row["id"]."'>".$row["titre"]."</option>";
}
}
echo "</select>";
// Une séparation de l'affichage et de la recherche des données serait mieux
?> |