Shouldn't line 1185 of Element.php be:

if ($translator && method_exists($validator, 'setTranslator')) {

  ?
It seems inefficient to call a method that just does some checks and then
sets the already null _translator member to null again.

It seems like a small issue, but just wanted to mention it as something that
could be cleaned up in the next release.


Element.php line 1185:

        foreach ($this->getValidators() as $key => $validator) {
            if (method_exists($validator, 'setTranslator')) {
                $validator->setTranslator($translator);
            }
-- 
View this message in context: 
http://www.nabble.com/Inefficient-setTranslator--tp17399157p17399157.html
Sent from the Zend Framework mailing list archive at Nabble.com.

Reply via email to