Kornel Lesiński <kor...@...> writes:

> 
> >> $template = new PHPTAL($this->_viewDir . DS . $fileName);
> >> $template->set($this->_viewVars);
> > 
> > Another thing that IMHO should be considered is that PHPTAL should
> > implement ArrayAccess so you could do
> > $template['menu'] = $menu;
> > 
> > Would a patch for this be accepted?
> 
> Sorry, no. That would duplicate $template->menu = $menu syntax, which works 
already.
> 
> I agree about setting all variables in one call, but I'd add separate method 
for it (setAll?)

Yeah, a separate method might be the better solution. The proposed solution was 
just a 1:1 copy from the magento template class we're extending.

Thank you & regards,
Moritz





_______________________________________________
PHPTAL mailing list
PHPTAL@lists.motion-twin.com
http://lists.motion-twin.com/mailman/listinfo/phptal

Reply via email to