Reacties op: Symfony: Multiple embedded forms in a specific layout http://www.paulderaaij.nl/2010/02/27/symfony-multiple-embedded-forms-layout/ Algemeen weblog van Paul de Raaij Tue, 13 Dec 2011 09:01:53 +0000 hourly 1 http://wordpress.org/?v=3.1 Door: Paul de Raaij http://www.paulderaaij.nl/2010/02/27/symfony-multiple-embedded-forms-layout/comment-page-1/#comment-285 Paul de Raaij Sat, 24 Jul 2010 08:39:40 +0000 http://www.paulderaaij.nl/?p=162#comment-285 Hi Arif, Glad this post helped you with you journey in Symfony. About the refactoring, you're absolutely right. But when I was figuring out this stuff I've changed the code so much, that the MVC principle got lost along the way. In the end result everything was refeactored the correct way :) Extending the baseform is a very good idea! Thanks for your feedback! Hi Arif,

Glad this post helped you with you journey in Symfony.

About the refactoring, you’re absolutely right. But when I was figuring out this stuff I’ve changed the code so much, that the MVC principle got lost along the way. In the end result everything was refeactored the correct way :)

Extending the baseform is a very good idea! Thanks for your feedback!

]]> Door: Arif Gebhardt http://www.paulderaaij.nl/2010/02/27/symfony-multiple-embedded-forms-layout/comment-page-1/#comment-284 Arif Gebhardt Mon, 19 Jul 2010 21:49:36 +0000 http://www.paulderaaij.nl/?p=162#comment-284 Hi Paul, thank your for this post, it helped me a lot! I am currently working on something similar (no football, but the object relations are similar). But you definetly should refactor your code by putting something like Doctrine_Query::create()->from('Predictions')->where('user_id = ? ', sfContext::getInstance()->getUser()->getProfile()->getUserID())->andWhere('game_id = ?', $this->getObject()->getID()); into the corresponding model class! In this case it is PredictionsTable.class.php, i think. That fits a lot better into the MVC pattern as the query stuff is done in the model, where it belongs. One last thing: You complained about the form extending BaseGamesForm which is forcing you to unset all widgets first. You can avoid this workaround by simply building your custom form class which then extends BaseForm. Arif Hi Paul,
thank your for this post, it helped me a lot!
I am currently working on something similar (no football, but the object relations are similar).
But you definetly should refactor your code by putting something like

Doctrine_Query::create()->from(‘Predictions’)->where(‘user_id = ? ‘, sfContext::getInstance()->getUser()->getProfile()->getUserID())->andWhere(‘game_id = ?’, $this->getObject()->getID());

into the corresponding model class! In this case it is PredictionsTable.class.php, i think.
That fits a lot better into the MVC pattern as the query stuff is done in the model, where it belongs.

One last thing:
You complained about the form extending BaseGamesForm which is forcing you to unset all widgets first.
You can avoid this workaround by simply building your custom form class which then extends BaseForm.

Arif

]]>