merged branch Adel-E/patch-1 (PR #767)

Commits
-------

0cb37ba Fix typehint "int" to "integer"

Discussion
----------

Fix typehint "int" to "integer"

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

by nikic at 2012-07-09T21:13:48Z

Why would you want to do that?

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

by Adel-E at 2012-07-09T21:21:34Z

Because in this file all @param use integer instead of int

scaled back on named HTML entities since HTML 5's XML serialisation is actually XML (not XHTML)

see https://github.com/zendframework/zf2/commit/d5862587ae7c2ff2da873f1bf30003103134a365

made the parsing independent of the template loaders

This basically reverts 57ff88255ef35ad8da325d0cb0bf140e653b132c

fixed PHP 5.2 issue in unit tests

fixed exception trace when an error occurs when rendering a child template

Fix typehint "int" to "integer"

fixed composer version

merged branch shieldo/patch-2 (PR #763)

Commits
-------

4e4d41b improved English in example text

Discussion
----------

improved English in example text

improved English in example text

merged branch stealth35/sanitize_ext (PR #761)

Commits
-------

9229ef3 Removed useless code

Discussion
----------

[Ext] Removed useless code

All init hooks could be bypassed (just return `SUCESS`)
There is no `ini` entries and no globals
Phpinfo automatically get the version when `MINFO` is null
Twig don't work with PHP 4 (ref to `20010901`)

Removed useless code

fixed PHP 5.2 incompatibilty

fixed url escaping strategy for PHP 5.2

merged branch Seldaek/date_modify (PR #760)

Commits
-------

2c9e26e Fix date_modify filter for PHP 5.2

Discussion
----------

Fix date_modify filter for PHP 5.2

The return of the object was added in 5.3, this hopefully fixes it.

Fix date_modify filter for PHP 5.2

added support for more escaping strategies (url, css, and html_attr)

tweaked previous commit

merged branch Seldaek/date_modify (PR #757)

Commits
-------

ae85586 Add docs for date_modify
0366cf6 Fix docs of date filter

Discussion
----------

Date modify filter

This is probably possible via `date(foo)|modify('-1day')|format()`, but it's non obvious and a dedicated filter might be a bit more prominent.

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

by fabpot at 2012-06-22T07:01:32Z

Just a quick thought: As it modifies the date, it should probably be a function, not a filter.

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

by stof at 2012-06-22T07:24:52Z

@fabpot Aren't filters meant to transform the value ?

fixed nested embed tag calls

merged branch meandmymonkey/clarify-pre_escape (PR #758)

Commits
-------

f840670 clarified documentation of pre_escape option

Discussion
----------

Clarified documentation of pre_escape option

The previous wording could be misunderstood as "tell the filter that the input has already been escaped", which is actually the opposite from how it works.

clarified documentation of pre_escape option

Add docs for date_modify

Fix docs of date filter

bumped version to 1.8.4-DEV

prepared the 1.8.3 release

fixed paths in the filesystem loader when passing a path that ends with a slash or a backslash (closes #750)

fixed typos

fixed escaping when a project defines a function named html or js (closes #724)

added some information about template recompilation (closes #743)

tweaked doc

fixed doc (closes #744)

added missing comment added (closes #476)

merged branch adrienbrault/patch-1 (PR #751)

Commits
-------

80a2a7e [Doc] Fix some typos

Discussion
----------

[Doc] Fix some typos

Hi!

[Doc] Fix some typos

fixed chmod mode to apply the umask correctly (closes #749)

fixed a phpdoc (closes #748)

added  a note in the CHANGELOG (refs #745)

merged branch catchamonkey/compiler_outdent (PR #747)

Commits
-------

459a076 The Compiler outdent method will now only reduce the indentation if it's OK  to do so, It moves and modifies the check up above the indentation modification.

Discussion
----------

Compiler outdent step validation moved up and modified

The Compiler outdent method will now only reduce the indentation if it's OK
to do so, It moves and modifies the check up above the indentation modification.
It's a small improvement but saves a wasted decrement of a variable, when you know an exception will be thrown immediately following it.

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

by nikic at 2012-06-07T23:18:00Z

Compared to throwing an exception decrementing a variable has practically no cost ;)

PS: Not saying that the change is bad or something like that, only saying that performance is the wrong reason to do it :)

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

by catchamonkey at 2012-06-07T23:20:23Z

Yeah, there is no performance gain, but the code only executes that which is necessary.
It's like there being no point preparing a response before you know if you are going to send it. (slightly OTT example, but you get what I mean) :)

The Compiler outdent method will now only reduce the indentation if it's OK
 to do so, It moves and modifies the check up above the indentation modification.

added Komodo to the list of supported IDEs

added Coda 2 in the list of supported IDEs

bumped version to 1.8.3

prepared the 1.8.2 release

fixed some CS issues

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 ?

Fixed DateTimeZone support in date filter.

Prevents the DateTime object to be updated when changing the DateTimeZone.

Unit tests added.

Updated changelog.

updated CHANGELOG

merged branch maartendekeizer/master (PR #738)

Commits
-------

f0a0036 Add abs filter including unit test and documentation

Discussion
----------

Add abs filter to Twig

With abs it is possible to make negative values positive and leave positive values positive.

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

by maartendekeizer at 2012-05-26T19:02:34Z

Example when using abs

{% if days_left < 0 %}
    {{ days_left|abs }} days to late!
{% else %}
    {{ days_left }} days left
{% endif %}

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

by stof at 2012-05-26T19:12:43Z

I'm not sure about adding this filter in the core. I think it would be better in the Twig-extensions repo. @fabpot thoughts ?

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

by maartendekeizer at 2012-05-26T21:18:47Z

@stof abs is a really basic/natural function it belongs to the base mathematic functions. If this was an implementation of for example the ABC-formula or Pythagoras it must be in the extension repo., but abs is more core related I think.

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

by fabpot at 2012-05-27T07:11:33Z

Can you add some unit tests and squash your commits? Thanks.

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

by maartendekeizer at 2012-05-27T14:40:56Z

First time squashing commits; learned something new :-). I will add a unit test tomorrow.

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

by maartendekeizer at 2012-05-27T17:36:58Z

Unit test and documentation added.

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

by fabpot at 2012-05-27T18:41:25Z

After fixing the markup issue, you will have to squash again ;)

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

by maartendekeizer at 2012-05-27T20:44:29Z

Done

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

by fabpot at 2012-05-28T19:19:55Z

Looks like you have not squash your commits.

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

by maartendekeizer at 2012-05-28T20:05:34Z

@fabot you want one commit for the complete pr?

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

by fabpot at 2012-05-28T20:06:53Z

For such a PR, yes.

Add abs filter
including unit test and documentation

merged branch hhamon/doc_fixes (PR #737)

Commits
-------

953a0ce [doc] fixed Composer section of the introduction. Added a side note to the Composer official documentation for getting more information.

Discussion
----------

[doc] fixed twig version in installation documentation.

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

by lyrixx at 2012-05-25T23:30:01Z

why not use something like "1.*" ?

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

by fabpot at 2012-05-26T05:55:16Z

This is just an example and we are not going to update it with each new release. So, we can either use `1.*` as suggested by @lyrixx or/and add a note referring to the Composer docs.

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

by hhamon at 2012-05-28T05:34:54Z

That's better indeed!

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

by hhamon at 2012-05-28T05:40:46Z

@fabpot you can merge this PR

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

by fabpot at 2012-05-28T06:07:05Z

Can you squash?

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

by hhamon at 2012-05-28T06:31:42Z

@fabpot squashed!

[doc] fixed Composer section of the introduction. Added a side note to the Composer official documentation for getting more information.

fixed an obsolete comment in phpdoc

fixed a regression when using a number in template attributes (closes #733)

added an example for the loop variable (closes #732)

fixed compiler when mbstring.func_overload is set to 2 (closes #731)

fixed CS

bumped version to 1.8.2-DEV

prepared the 1.8.1 release

merged branch Tobion/patch-5 (PR #729)

Commits
-------

b692268 removed leftover of 93c4406f8 and made some language improvements

Discussion
----------

removed leftover of 93c4406f8 and made some language improvements

removed leftover of 93c4406f8 and made some language improvements

fixed tests for PHP 5.2

fixed typos in the doc (closes #726)

updated CHANGELOG

fixed CHANGELOG

fixed a regression introduced by aa6b8358161e4ef35299be06a5ff61bfcba46f6a

removed obsolete code

switched to use mbstring whenever possible instead of iconv (because of issues with iconv and Japanese -- thanks @ganchiku for the tip)

merged branch Fustrate/master (PR #725)

Commits
-------

6c06a32 Coding style

Discussion
----------

Minor typos and coding style

- Unnecessary return in Lexer
- Typo in TokenParserBrokerInterface

Coding style

- Unnecessary return in Lexer
- Typo in TokenParserBrokerInterface

merged branch literal/v1.8.0 (PR #723)

Commits
-------

d13f742 fix example in doc for embed tag
b11412e improve documentation for embed tag

Discussion
----------

improve documentation for embed tag

https://github.com/fabpot/Twig/issues/722

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

by fabpot at 2012-05-10T05:10:37Z

I think this is a great improvement over the original version! The notion of "micro-layouts" certainly helps understand the concept behind the `embed` tag.

Thanks.

fix example in doc for embed tag

improve documentation for embed tag

fixed dump function is_safe value when html_errors is not defined in php.ini (closes #720)

bumped version to 1.9.0-DEV

prepared the 1.8.0 release

added documentation for the bitwise operators (closes #707)

added tests for bitwise operators

clarified usage of the dump function (closes #718)

enforced interface when adding tests, filters, functions, and node visitors from extensions

This avoids some hard to find bugs where you use a Twig_Function_Method
for instance for a filter. This is hard to find because the interface is
almost the same but the getSafe() contract is not the same.

updated CHANGELOG

merged branch stof/mutable_datetime (PR #717)

Commits
-------

a23d4fb Fixed the handling of the timezone for DateTime objects
a164469 Added a failing test demonstrating the bug with DateTime and timezones

Discussion
----------

Mutable datetime

PHP DateTime objects are mutable so changing the timezone when displaying
it affects the following code whereas it does not occur for other
supported formats.

Fixed the handling of the timezone for DateTime objects

Added a failing test demonstrating the bug with DateTime and timezones

PHP DateTime objects are mutable so changing the timezone when displaying
it affects the following code whereas it does not occur for other
supported formats.

added a missing entry in the CHANGELOG

fixed markup in the doc

merged two doc files

renamed a doc file

refactored the hacking twig doc chapter

tweaked doc

added a new recipe

fixed embed compilation when used more than once in a template (closes #714)

made the autoescape tag argument optional (defaults to 'html')

added a note about auto-escaping and the embed tag

We cannot use the parent template to guess the auto-escaping strategy
either as it can be any valid expression, for which we have the real
value only at runtime (auto-escaping happening at compilation time).

ensured that each template (Twig_Node_Module) is only visited once

simplified code

added a way to change the default escaping strategy after creating the Twig environment

tweaked doc for the escape filter

simplified usage of the autoescape tag; the only argument is now the escaping strategy or false

tweaked an error message

added a way to dynamically change the auto-escaping strategy according to the template filename