merged branch blaugueux/master (PR #736)
authorFabien Potencier <fabien.potencier@gmail.com>
Tue, 29 May 2012 18:09:42 +0000 (20:09 +0200)
committerFabien Potencier <fabien.potencier@gmail.com>
Tue, 29 May 2012 18:09:42 +0000 (20:09 +0200)
commit26db55023028f410da6b09baf9d56ef75d784772
treee37d5c4cac9128b1becebf6ed98d684cc4a8d051
parent78a935b6ffbcbd8f4e740df895f87a93347b551c
parentc53a02de1a1781c356a2620e722a3f0e20ed32fa
merged branch blaugueux/master (PR #736)

Commits
-------

c53a02d Fixed DateTimeZone support in date filter.

Discussion
----------

Fixed DateTimeZone support in date filter.

Now $date converted to a DateTime has a DateTimeZone in every case.

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

by fabpot at 2012-05-28T19:20:47Z

Can you add some unit tests to avoid regression to happen later? Thanks.

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

by blaugueux at 2012-05-29T07:52:58Z

@fabpot Here they are.

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

by fabpot at 2012-05-29T10:29:28Z

Great! Can you add a note in the CHANGELOG file and then squash your commit before I merge? Thanks.

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

by blaugueux at 2012-05-29T12:52:22Z

Changelog added. First squash of my life, i cannot say if it's good :(

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

by stof at 2012-05-29T12:55:26Z

it is not. you merged your remote branch instead of forcing the push

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

by blaugueux at 2012-05-29T17:12:05Z

Ok @stof, thanks for your help. I think it's good now.

@fabpot Is it ok for you ?
CHANGELOG
lib/Twig/Extension/Core.php