merged branch fabpot/twig-c-bug (PR #920)
authorFabien Potencier <fabien.potencier@gmail.com>
Thu, 6 Dec 2012 07:16:52 +0000 (08:16 +0100)
committerFabien Potencier <fabien.potencier@gmail.com>
Thu, 6 Dec 2012 07:16:52 +0000 (08:16 +0100)
commitbc3a7e806f7ed2a5254a6176e7c183a0a6780b2a
tree32e4fd5caffeed41cad67ef7e5dbaded8f7737b0
parent6810dbe8a7be204813a4397b1bddcfdb875fe2e2
parentdf13370b224f6c3f06aadc6dfcbba86fdcbae364
merged branch fabpot/twig-c-bug (PR #920)

This PR was merged into the master branch.

Commits
-------

df13370 added a missing test
33e690b fixed some tests when the extension is not enabled
51e707f merged branch char101/fix-ext-retval (PR #921)
014f459 added some missing test for Template::getAttribute()
e8d12da Fix empty string comparison
4311df3 added some missing test for Template::getAttribute()

Discussion
----------

The C extension does not behave in the same way as the PHP code

The added test shows that the C extension does not behave in the same way as the PHP code. I have no idea why. The problem is probably around line 1033 in the extension code: https://github.com/fabpot/Twig/blob/master/ext/twig/twig.c#L1033

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

by stof at 2012-12-05T17:07:00Z

The testsuite is broken when you run it without the C Twig extension. The test should be skipped when ``useExt`` is ``true`` and the extension is not available