Comments on: Zend Framework 1.8 is upon us!

By: Marcelo

Marcelo — Wed, 22 Jul 2009 17:51:46 +0000

Great, i commented the wrong post… hahaha

stupid!

right one => Creating simple, extendible CRUD, using Zend Framework

By: Marcelo

Marcelo — Wed, 22 Jul 2009 17:30:00 +0000

Btw, this post is awesome. Thank you for teaching good code!

By: Marcelo

Marcelo — Wed, 22 Jul 2009 17:27:47 +0000

I confess i didn’t test it, but there is one thing i dint understand. How is the form populated when editing (nothing posted yet).

Shouldn’t $users->getEditUserForm($post) also take $id as parameter just in case nothing was posted?

Sorry if i am talking BS, but i am very used to my massed spaghetti code pattern =/

By: Ryan

Ryan — Tue, 02 Jun 2009 19:51:49 +0000

Kenrick, Thank you for the comments, and I’m glad you like it

I have come across the same problem before, and I found that the way I dealt with this was to add the validator to the relevant element later than init(), as I usually have a process method in my forms which takes the post data, and a table row, and sets defaults, and determines if the post is empty, and then calls isValid, this was the ideal place for me to add this validator to the element, as I could get the ID from the row being passed to the method. and insert the validator just before calling isValid.
This doesn’t really save any overhead, but it sure looks prettier than overloading the constructor.

By: kenrick

kenrick — Tue, 02 Jun 2009 19:08:16 +0000

hey that new validator is great, though for an idiot like me I had a gotcha when using it in conjunction with extending a Zend_Form. You can’t just call it from inside of the init() function, and expect it to be able to get at the values being passed so you can validate against stuff. Uh lemme explain:

myForm extends Zend_Form
{
function init()
{
$this->addElement(….array(
‘validators’=>array(
array(‘Db_NoRecordExists’,
false,
array(‘users’,
‘email’,
array(
‘field’=>’id’,
‘value’=>$this->_userId // this has to come from someplace!
// you can just do a $this->id->getValue(), since
// this is inside the init() function…
)
))
)
))
}
}

well I just had to modify the __construct method of myForm to accept some kind of value that could be used in the init() function for check records in the database. Maybe this is totally obvious to everyone else, but I’m working on no sleep[ and 6 cups of coffee right now…

Thanks again for the contribution.