par
or 1 » 15 mai 2022, 09:22
un appel ajax n'est pas la bonne solution. il faut ouvrir une nouvelle url comme c'est fait lorsque l'on met un lien. il est possible de conserver votre bouton et se compliquer la vie avec du javascript, mais en appelant une url au lieu de cet appel ajax.
dans le code qui génère le pdf, passer par du json est inutile, les données sont dans le tableau. comme il y a un limit 1 (qui est superflu si l'id est bien unique), un simple fetch est une meilleure idée qu'un fetchall. ainsi, l'on peut faire :
$post = get_Post_By_Id($idPost);
echo $post['titre'];
le code de connexion à la bdd ne devrait pas être dans la fonction get_Post_By_Id.
si l'on fait :
echo get_Post_By_Id(1)['titre'];
echo get_Post_By_Id(2)['titre'];
cela va exécuter 2 fois la connexion à la base de données, dont une fois inutilement. s'il y a ce code dans toutes les fonctions analogues, le jour où il y a quelque chose à modifier, il n'y a pas à faire le changement à un seul endroit.
un appel ajax n'est pas la bonne solution. il faut ouvrir une nouvelle url comme c'est fait lorsque l'on met un lien. il est possible de conserver votre bouton et se compliquer la vie avec du javascript, mais en appelant une url au lieu de cet appel ajax.
dans le code qui génère le pdf, passer par du json est inutile, les données sont dans le tableau. comme il y a un limit 1 (qui est superflu si l'id est bien unique), un simple fetch est une meilleure idée qu'un fetchall. ainsi, l'on peut faire :
$post = get_Post_By_Id($idPost);
echo $post['titre'];
le code de connexion à la bdd ne devrait pas être dans la fonction get_Post_By_Id.
si l'on fait :
echo get_Post_By_Id(1)['titre'];
echo get_Post_By_Id(2)['titre'];
cela va exécuter 2 fois la connexion à la base de données, dont une fois inutilement. s'il y a ce code dans toutes les fonctions analogues, le jour où il y a quelque chose à modifier, il n'y a pas à faire le changement à un seul endroit.