merged branch kosssi/feature/fix-typo (PR #1090)

This PR was merged into the master branch.

Discussion
----------

fix documentation code block

Commits
-------

e9c1979 fix documentation code block

fix documentation code block

merged branch bamarni/docs-escape (PR #1087)

This PR was merged into the master branch.

Discussion
----------

[doc] switched a BC call

Commits
-------

12a5d02 [doc] switched a BC call

[doc] switched a BC call

bumped version to 1.13.1-DEV

prepared the 1.13.0 release

fixed CHANGELOG

merged branch Tobion/patch-1 (PR #1030)

This PR was merged into the master branch.

Discussion
----------

fix numeric & boolean & float keys in array

Reopened PR of #878

Commits
-------

2c56b70 Merge pull request #3 from tucksaun/patch-1
17d6cc7 [ext] fixed tabs
46012ce [ext] improvements
21f8062 [ext] code cleaning
9347c52 [ext] updated PHP code documenting C code
b41ce60 updated C extension accordingly
e5291e1 rename item to attribute in the exception message because that's the term that is used in twig, e.g. the  function
a93f0dc further improved exception message and distinguish array_call and any_call access
26bac14 added tests for array access with confusable keys
db744d5 fix test that that would not fail if no exception is thrown
9ea16ec refactored getAttribute
6fb5afa improved error message for non-existent or invalid attributes
fdfd506 added changelog entry about fixed boolean array access
da616c5 fixed getAttribute for array access with a boolean or float key
b6acf9d updated CHANGELOG
c4d5b3f added some unit tests for previous merge
e84448b fix numeric keys in array

udpated CHANGELOG

merged branch Seldaek/optim (PR #1077)

This PR was merged into the master branch.

Discussion
----------

Some more optimizations

Please look at commits one by one. Overall this makes the twig_escape_filter almost 20% faster here for happy path (escaping string for html with UTF-8) on 1000 calls. Still pretty darn slow, but faster.

Commits
-------

d886b12 Make happy path (strings) faster by avoiding the cast, same perfs for the other cases
97bf5e5 Avoid unnecessary is_object call
c1c0dcc Use uppercased charsets consistently (as in js/css/html_attr strategies)
a23fa62 Add UTF-8 in uppercase since it is so common
c891c53 Avoid strtolower call at every twig_escape_filter call
61f2b2a Use plain string comparison which is twice as fast
d4eec01 Move html case up since it is the most likely to be hit

merged branch Seldaek/optim2 (PR #1078)

This PR was merged into the master branch.

Discussion
----------

Optimize resume after for loops by using + vs array_merge

The plus operator seems to be almost twice as fast as array_merge, so I think in this case and given it's compiled code not made for humans it's worth it. The only potential problem is if you use ints as keys then it won't have the same behavior as array_merge, but I don't know if twig supports this at all, and especially I doubt it's a good idea.

Commits
-------

d753241 Optimize resume after for loops by using + vs array_merge

Optimize resume after for loops by using + vs array_merge

Make happy path (strings) faster by avoiding the cast, same perfs for the other cases

Avoid unnecessary is_object call

Use uppercased charsets consistently (as in js/css/html_attr strategies)

Add UTF-8 in uppercase since it is so common

Avoid strtolower call at every twig_escape_filter call

Use plain string comparison which is twice as fast

Move html case up since it is the most likely to be hit

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

This PR was merged into the master branch.

Discussion
----------

Fixed php doc in Twig/Extension/StringLoader

Commits
-------

95353cb Fixed php doc in Twig/Extension/StringLoader

merged branch Seldaek/optim (PR #1074)

This PR was merged into the master branch.

Discussion
----------

Optimize initGlobals to do only one merge call

Commits
-------

086088e Optimize initGlobals to do only one merge call

Fixed php doc in Twig/Extension/StringLoader

Optimize initGlobals to do only one merge call

Merge pull request #1073 from fabpot/special-chars

changed ☃ to § in tests (closes #996)

changed ☃ to § in tests (closes #996)

fixed a unit test when run in isolation

merged branch fabpot/named-args (PR #1072)

This PR was merged into the master branch.

Discussion
----------

enforced usage of named arguments after positional ones

When we introduced named arguments, the behavior for when both positional and named arguments were used in a call was not clearly defined.

This PR addresses this issue by following Python rules:

 * No positional argument after named ones
 * No possibility to define an argument with both a positional argument and a named one

This introduces a BC break if people were using positional arguments after named ones, but I would argue that this is never a good idea anyway.

This is an alternative to #995

Commits
-------

135b618 enforced usage of named arguments after positional ones

enforced usage of named arguments after positional ones

added a precedence example (refs #1057)

added missing operators in the precedence docs (closes #1057)

fixed the autoloader for PHP 5.2

simplified code

merged branch hason/tests_autoloader (PR #994)

This PR was merged into the master branch.

Discussion
----------

Fixed autoloader registration for tests if phpunit is installed as package together with Twig

Commits
-------

19869ce Fixed autoloader registration for tests if phpunit is installed as package together with Twig

tweaked docs for the set tag (closes #1029)

merged branch include-x/master (PR #1068)

This PR was merged into the master branch.

Discussion
----------

Fixes typo in documentation, "if" tag

Commits
-------

da635ba Fixes typo

Fixes typo

removed obsolete call to ini_set

merged branch lyrixx/feat-cli (PR #1062)

This PR was merged into the master branch.

Discussion
----------

Debug extension is safe in a cli context

Commits
-------

08b4a73 Debug extension is safe in a cli context

merged branch jezhalford/patch-1 (PR #1067)

This PR was merged into the master branch.

Discussion
----------

Made macro documentation clearer

Previous wording implied (to me at least) that macros *are* PHP functions.

Commits
-------

c155932 Made macro doc clearer

Made macro doc clearer

Previous wording implied (to me at least) that macros *are* PHP functions.

Fixed autoloader registration for tests if phpunit is installed as package together with Twig

Debug extension is safe in a cli context

merged branch weaverryan/patch-1 (PR #1060)

This PR was merged into the master branch.

Discussion
----------

Fixed small typo in link for "pre-built DLL"

Hey!

Very easy - this fixes a link that wasn't generating correctly.

Thanks!

Commits
-------

28673b1 Fixed small typo in link for "pre-built DLL"

Fixed small typo in link for "pre-built DLL"

This fixes a link that wasn't generating correctly.

merged branch cordoval/patch-2 (PR #1056)

This PR was merged into the master branch.

Discussion
----------

improve readability

Commits
-------

a4daa4c update
6a25c5d improve readability

update

improve readability

merged branch jibriss/fix_defined (PR #1051)

This PR was merged into the master branch.

Discussion
----------

Fixed : fatal error when using defined test on an invalid expression

Commits
-------

e285228 Fixed : fatal error when using defined test on an invalid expression

merged branch cordoval/patch-2 (PR #1055)

This PR was merged into the master branch.

Discussion
----------

typo

Commits
-------

e3b9e99 typo

typo

merged branch drewjw81/extension_global_fix (PR #1053)

This PR was squashed before being merged into the master branch (closes #1053).

Discussion
----------

Prevent extensions from clobbering all globals.

It's possible for a poorly written extension to destroy all global variables by returning nothing.  This prevents that and throws an E_WARNING to notify the developer.

Commits
-------

7c8acf7 Prevent extensions from clobbering all globals.

Prevent extensions from clobbering all globals.

removed unused variable

Fixed : fatal error when using defined test on an invalid expression

Merge pull request #3 from tucksaun/patch-1

Updating C extension

[ext] fixed tabs

[ext] improvements

[ext] code cleaning

[ext] updated PHP code documenting C code

bumped version to 1.12.4-DEV

prepared the 1.12.2 release

updated CHANGELOG for previous commit

Fixed twig filesystemloader security issue + test (closes #1026)

tweaked previous merge

merged branch ThatGuyBen/patch-1 (PR #1048)

This PR was merged into the master branch.

Discussion
----------

Update json_encode.rst

a more-useful description of the "options" argument >_>

Commits
-------

742043f Update json_encode.rst

Update json_encode.rst

a more-useful description of the "options" argument >_>

updated CHANGELOG

merged branch ShadowPrince/master (PR #1042)

This PR was squashed before being merged into the master branch (closes #1042).

Discussion
----------

Registration a Twig_SimpleFunction after extensions initialized throw not LogicException, but Fatal Error

Registration a Twig_SimpleFunction after extensions initialized throw not LogicException, but Fatal Error:

$environment->addFunction(new Twig_SimpleFunction(...))
after 
$environment->render(...)
throws Fatal Error: 
object of Twig_SimpleFunction could not be converted to string

Commits
-------

92e97b9 Registration a Twig_SimpleFunction after extensions initialized throw not LogicException, but Fatal Error

Registration a Twig_SimpleFunction after extensions initialized throw not LogicException, but Fatal Error

tweaked previous merge

merged branch beryllium/patch-1 (PR #1032)

This PR was merged into the master branch.

Discussion
----------

Adding a section about the slice filter to doc/tags/for.rst

This code example shows how to loop over a subset of values in order to implement one common use-case of "break" in PHP. I decided to use the "full" notation of slice rather than the sugared version (that example would just be "for user in users[:10]")

Commits
-------

202641a Adding link to slice filter per Stof's suggestion
8ebbaaa Adding a section about the slice filter

merged branch ricardclau/typo_sort_doc (PR #1039)

This PR was merged into the master branch.

Discussion
----------

Minor typo in sort filter documentation

I've found this small typo while working on shuffle filter documentation in https://github.com/fabpot/Twig/pull/1038

I decided to create separate PR in case https://github.com/fabpot/Twig/pull/1038 got rejected

Best regards!

Commits
-------

1443b76 minor typo in sort documentation

merged branch tommygnr/fix-docs-batch-filter (PR #1040)

This PR was merged into the master branch.

Discussion
----------

Fix filename for batch filter documentation

The documentation on the website for the batch filter currently 404s See: http://twig.sensiolabs.org/doc/filters/batch.html

I believe this is because the filename is incorrect. This PR fixes that.

Commits
-------

bb05942 Fix filename for batch filter documentation

Fix filename for batch filter documentation

minor typo in sort documentation

merged branch tucksaun/fix-5.2-TSRM (PR #1037)

This PR was merged into the master branch.

Discussion
----------

Revert "merged branch pierrejoye/master (PR #855)"

This reverts commit 347bc81d6ed8e2bb793236aa9917c7e83f81c2b3

This commit make compilation failed for PHP5.2 with thread-safe enabled (see https://travis-ci.org/fabpot/Twig/jobs/5523865)

Commits
-------

4383388 Revert "merged branch pierrejoye/master (PR #855)"

Revert "merged branch pierrejoye/master (PR #855)"

This reverts commit 347bc81d6ed8e2bb793236aa9917c7e83f81c2b3, reversing
changes made to 5d7baaf767ec3b8f6599738e7c98bde6c25cf7c1.

merged branch specialguest/patch-1 (PR #1034)

This PR was merged into the master branch.

Commits
-------

c6ef23a Update striptags.rst

Discussion
----------

Update striptags.rst

Hello

I just changed
{% some_html|striptags %}
to
{{ some_html|striptags }}

Regards

Julien

Adding link to slice filter per Stof's suggestion

Update striptags.rst

Adding a section about the slice filter

This code example shows how to loop over a subset of values in order to implement one common use-case of "break" in PHP. I decided to use the "full" notation of slice rather than the sugared version (that example would just be "for user in users[:10]")

updated C extension accordingly

rename item to attribute in the exception message because that's the term that is used in twig, e.g. the  function

further improved exception message and distinguish array_call and any_call access

added tests for array access with confusable keys

array keys like boolean, numeric but string, floats, null

fix test that that would not fail if no exception is thrown

refactored getAttribute

improved error message for non-existent or invalid attributes

added changelog entry about fixed boolean array access

fixed getAttribute for array access with a boolean or float key

updated CHANGELOG

added some unit tests for previous merge

fix numeric keys in array

merged branch pborreli/typos (PR #1028)

This PR was merged into the master branch.

Commits
-------

49488b8 Fixed typos

Discussion
----------

Fixed typos

Fixed typos

merged branch carlwitt/patch-1 (PR #1022)

This PR was merged into the master branch.

Commits
-------

59d0eed Update merge.rst

Discussion
----------

Update merge.rst

Typo in first line:
The merge filter merges an array with [removed: the] another array

Update merge.rst

Typo in first line:
The merge filter merges an array with [removed: the] another array

fixed phpdoc

merged branch Tobion/buildquery (PR #1009)

This PR was squashed before being merged into the master branch (closes #1009).

Commits
-------

fcf9525 let url_encode filter also accept array of query parameters

Discussion
----------

let url_encode filter also accept array of query parameters

BC break: no
feature addition: yes
tests pass: yes
documentation: yes

This is also supported in Jinja and so it's consistent: [jinja urlencode](http://modular.math.washington.edu/home/wstein/www/home/bjarke/sage-4.4.4/local/LIB/python/site-packages/Jinja-1.2-py2.6-linux-x86_64.egg/docs/html/builtins.html)

let url_encode filter also accept array of query parameters