Discussion :

[oliviercuyp]

Bonjour tout le monde,

Je me demande comment fonctionne le controller helper '_forward'.
Voici mon soucis:
J'ai une action (monAction) qui qui contient une form.
La form envoi son post sur un autre action du même controller (formAction).
Dans cette dernière, je vérifie le post et en cas de succès je redirige sur une dernière action(succesAction) sinon je forward sur monAction pour réafficher le formulaire.
Le problème c'est qu'après le forward le formulaire est réinitialisé à alors que c'est un attribut de mon controller. Ce qui sous entend que le forward ré-instancie le controller ...

Un peu de code pour illustrer ceci :

Code :

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
27
28
29
30
31
32
33
34
35
<?php
class monController extends Zend_Controller_Action {
  $_form;
 
/**
* Returns the comment form
*/
protected function _getForm() {
  if(null === $this->_form) {
    $this->_form = new My_Form();
  }
 
  return $this->_form;
}
 
public function monAction() {
	$this->form = $this->_getForm();
}
 
public function formAction() {
	$form = $this->_getForm();
 
	if($this->getRequest()->isPost() && $form->isValid($this->getRequest()->getPost())) { // IsValid rempli la form
		$this->_redirect('http://monsite/mon/succes');
	}
	else {
		$this->view->message('Données incorrectes');
	}
	$this->_forward('mon');
}
 
public function succesAction() {
	$this->view->message('Données enregistrées');
}
}

Avec ce code, mon message d'erreur s'affiche bien mais la form est vide alors qu'elle devrait contenir ce qui a été posté ...

[Nighty]

Je vais peut être dire une énormité, étant donné que j'utilise très peu de forwarding entre mes actions mais, l'helper FlashMessenger ne sert-il justement pas à faire passer des messages d'une action à l'autre ?

[GrandFather]

Bonjour,

Le problème c'est qu'après le forward le formulaire est réinitialisé à alors que c'est un attribut de mon controller. Ce qui sous entend que le forward ré-instancie le controller ...

En fait, ta variable $form dans la méthode formAction n'est qu'une variable locale, détruite dès la sortie de la méthode. Il faut l'affecter à $this->form pour que ça fonctionne. D'autre part, le _getForm() devrait juste instancier un My_Form et pas l'affecter à $this->_form (quelle différence d'ailleurs avec $this->form ?), en principe une méthode getXXX n'est pas censée avoir d'effets de bord. Enfin, je ne saisis pas l'intérêt du _forward() dans ce cas de figure.

[oliviercuyp]

@Nighty

Le code ci-dessus était juste pour illustrer le problème sur le forward

@GrandFather

Il me semblait bien que ça devait être un soucis du genre.
Je pensais que qu'en PHP les objets était toujours passé par référence.

Le but du _getForm() était de vérifier si il y avait déjà une instance crée (et donc initilisée avec des données) pour $this->_form et de ma la renvoyer ou de l'instancier si elle n'existait pas et puis de me renvoyer l'instance (pseudo singleton).

Le but du forward est que dans le projet réel, monAction initialise beaucoup plus que la form. Le forward me permet de ne pas dédoubler des traitements identiques et de ne traiter que le post dans formAction.

Si je suis ton raisonement, je devrais directement gérer le post dans monAction et pas dans une action séparée (formAction) ?

Merci pour tes lumières

[patbator]

Hello,

Le forward exécute une action après l'exécution de l'action courante.
Méthodes utiles du Zend_Controller_Action

Ca veut dire re-dispatching d'une requete vers l'autre action, donc oui, ré-instanciation de votre controller_action.

[vg33]

La façon la plus facile de traiter un form est d'implémenter l'initialisation et le traitement du form dans la même action... justement pour ne pas perdre les données POST.
Concrètement, le form pointe sur la même page, et dans ton controller tu utilises isPost() et isValid(). La doc est très claire sur le sujet.

[littleman]

ou alors mettre l'initialisation dans la méthode init() du contrôleur...c'est utile si le formulaire sert à la fois pour les actions "ajouter" et "modifier" et qu'il est par exemple généré à partir d'un fichier de config.
Personnellement, j'ai ajouté une méthode isPost à ma classe MyForm, celle de l'objet request n'était plus suffisante dès que plusieurs formulaires existaient sur la page.