Commentaires sur : Snippet sfValidatorEmailList

Par : Richtermeister

Richtermeister — Mon, 19 Oct 2009 00:02:29 +0000

Very nice validator! Elegant and useful, thank you for sharing.

I think you should try to submit this to be included in the symfony core in the future.

Have a great day,
Daniel

Par : NiKo

NiKo — Wed, 14 Oct 2009 14:13:57 +0000

C’est pour ça que j’aime bien le setOption() qui se chaîne aisément grâce à son interface fluide. J’aime pas trop le passage d’arguments au constructeur, notamment du fait de la capacité de ce dernier à être surchargé en PHP.

Les goûts et les couleurs…

Par : yoye

yoye — Wed, 14 Oct 2009 14:06:44 +0000

Je viens de corriger cela par contre au niveau de mon action je préfère instancier mon objet comme suit :

$this->form = new lkInvitationForm(array(), array(‘mailer’ => $this->getMailer()));

je ne sais pas si c’est une bonne ou mauvaise pratique mais je n’aime pas trop surcharger mon action.

Après on peut aussi se poser la question à savoir si on peut rédéfinir le constructeur car le tableau vide en premier paramètre va correspondre à 90% de mes instanciations et le mailer étant obligatoire cela devient de la dependency injection.

Par : [MA]Pascal

[MA]Pascal — Wed, 14 Oct 2009 14:04:07 +0000

Pratique comme validateur, par contre comme NiKo je comprends pas trop l’implémentation.

Pourquoi passer sfWebRequest $request à bindAndSend ? tu es déjà dans un form qui est normalement valide donc hydraté.

Quitte à mettre des params à cette méthode c’est ici que j’aurai fait passer $this->getMailer();

Pourquoi le mettre en param de tout le form ?

Par : NiKo

NiKo — Wed, 14 Oct 2009 13:45:39 +0000

C’est pas une super idée d’utiliser sfContext::getInstance() dans le formulaire ; il vaut mieux passer le mailer en option de ce dernier (et du même coup le mail de l’expéditeur tant qu’à faire), comme ça par exemple : http://pastie.org/654442

Ainsi le formulaire devient facilement testable, grâce au découplage ainsi opéré.