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

Revert "Revert "merged branch arnaud-lb/hash-key-expr (PR #514)""

This reverts commit 99b2c5bf99ea299685886e33d54fbbcd6348b849.

merged branch javiereguiluz/add-nl2br-notice (PR #568)

Commits
-------

7194b1a Added a notice in the `nl2br` filter documentation about the proper Twig version to use it

Discussion
----------

Added a notice in the `nl2br` filter documentation ...

... about the proper Twig version to use it (1.5)

Added a notice in the `nl2br` filter documentation about the proper Twig version to use it

Revert "merged branch arnaud-lb/hash-key-expr (PR #514)"

This reverts commit 8665c6463461ccd0afec1ff943b45b56ad01a06e, reversing
changes made to 4aba337cda03b0170cbf30b43cd82cb83146f2b4.

updated doc

updated CHANGELOG

merged branch arnaud-lb/hash-key-expr (PR #514)

Commits
-------

542e9e7 added getKeyValuePairs() on Twig_Node_Expression_Array
3f64f1d added test for hash keys
4bdb3a9 fixed tests
3b1f269 support any expression as hash key

Discussion
----------

Allow hash keys to be any expression

This allows hash keys to be any expression

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

by fabpot at 2011/12/07 02:08:19 -0800

I had not implemented this feature back then because I'm not sure it makes sense to support it in the context of a templating system. I'm still not convinced that this is useful.

added a do tag (closes #446)

added doc for the flush tag

added IDE support for Coda in the doc (closes #566)

tweaked doc (closes #567)

replaced zend_error_noreturn calls to zend_error as zend_error_noreturn is not exported (closes #554)

merged branch arnaud-lb/ext-fix (PR #565)

Commits
-------

b26d69a removed bogus TSRMLS_CCs

Discussion
----------

Fix strict mode in extension when building in ZTS mode

This removes a few misplaced TSRMLS_CCs causing if() expressions to always evaluate to true. This caused twig_template_get_attributes to always behave as if strict mode were disabled.

fixed typo

removed bogus TSRMLS_CCs

added a flush tag

merged branch nikic/fixLoopIncrement (PR #563)

Commits
-------

8255b8b Fix test for previous commit
834ce4c Fix loop increment on some for loops (fix #562)

Discussion
----------

Fix loop increment

PR for #562.

Fix test for previous commit

Fix loop increment on some for loops (fix #562)

The inner loop incrementing code was not added on for loops where the body was not a Twig_Node (but e.g. a Twig_Node_If)

fixed dump when xdebug.overload_var_dump is not defined in php.ini

added support for dynamically named filters and functions

made the dump function registration explicit and disable its output when debug is off

fixed dump function when the context contains Twig_Template instances

added a dump function to help debugging templates

fixed the random function

added a nl2br filter

added missing documentation for the random function

added a random function

added a way to change the default format for the date filter

merged branch arnaud-lb/ext-crash2 (PR #560)

Commits
-------

9e6f285 fixed crash in twig_template_get_attributes when input is not an array or object

Discussion
----------

Fix crash in twig_template_get_attributes when input is not an array or object

This fixes a crash when the input of twig_template_get_attributes() is not an array or object, and strict mode is enabled.

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

by nikic at 2011/12/16 12:03:21 -0800

Could you maybe place the exception tests in a different method?

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

by arnaud-lb at 2011/12/16 12:13:35 -0800

This method already expects many of its inputs to cause an exception to be thrown (it tests getAttribute() in strict mode, and in many cases the attribute isn't defined). I don't see a real benefit from moving these tests in a separate method in this case.

fixed crash in twig_template_get_attributes when input is not an array or object

updated NetBeans plugin URL

merged branch arnaud-lb/ext-crash (PR #558)

Commits
-------

c920344 code style
50da1a2 don't modify function name

Discussion
----------

Fix crash in extension

This fixes a crash caused by `php_strtolower()` modifying its input. (Crashes on amd64, but not on x86 for me.)

code style

don't modify function name

merged branch JEDIBC/master (PR #556)

Commits
-------

3bb822e . misspell function _twig_convert_encoding -> twig_convert_encoding

Discussion
----------

misspell function _twig_convert_encoding -> twig_convert_encoding

Located in _twig_escape_js_callback, the function call was misspelled and raised a PHP Fatal error:  Call to undefined function _twig_convert_encoding() in ~/www/project/vendors/Twig/Extension/Core.php on line 585

. misspell function _twig_convert_encoding -> twig_convert_encoding

moved some common compiled code to Twig_Template

fixed the lexer when an operator ending with a letter ends a line

added Twig C extension version in phpinfo

moved regexes in the lexer to their own array

added documentation for the new string interpolation feature

merged branch arnaud-lb/str-interpolation (PR #515)

Commits
-------

d41d10c simplification
2dbb420 moved hard-coded regexes
61986c1 shortcut for non-interpolated double quoted strings
062ed5c regex optimization
0e63abb integration tests for interpolated strings
773fff5 parser tests for interpolated strings
098634e parser support for interpolated strings
c8bba24 lexer tests for interpolated strings
a203686 lexer support for interpolated strings
0bd63dd stack lexer states

Discussion
----------

String interpolation syntax

This adds support for string interpolation:

```
{{ "foo #{ any expression here } bar" }}
```

I find string interpolation to be often more readable than concatenation, and doesn't have the precedence ambiguity of the concatenation operator:

```
Interpolation:

{{ "foo #{bar} baz" }}
{{ "foo #{1+2} baz" }}
{{ "foo #{1+2} baz"|escape }}

Concatenation:

{{ "foo "~bar~" baz" }}
{{ "foo "~(1+2)~" baz" }} extra parentheses
{{ ("foo "~(1+2)~" baz")|escape }} more extra parentheses
```

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

by nikic at 2011/11/12 12:41:54 -0800

Really nice idea. We should discuss the exact syntax though :) Dart for example uses `${expr}` for interpolation or `$name` if the expression is just a name. I think it makes sense to have a special short syntax for the name case, because it is the most common one.

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

by stof at 2011/11/12 13:06:51 -0800

@nikic could you comment on the PR instead of commenting on each commit individually ? it would make it easier to see what is concerned in the PR.

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

by arnaud-lb at 2011/11/12 14:22:14 -0800

@nikic Thanks for your review, I'll take care of your remarks.

For the syntax, I have no preference. I've taken this one because it has the same semantics. In Ruby and Coffeescript the `#{...}` allows any expression between the brackets. In PHP the expression in the `${...}` must start with a variable name. As you mentioned Dart's `${...}` seems to allow any expression.

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

by nikic at 2011/11/12 15:03:48 -0800

@arnaud-lb If Ruby and Coffeescript use this, the #{...} syntax should be okay. Especially as ${...} might be confusing for PHP developers as it has the semantics of variable variables in PHP.

@stof Thanks, will do so next time.

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

by damianb at 2011/11/12 15:38:31 -0800

@stof he doesn't need to - this is something github handles already. in the PR view, beside the "@nikic commented" bits, there's a set of links that shows where exactly the comment was in the format of commit sha, filename, and line inside the diff.

You can see for yourself right here, upper left:

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

by stof at 2011/11/13 01:20:38 -0800

@damianb The comment appears, but not the context in which the comment was. We have to follow each link to see it whereas comments done on the PR appear directly with their context. This is why I said it is easier when comments are on the PR.

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

by nikic at 2011/12/02 15:13:30 -0800

So what about this now?

Btw, `REGEX_STRING` and `REGEX_DQ_STRING_PART` still are hardcoded and do not depend on the specified `interpolation` tags. But this looks hard to change, at least the regexes are quite complicated.

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

by arnaud-lb at 2011/12/02 15:22:45 -0800

I think I've addressed all issues (except hardcoding of `#{` and `}` in `REGEX_STRING` and `REGEX_DQ_STRING_PART`.)

Can this be merged after v1.4 is release if the syntax is accepted ?

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

by nikic at 2011/12/03 01:53:41 -0800

This would be my go for `STRING` and `DQ_STRING_PART` (isn't tested):

```php
<?php
$interpolChar = preg_quote(substr($options['interpolation'][0], 0, 1), '/');
$interpolPart = '[^' . $interpolChar . '"\\\\]*';
if (strlen($options['interpolation'][0]) > 1) {
    $interpolRest = preg_quote(substr($options['interpolation'][0], 1), '/');
    $interpolRegex = $interpolPart . '(?:(?:\\\\.|' . $interpolChar . '(?!' . $interpolRest . '))' . $interpolPart . ')*';

} else {
    $interpolRegex =  $interpolPart . '(?:\\\\.' . $interpolPart . ')*';
}
$this->options['dq_string_part_regex'] = '/' . $interpolRegex . '/As';
$this->options['string_regex'] = '/"' . $interpolRegex . '"|\'([^\'\\\\]*(?:\\\\.[^\'\\\\]*)*)\'/As';
```

It should allow both a single char interpolation syntax (like `{`) and multichar ones (like `#{`).

By the way, could we maybe move the compiled regexes out of `$this->options` into `$this->regexes`?

added the CHANGELOG to the PEAR package

enhanced exceptions for unknown filters, functions, tests, and tags

removed dead code

added a way to get all registered tags

Removed trailing white spaces

Help the developer when they specify an invalid function by providing some alternatives

merged branch jonathaningram/patch-1 (PR #549)

Commits
-------

989bb22 Removed trailing white spaces
f880bab Help the developer when they specify an invalid filter by providing some alternatives

Discussion
----------

Help the developer when they specify an invalid filter by providing some...

... alternatives.

Bug fix: no
Feature addition: yes
Backwards compatibility break: no

Uses a simple `strpos` search (no fancy diff to determine what the filter could be). There is no impact to runtime performance because this is all happening during template compilation.

Examples:

```jinja
{{ my_entity|json }}
```

Shows exception:

`The filter "json" does not exist. Did you mean "json_encode"?`

```jinja
{{ my_date|dat }}
```

Shows:

`The filter "dat" does not exist. Did you mean "date"?`

```jinja
{{ my_var|f }}
```

Shows:

`The filter "f" does not exist. Did you mean "format", "default", "_default", "format_args", "format_args_as_text", "file_excerpt", "format_file", "format_file_from_text", "file_link"?`

Also good for custom filters created in your own domain `st_`:

```jinja
{{ my_var|st_ }}
```

Shows:

`The filter "st_" does not exist. Did you mean "st_friendly_date", "st_site_test_url"?`

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

by stof at 2011/12/04 16:34:27 -0800

I like the idea to provide help to the developer in case of error. You should probably do the same for functions

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

by Tobion at 2011/12/05 05:56:29 -0800

+1 for a combination of string starts with (suggesting domain specific filters, e.g. st_*) and levenshtein for typos

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

by chucktrukk at 2011/12/05 08:13:10 -0800

+1. This would be really helpful.

bumped version to 1.5.0-DEV

prepared the 1.4.0 release

added a unit test test for previous merge

merged branch duo-criativa/master (PR #544)

Commits
-------

11b8689 Refactoring: using && instead of nested if's
7aa6757 Bug correction: Parsing integers large than PHP_INT_MAX was generating trucated token values.
046e4ff Bug correction: Parsing integers large than PHP_INT_MAX was generating trucated token values.

Discussion
----------

Fixed bug that was trucating integers large then PHP_INT_MAX

The following code

{% set sizes = [7077888, 452984832, 28991029248, 1855425871872, 9223372036854775807] %}
{% for i in sizes %}
   {{ i }}
{% endfor %}

was generating the following output

7077888
452984832
2147483647
2147483647
2147483647

With the fix, the output looks like the following on Ubuntu 11.04

7077888
452984832
28991029248
1855425871872
9.2233720368548E+18

Removed trailing white spaces

Help the developer when they specify an invalid filter by providing some alternatives

added missing preserveKeys argument to the reverse filter

fixed macros containing filter tag calls

merged branch hason/arrayloader (PR #542)

Commits
-------

16d7800 fixed a crash when an object with __toString() method is passed as template name

Discussion
----------

fixed a crash when an object with __toString() method is passed as templ...

...ate name

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

by fabpot at 2011/12/01 03:58:47 -0800

Not sure about this one as the phpdoc clearly state that the name is a string. Do you have a specific user case in mind?

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

by hason at 2011/12/01 04:26:47 -0800

I use the class "Twig_Loader_Chain" as template loader in Symfony2 application. One of the embeded loaders is "Twig_Loader_Array". In Symfony2 is used "Symfony\Component\Templating\TemplateReferenceInterface" for internal representation of a template and it causes a crash.

Refactoring: using && instead of nested if's

Bug correction: Parsing integers large than PHP_INT_MAX was generating trucated token values.

Bug correction: Parsing integers large than PHP_INT_MAX was generating trucated token values.

fixed a crash when an object with __toString() method is passed as template name