merged branch arnaud-lb/792 (PR #814)
authorFabien Potencier <fabien.potencier@gmail.com>
Thu, 23 Aug 2012 12:18:58 +0000 (14:18 +0200)
committerFabien Potencier <fabien.potencier@gmail.com>
Thu, 23 Aug 2012 12:18:58 +0000 (14:18 +0200)
Commits
-------

8fa97bb php5.2 fix
d6fc86a [Tests] Test accessing a public property when \ArrayAccess is implemented
6704227 [ext] made ext consistent with 8ec73cf4759f90b961de1732da1732e60c1dbfff
228d2b0 [ext] use only read_property handler

Discussion
----------

Alternative fix for 792

This fixes #792 as described in https://github.com/fabpot/Twig/issues/792#issuecomment-7482480

- 228d2b0 removes the *object-to-array convertion* trick, since the bug comes from there, and it doesn't appear to be needed
- 6704227 is the equivalent of aa6b8358161e4ef35299be06a5ff61bfcba46f6a + 8ec73cf4759f90b961de1732da1732e60c1dbfff (this were not needed before because the bug was hidden by the object to array trick: numeric strings and integers are treated equally in arrays keys)

---------------------------------------------------------------------------

by stof at 2012-08-23T10:19:17Z

@arnaud-lb you need to fix the PHP 5.2 support

1  2 
ext/twig/twig.c

diff --cc ext/twig/twig.c
Simple merge