Matt – when you document this, and also in the class-level docblock, please put a notice that phptal is licensed under the LGPL and that users should only use the wrapper and phptal when their project license will not conflict with the LGPL.

Otherwise, looks good!

Matt,

First of all thank you for your work on this class, I was thinking about implementing it when I found it

What would you thing of changing __get() to the following:

if ($this->__isset($key)) {
  if (is_array($this->_variables[$key])) {
    return new ArrayObject($this->_variables[$key]);
  }
  return $this->_variables[$key]; 
} 
return null; 

This change (returning an ArrayObject for array variables) allows to have read-write access on the variable when accessing it from the view object. Like in the following example:

$this->view->errors = array();
$this->view->errors[] = 'First error';
$this->view->errors[] = 'Second error';

I guess it's not the fastest code but it's quite convenient.

Besides, setting a variable starting with an underscore ('_') just silently ignores the action. I think it should throw an exception to notify the developer. Moreover, the assign() method does not even check for the underscore which could result on a exception being thrown from Phptal at render time, since the variables are actually passed over then.

Just a last thing. Why are the script paths using 'templates' as the default folder name instead of 'scripts'? Is it intended or just a left over from previous ZF versions?

cheers,
Iván

Is this proposal dead? I hope not! I'm looking forward to see such a component..

I have modified Your code to extend Zend_View_Abstract. Unfortunately there is no english version in this moment so You have to use translator (comments are in English, I'll try to rewrite it soon).

PHPTAL View (Zend_View_Abstract)

Gtranslate has some problems with valid syntax so please refer to original text.
Article contains examples both with Zend_Application and standard aproach. Additionaly `helper:` modifier is available inside templates (e.g. <a tal:attributes="href helper:url(array('username' => 'martel'), 'user'))">Martel</a>), templates could be gzipped and correct headers are set (because ZF does not have any Zend_Client class i put my own My_Browser class).