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)
commitcf50380b8c33d885ac04403a19da2a64141c4031
tree04a7d528a0163278734d867f9f1df766810f5705
parent6c442bd9baff5c83759a50b2e574eaeb6a96fe93
parentf2a1c2b539244eb9a710cf9918bc75735235a003
merged branch nikic/nestedDefinedTestWithStrictVars (PR #359)

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 ;)