merged branch nikic/nestedDefinedTestWithStrictVars (PR #359)
authorFabien Potencier <fabien.potencier@gmail.com>
Thu, 16 Jun 2011 07:41:50 +0000 (09:41 +0200)
committerFabien Potencier <fabien.potencier@gmail.com>
Thu, 16 Jun 2011 07:41:50 +0000 (09:41 +0200)
Commits
-------

f2a1c2b Refactor and add additional default filter tests
e760483 Make `a.b is defined` not throw an exception if a is not defined (in strict mode)

Discussion
----------

Make `a.b is defined` not throw an exception if a is not defined (in strict mode)

This commit does two things: Firstly (that's the main part) it makes `a.b is defined` not throw an error in strict mode if `a` is undefined (applies for `a.b.c` aso too) [fixes #337]. Additionally it allows `a.b|default` just like it is allowed to do `a|default`.

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

by fabpot at 2011/06/09 22:21:11 -0700

This is quite a big change. Can you add some tests?

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

by nikic at 2011/06/10 08:19:49 -0700

I added some more tests for the default filter in the fixtures. But I wasn't sure how I should test the default test's behavior in strict mode. If you could point me in the right direction here I would add those tests, too ;)


Trivial merge