Discussion :

[laurentSc]

Bonjour,
je souhaite connaître une information (l'ID d'un enregistrement), contenue dans la variable $_POST.
En effet, j'ai un formulaire qui permet à l'utilisateur de sélectionner un enregistrement à supprimer :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
<form method="post" name="del_or_update" action="admin-actu-form-maj-or-del">
 <table>
    <?php
 
    foreach ($Tab_Actus as $one_actu) {
        if ($i++ < 10){//on affiche que les 10 premières actualités
            echo "<tr><td>";
            echo nl2br(substr($one_actu['actu_text'], 0, 40)); //on affiche que les 40 premiers caractères
 
            echo '</td>
              <td><input type="radio" name="update_or_del" value="update/'.$one_actu["actu_id"].'" onChange="del_or_update.submit()"/></td>
              <td><label for="update">Modifier &nbsp;</label></td>
              <td><input type="radio" name="update_or_del" value="delete/'.$one_actu["actu_id"].'" onChange="del_or_update.submit()"/></td>
              <td><label for="delete">Supprimer</label></td>';
 
    echo "</tr>";
        }
    }
?>
 
    </table>
 
 </form>

Grâce à ce formulaire, on récupère dans la variable $_POST l'ID de l'enregistrement (dans $_POST['update_or_del'] via un explode), qui va être disponible dans le contrôleur ActuMajOrDel.php (l'action du formulaire). Dans ce contrôleur, si l'action demandée par l'utilisateur est une suppression, on redirige vers un autre contrôleur qui sera chargé de la suppression :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
if ($action[0] == "delete")
            header('Location:admin-actu-form-del');

Le nouveau contrôleur est donc ActuFormDel.php (la redirection est gérée par un routeur). J'espérais retrouver l'info de l'ID dans la variable $_POST, mais cette variable est vide (alors qu'elle ne l'était pas avant la redirection). Comment, donc, communiquer l'info de l'ID ?

[cavo789]

Pour la première partie de ta question (pourquoi POST est vide), si tu prends 5 secondes de réflexion nul doute tu le sauras.

Pourquoi rediriger vers un contrôleur ? C'est pas du tout comme ça qu'il faut faire. Zéro rédirection.

Tu as un contrôleur qui ensuite fait appel à des classes. Tu aurais une classe DeleteTruc que tu instancies avec new() et tu peux déjà passer l'ID dans ton constructeur p.ex.

Il est souvent fait mention de redirection dans ce forum, c'est quelque chose que je n'utilise jamais en php. C'est le job du contrôleur de faire le taf.

[cavo789]

Note aussi, on a déjà parlé du unique point d'entrée d'une application. Le fichier index.php (nom générique) est l'unique point d'entrée. Dès le moment où tu fais un redirect, ben, tu as donc d'autres points d'entrée... C'est donc quelque chose de "pas bien".

[laurentSc]

Bonsoir Christophe,

si tu prends 5 secondes de réflexion nul doute tu le sauras.

Raté car même après plus de 5 heures (bon, j'ai pas pensé à ça pendant 5 heures), je vois pas.

A moins que la réponse soit là :

Dès le moment où tu fais un redirect, ben, tu as donc d'autres points d'entrée..

Plus d'explication STP, car même si la raison est probablement là, je vois pas en quoi ça crée un autre point d'entrée vu que je redirige justement vers index.php...(donc vers le même fichier)

on a déjà parlé du unique point d'entrée d'une application.

C'est bien mon cas (et le point d'entrée unique porte bien le nom générique).

Tu aurais une classe DeleteTruc que tu instancies avec new() et tu peux déjà passer l'ID dans ton constructeur p.ex.

J'ai essayé de passer par le constructeur, mais ça passait rien.
Néanmoins, j'ai abandonné la redirection et ai instancié la classe (contrôleur)(et je transmets la donnée par paramètre (c'est un argument d'une méthode de la classe)) :

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 if ($action[0] == "delete") {
            $ref_ActuFormDel = new ActuFormDel();
            $ref_ActuFormDel->invoke([$actu_id]);
            // header('Location:admin-actu-form-del');
        }

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
class ActuFormDel
{
 public function invoke(array $url_data = []): void
{
...

[SpaceFrog]

Tu peux basculer ton $_POST dans une variable de session avant le location ...

[cavo789]

Bonjour

Ton fichier index.php fait un redirect vers ... lui-même ? Ne trouves-tu pas ça étrange ?

J'extrapole et je me dis que tu veux supprimer puis ré-afficher ton formulaire alors pourquoi pas un truc comme (en pseudo-code):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
new DeleteTruc($_POST['id']);
new ShowForm();

Je te le mets à la va-vite juste pour illustrer et parce que je ne connais pas ta situation mais voilà, tu supprimes ton record et tu ré-appelles la fonction de génération du formulaire.

Il me semble, à tout le moins d'expérience, qu'il n'y a aucune raison d'utiliser un redirect en PHP dès lors que tu as un code propre derrière avec des classes et des méthodes.

Note : si ton formulaire supprime des records puis se ré-affiche, peut-être une solution basée sur Ajax serait bien plus fluide. Plus compliquée à mettre en oeuvre toutefois.