merged branch arnaud-lb/ext (PR #530)
authorFabien Potencier <fabien.potencier@gmail.com>
Wed, 23 Nov 2011 13:14:41 +0000 (14:14 +0100)
committerFabien Potencier <fabien.potencier@gmail.com>
Wed, 23 Nov 2011 13:14:41 +0000 (14:14 +0100)
commit8d9e08518e13efe068e138f9c678c4e31f02c3c4
tree6336d3b1bbfa8369eb8200b7c4688358bf473cd4
parentfa71922a19fe07bb522cec1176c1b7c3d39586cc
parent569f782ad0e092d95dd48f0e07835edfbc50aaf5
merged branch arnaud-lb/ext (PR #530)

Commits
-------

569f782 fix ZTS build
1606910 Fix undefined behavior
0ce8d3c Convert item argument to string
d807981 don't try to access non public property or method
7e566b8 test twig_template_get_attributes

Discussion
----------

twig_template_get_attributes tests and fixes

This enables testing of twig_template_get_attributes (every TemplateTest test is ran against twig_template_get_attributes too) and fixes some bugs I've found.

There is still 3 failing tests due to a difference of how twig_template_get_attributes and Twig_Template::getAttribute use Twig_Template::$cache.

Twig_Template::getAttribute doesn't cache properties, and twig_template_get_attributes expects $cache[$class]['properties'] to be set if $cache[$class] is set. So twig_template_get_attributes can fail if the cache is already populated by Twig_Template::getAttribute.