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.

[Pytet]

Bonjour,

Le web repose sur le protocole http :
- à la soumission du formulaire, le client envoi une requête http de type post avec les paramètres
- suite à la redirection, le client envoi une nouvelle requête http de type get, les paramètres post n'existe pas puisqu'il s'agit d'une nouvelle requête http

C'est une bonne pratique de faire une redirection get après une requête post, notamment pour éviter que le client renvoie la requête post en rafraichissant la page du navigateur après soumission du formulaire (pattern Post-Redirect-Get).
Néanmoins cette redirection devrait être réalisée après le traitement PHP (après la suppression).

[laurentSc]

Merci pour vos réponses. Celle de Pytet m'explique pourquoi $_POST est vide.

Je préfère une solution propre à la "bidouille" proposée par Spacefrog.

Si je redirige index.php vers index.php, je ne vais pas expliquer ; c'est complexe.

Même si j'avais une solution (que je retrouve pas...), je voudrais faire marcher celle avec le constructeur. Je me suis trompé vu que la data n'est pas transmise. Merci de me dire où :

Code php :

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

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

controller/admin/ActuFormDel.php :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
class ActuFormDel
{
    public $id;
 
    public function construct($id){
        $this->id = $id;
    }
 
     public function invoke(array $url_data = []): void
    {
        echo "<br/>ctrl admin actuformdel 18 id<br/>"; var_dump($this->id);echo "<br/>";//$this->id vaut NULL

[Pytet]

__construct à la place de construct

[laurentSc]

Merci ; j'avais perdu l'habitude...y a du mieux, mais :

Fatal error: Uncaught Error: Class "PasDePanique\controller\Admin\PasDePanique\page\Admin\ActuFormDel" not found in C:\projets\arnaudpoo\src\controller\Admin\ActuFormDel.php:20 Stack trace: #0 C:\projets\arnaudpoo\src\controller\Admin\ActuMajOrDel.php(33): PasDePanique\controller\Admin\ActuFormDel->invoke() #1 C:\projets\arnaudpoo\index.php(36): PasDePanique\controller\Admin\ActuMajOrDel->invoke(Array) #2 {main} thrown in C:\projets\arnaudpoo\src\controller\Admin\ActuFormDel.php on line 20

Pourtant, je passe bien un paramètre quand j'instancie :

Code php :

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

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

[Pytet]

Class "PasDePanique\controller\Admin\PasDePanique\page\Admin\ActuFormDel" not found

Il semble y avoir un problème sur la déclaration de tes espaces de nom, l'autoloader essaye de charger la classe PasDePanique\controller\Admin\PasDePanique\page\Admin\ActuFormDel au lieu de PasDePanique\controller\Admin\ActuFormDel

[laurentSc]

En effet, même pas vu...

Pas le temps tout de suite de continuer...Pas avant ce soir.

[cavo789]

Si je redirige index.php vers index.php, je ne vais pas expliquer ; c'est complexe.

Si c'est complexe alors c'est qu'il y a un souci de conception; désolé.

[laurentSc]

Il semble y avoir un problème sur la déclaration de tes espaces de nom

je vois bien qu'il y a un problème, mais pas moyen de voir lequel. J'ai fait 2 essais avec un message d'erreur identique (différent de celui du post #10) :

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
24
25
26
<?php
class ActuMajOrDel
{
    public function invoke(array $url_data = []): void
    {
        $page = new \PasDePanique\view\page\Admin\ActuMajOrDel();
        ...
        if ($action[0] == "delete") {
            $ref_ActuFormDel = new PasDePanique\controller\admin\ActuFormDel($actu_id);
        }
 
        /* Fatal error: Uncaught Error: Class "PasDePanique\controller\Admin\PasDePanique\controller\admin\ActuFormDel" not found in C:\projets\arnaudpoo\src\controller\Admin\ActuMajOrDel.php:32 Stack trace: #0 C:\projets\arnaudpoo\index.php(36): PasDePanique\controller\Admin\ActuMajOrDel->invoke(Array) #1 {main} thrown in C:\projets\arnaudpoo\src\controller\Admin\ActuMajOrDel.php on line 32
         */
 
////////////////////
    use PasDePanique\controller\admin\ActuFormDel;
 
        class ActuMajOrDel
        {
            public function invoke(array $url_data = []): void
            {
                $page = new \PasDePanique\view\page\Admin\ActuMajOrDel();
 
                if ($action[0] == "delete") {
                    $ref_ActuFormDel = new ActuFormDel($actu_id);
                }

controller/admin/ActuFormDel.php :

Code php :

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

1
2
3
4
5
6
7
8
9
<?php declare(strict_types=1);
 
namespace PasDePanique\controller\Admin;
 
class ActuFormDel
{
...]
        /* Fatal error: Uncaught Error: Class "PasDePanique\controller\Admin\PasDePanique\controller\admin\ActuFormDel" not found in C:\projets\arnaudpoo\src\controller\Admin\ActuMajOrDel.php:32 Stack trace: #0 C:\projets\arnaudpoo\index.php(36): PasDePanique\controller\Admin\ActuMajOrDel->invoke(Array) #1 {main} thrown in C:\projets\arnaudpoo\src\controller\Admin\ActuMajOrDel.php on line 32
        */

[Pytet]

Code :

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

$page = new \PasDePanique\view\page\Admin\ActuMajOrDel();

Sur cette ligne, pas d'ambiguité puisque tu indiques le nom complet de la classe (FCQN)

Code :

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

$ref_ActuFormDel = new PasDePanique\controller\admin\ActuFormDel($actu_id);

Ici tu indiques un nom de classe de manière relative (sans le \ initial). Si tu es dans l'espace de nom PasDePanique\controller\Admin, PHP va alors chercher la classe dans ce namespace, soit PasDePanique\controller\Admin\PasDePanique\controller\admin\ActuFormDel

Indique le nom complet de la classe ou fait un use pour importer la bonne classe (ce que tu sembles avoir fait à la ligne 16).

[laurentSc]

Si c'est complexe alors c'est qu'il y a un souci de conception; désolé.

Je dirais plutôt que j'utilise une conception merveilleuse de rawsrc mais qu'elle est trop complexe (pour moi) à décrire (elle utilise la POO et la structure MVC). Par contre, elle est super simple à utiliser (même moi en suis capable )

[laurentSc]

Grâce à ta réponse du post #15, Pytet, j'ai corrigé mon bug ($ref_ActuFormDel = new \PasDePanique\controller\admin\ActuFormDel($actu_id);)