improve test coverage and support negative int for random function

tweaked doc

merged branch Tobion/patch-4 (PR #614)

Commits
-------

a15e8f7 typo again
2561aa2 typo
faa90c9 updated doc for random function
fe66358 added tests for random function
6f5ceee Made the random function more versatile

Discussion
----------

Made the random function more versatile

The point is to meet user expectations and to make it more versatile, so it works under more circumstances.
At the moment the random function does only return the supplied parameter when it's not an array. That is not very useful. With this PR it will behave like

    random() => integer as with mt_rand()
    random(5) => integer between 0 and 5 as with mt_rand(0, 5)
    random('foobar') => random character from 'foobar'
    random(array(...)) => random element

I will add tests and documentation if you accept this PR.

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

by fabpot at 2012-01-26T07:29:35Z

Looks good to me. Can you update the docs and add some unit tests?

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

by Tobion at 2012-01-26T11:50:17Z

ready

typo again

typo

updated doc for random function

added tests for random function

Made the random function more versatile

merged branch Tobion/patch-3 (PR #613)

Commits
-------

e81a9e2 Improved random function

Discussion
----------

Improved random function

- dealing with empty array (otherwise PHP warning is raised)
- using array_rand() and thus saving the $keys variable

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

by Tobion at 2012-01-25T12:41:32Z

How about allowing `mt_rand ( int $min , int $max )` with optional parameters to be used when the $values ist not an array?
This would solve use case #612

Improved random function

tweaked previous merge (no need to duplicate visitors, handlers and the expression parser when parsing a template inside another one)

add a unit test for previous merge

merged branch Seldaek/inception (PR #610)

Commits
-------

0d656f5 Add comments
0255764 Updated CHANGELOG
b327a48 Protect the Parser against recursive parsing issues

Discussion
----------

Make the parser Inception-Proof

Spent half a day debugging before I realized what happened, but I'll try to keep a long story short:

When the cache is empty, and the first template containing an assetic `{% javascripts %}` or similar tag is parsed, it will build up the assetic "assets" or recipes cache, this in turn will tokenize and parse all your templates to find assetic tags and cache that information.

At this point the parser is parsing something else in the middle of a parse() call, and since there is a single instance in the environment, it means all the instance vars are messed up and contain incorrect references to the latest TokenStream that was parsed by assetic, etc.

This had two effects on my application, both appearing seemingly randomly because it highly depends on the order of things, the state of your cache and probably other factors:

- The first thing that happened is that a template was compiled using the wrong template filename, which means I had a `__TwigTemplate_abcd` in the file named `dcba.php`, and it would never find the right class.
- The second issue (could not reproduce but I assume it was caused by this as well) is that the parse tree is completely broken and you end up with a parse error because it thinks it's at the end when it's not, or similar problem.

The proposed fix basically pushes/pops all the vars into a stack whenever the parser starts/stops, which worked very effectively here and does not introduce much breakage or complexity.

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

by stof at 2012-01-24T17:28:06Z

@Seldaek are you able to create a reproducible testcase for this (which should be failing before this fix) ? It would avoid further regressions

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

by Seldaek at 2012-01-24T17:45:37Z

I'll try to improve on this according to feedback tomorrow. I saw this enough for today :)

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

by fabpot at 2012-01-24T17:45:49Z

I'm writing some unit tests

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

by Seldaek at 2012-01-24T17:47:02Z

Ok then I'll add @dzuelke's comments real quick.

Add comments

merged branch Seldaek/typo (PR #609)

Commits
-------

d8527fe Fix docblock

Discussion
----------

Fix docblock

Updated CHANGELOG

Protect the Parser against recursive parsing issues

Fix docblock

removed duplicated tests

added the possibility to skip some tests depending on the PHP version

fixed typo

made a speed optimization to macro calls when imported via the from tag

updated CHANGELOG

fixed the attribute function when passing arguments (closes #608)

added some unit tests for macros

Revert "made a speed optimization to macro calls when imported via the from tag"

This reverts commit 3b87da344ade9a719d3e105eb7ca20864b725875.

made a speed optimization to macro calls when imported via the from tag

merged branch Tobion/patch-2 (PR #606)

Commits
-------

83a3073 typo

Discussion
----------

typo

typo

fixed typo in doc (closes #604)

merged branch Seldaek/globals_fix (PR #600)

Commits
-------

43ebae1 Reset parsers, visitors, filters, tests, functions when a new one or new extension is added
3db62da Reset globals when a new global is added

Discussion
----------

Reset globals when a new global is added

if getGlobals() is called before adding a new global, it is never taken into account

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

by fabpot at 2012-01-16T14:27:02Z

Is it related to #594?

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

by Seldaek at 2012-01-16T14:30:08Z

Seems like it yes. It means that reading globals + writing + reading again will call getGlobals on all extensions again though.

A slight improvement would be to cache the extension globals in another var that can be re-applied at once over the staged globals.

The fastest would be to just set directly in ->globals if it's already initalized, but that means addGlobal can override extension globals. Not sure if that is a problem, but it creates a somewhat non-deterministic factor.

I can adjust as required.

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

by stof at 2012-01-16T14:45:49Z

@Seldaek shouldn't the same be done for other methods (filters, functions, tags, tests and operators) ?

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

by Seldaek at 2012-01-16T14:53:28Z

Indeed, I was just focused on my crazy issue here :) Once @fabpot decides which way he prefers I can apply it to everything.

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

by fabpot at 2012-01-16T21:05:19Z

+1 for the the approach in this PR. @Seldaek: Can you apply it to everything else as well? Thanks.

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

by Seldaek at 2012-01-17T00:54:08Z

Done, also did it for adding and removing extensions since they potentially override stuff.

Reset parsers, visitors, filters, tests, functions when a new one or new extension is added

Reset globals when a new global is added

merged branch Partugal/twig-ext (PR #598)

Commits
-------

75bfb4e Syncing Twig extension with latest changes in Twig master

Discussion
----------

Syncing Twig extension with latest changes in Twig master

Syncing Twig extension with latest changes in Twig master

made a cosmetic change

merged branch lyrixx/patch-4 (PR #596)

Commits
-------

fa2501f Added a link to the documentation site

Discussion
----------

Added a link to the documentation site

Now, on the github project page, there is a link to http://twig.sensiolabs.org/documentation. (like silex)

merged branch aerialls/patch-2 (PR #597)

Commits
-------

924c8b3 fixed CS

Discussion
----------

fixed CS

fixed CS

Added a link to the documentation site

added support for optional length in the slice [] notation

added a slice filter

fixed core filters phpdoc

added string support for the reverse filter

tweaked doc for the merge filter

tweaked documentation

fixed some phpdoc

fixed charset for previous commit

fixed the empty test and the length filter for Twig_Markup instances (closes #589)

added one more rule in the coding standards (closes #587)

added one more rule to the coding standards

re-added the version in composer.json (if not, the version on packagist is master-dev instead of the more desirable 1.6-dev)

merged branch aerialls/patch-1 (PR #585)

Commits
-------

3c934f4 Update doc/coding_standards.rst

Discussion
----------

Update doc/coding_standards.rst

Update doc/coding_standards.rst

fixed markup in doc

added coding standards in the docs

merged branch lyrixx/patch-dead-code (PR #584)

Commits
-------

6ca2ca6 [Tests] Removed dead code in core test

Discussion
----------

[Tests] Removed dead code in core test

$core is never used. Test keeps passing.

[Tests] Removed dead code in core test

added a recipe for stateful node visitors

added missing phpdoc

added a date function to ease date comparison (closes #571)

updated CHANGELOG

fixed unary operators precedence (closes #583)

merged branch vicb/doc/use (PR #581)

Commits
-------

28b9b7b [Twig] Add a note about multiple inheritance (use tag)

Discussion
----------

[Twig] Add a note about multiple inheritance (use tag)

To add some visibility to this feature

[Twig] Add a note about multiple inheritance (use tag)

bumped version to 1.6.0-DEV

prepared the 1.5.1 release

fixed regression in string parsing (closes #580)

merged branch lenar/patch-1 (PR #578)

Commits
-------

5214794 compiles -> compile

Discussion
----------

compiles -> compile

compiles -> compile

added a note about the Twig extension DLL for Windows

bumped version to 1.6.0-DEV

prepared the 1.5.0 release

fixed doc (closes #577)

merged branch lyrixx/patch-1 (PR #576)

Commits
-------

7e8df56 Fixed CS

Discussion
----------

Fixed CS

I just fixed the CS in the doc

Fixed CS

fixed the join filter when several items have the same key

simplified code

added a unit test

added Traversable objects support for the join filter

prepared the 1.5.0-RC2 release

updated doc for the date filter

added a way to set the default global date interval format

refactored date filter code

merged branch alefranz/patch-2 (PR #575)

Commits
-------

b295274 fix Date Format filter: DateInterval does not provide setTimezone()

Discussion
----------

fix Date Format filter: DateInterval does not provide setTimezone()

fix Date Format filter: DateInterval does not provide setTimezone()

refactored Twig_Template::display() to ease its extension

tweaked previous merge

merged branch markstory/number-format (PR #573)

Commits
-------

e3b81ad Add documentation page for number_format.
0edcfdd Add number_format filter.

Discussion
----------

Implement number_format filter

Implement a `number_format` filter that wraps the native PHP `number_format`.

Fixes #417

Add documentation page for number_format.

Add number_format filter.

Refs #417

fixed Twig_Node_Expression_Array::hasElement()

prepared the 1.5.0-RC1 release

fixed markup

added missing doc for the do tag

fixed typo

changed hash key expression support (expression must now be enclosed with parentheses and hash keys can now be names)

merged branch jeserkin/jeserkin (PR #570)

Commits
-------

fd78ec0 Fixed PhpDoc for Twig_TokenParserInterface and all other TokenParsers

Discussion
----------

Fixed PhpDoc for Twig_TokenParserInterface and all other TokenParsers

Fixed PhpDoc for Twig_TokenParserInterface and all other TokenParsers

added convenience methods to manipulate an array node