Fixed usage of LimitIterator

bumped version to 1.16.2-DEV

prepared the 1.16.1 release

updated CHANGELOG

bug #1530 Improved error reporting in a sandboxed template (hason)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Improved error reporting in a sandboxed template

Commits
-------

58efc42 Improved error reporting in a sandboxed template

updated CHANGELOG

bug #1528 Fixed guessing a template info for exceptions (hason)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Fixed guessing a template info for exceptions

Commits
-------

42cd697 Fixed guessing a template info for exceptions

updated CHANGELOG

bug #1527 Fixed resetting debug info in compiler (hason)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Fixed resetting debug info in compiler

Commits
-------

0e7cec5 Fixed resetting debug info in compiler

Improved error reporting in a sandboxed template

Fixed guessing a template info for exceptions

Fixed resetting debug info in compiler

updated CHANGELOG

bug #1470 Fixed "in" operator (hason)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Fixed "in" operator

Commits
-------

9b6497f Fixed "in" operator

udpated CHANGELOG

feature #1503 Use LimitIterator on Iterable objects within the slice filter (Anthony Sterling)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Use LimitIterator on Iterable objects within the slice filter

Previously the `slice` filter would create an array of items if the subject to be sliced implemented Traversable, this PR instead passes the subject to LimitIterator first - but only if `$start` and `$length` are positive values as LimitIterator does not support the "n'th from end" behaviour.

Our use case is that we have an API client, which we pass to slice, that implements Traversable but lazy loads the items to be returned via an API when `Iterator::current()` is called. Here a quick example:-

```
{% for item in client.items | slice(0,5) %}
    {{ item.name }}
{% endfor %}

<img src="/assets/useless-sodding-banner.jpg" />

{% for item in client.items | slice(5,10) %}
    {{ item.name }}
{% endfor %}
```

Current behaviour would mean that the client would issue 15 requests to the API, 5 for the first block, and 10 for the second (but only return items 5 to 10). This PR means only 10 requests are made, 5 in each block.

I'd imagine this would lower memory usage too.

Commits
-------

fdddb1c Use LimitIterator on Iterable objects within the splice filter

fixed typo

deprecated Node::toXml()

minor #1509 Update convert_encoding.rst: wrong argument order (binarious)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Update convert_encoding.rst: wrong argument order

The arguments are in incorrect order. The first should be `to` and the second should be `from` - just like the text describes it.

Commits
-------

4e1d9a7 Update convert_encoding.rst

fixed some assertions

bug #1500 fix inconsistent response from twig_slice (rodnaph)

This PR was squashed before being merged into the 1.16-dev branch (closes #1500).

Discussion
----------

fix inconsistent response from twig_slice

twig_slice can return a boolean when slicing a string outside of its bounds and mbstring is not available.  this would cause a knock-on error in twig_first/twig_last when calling current()

this (as suggested) adds a cast to emulate the behaviour of mb_substr for substr.

Commits
-------

c740060 fix inconsistent response from twig_slice

fix inconsistent response from twig_slice

minor #1520 date() documentation correction (brandonkelly)

This PR was merged into the 1.16-dev branch.

Discussion
----------

date() documentation correction

The date() docs were incorrectly stating that the argument should be in a format supported by PHP’s [date()](http://php.net/manual/en/function.date.php) function. That’s true of the |date filter which is used to _format_ dates, but not the date() function, which is used to _create_ new dates.

I double-checked the code and whatever you pass into date() ultimately will get passed to a [DateTime constructor](http://php.net/manual/en/datetime.construct.php), which accepts an actual date string, formatted in one of PHP’s [date and time format](http://php.net/manual/en/datetime.formats.php), which is a different beast than the actual format definition string you might pass into PHP’s date() function.

Commits
-------

2f5a927 date() documentation correction

date() documentation correction

Update convert_encoding.rst

The arguments were in incorrect order. The first should be `to` and the second should be `from` - just like the text describes it.

Use LimitIterator on Iterable objects within the splice filter

minor #1492 add PHP 5.6 to .travis.yml (sasezaki)

This PR was merged into the 1.16-dev branch.

Discussion
----------

add PHP 5.6 to .travis.yml

Commits
-------

ed7004b add PHP 5.6 to .travis.yml

add PHP 5.6 to .travis.yml

minor #1474 Update installation.rst (soatok)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Update installation.rst

Refer to PGP signature verification in the installation instructions for tarballs.

Commits
-------

3c48cb1 Update installation.rst

minor #1487 Added note about array_merge php function (pulzarraider)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Added note about array_merge php function

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT

Just a small doc improvement.

Commits
-------

f712b8f Added note about array_merge php function

minor #1484 Delimit comment in template using correct syntax (johnberberich)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Delimit comment in template using correct syntax

Enclose the comment in '{# #}' instead of beginning line with '#'.

Commits
-------

4e1515c Delimit comment in template using correct syntax

Added note about array_merge php function

Delimit comment in template using correct syntax

Enclose the comment in '{# #}' instead of beginning line with '#'.

minor #1475 Improved exception message while trying to access an attribute from an empty array (phansys, tucksaun)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Improved exception message while trying to access an attribute from an empty array

Rebased #1439 in master and updated twig extension

Commits
-------

759a77d [Ext] updating Twig extension for previous commit
15ce450 Improved exception message while trying to access an attribute from an empty array.

[Ext] updating Twig extension for previous commit

Improved exception message while trying to access an attribute from an empty array.

Fixed "in" operator

updated CHANGELOG

Update installation.rst

Refer to PGP signature verification in the installation instructions for tarballs.

changed the way we check for HHVM

minor #1471 Fixed a test on hhvm (hason)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Fixed a test on hhvm

Commits
-------

0c169ce Fixed a test on hhvm

Fixed a test on hhvm

bug #1468 Fixed "starts with" a "ends with" operators for non-string arguments (hason)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Fixed "starts with" a "ends with" operators for non-string arguments

Commits
-------

4c26981 Fixed "starts with" a "ends with" operators for non-string arguments

Fixed "starts with" a "ends with" operators for non-string arguments

bug #1452 Fixed 'starts with' operator for empty needle (hason)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Fixed 'starts with' operator for empty needle

Fixes #1432

Commits
-------

790305a Fixed 'starts with' operator for empty needle

Fixed 'starts with' operator for empty needle

added more docs for the if tag

fixed the attribute() function when passing a variable for the arguments

minor #1465 Some minor cleanup (JeroenDeDauw)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Some minor cleanup

Commits
-------

e6da6bb Restore unused var
d8e1077 Get rid of naming clash
3e50672 Remove unused local vars
291141f Remove unused var
9921554 Fix type hint
1d2b877 Fix type hint

minor #1462 Simplified Twig_Node_Include (hason)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Simplified Twig_Node_Include

Commits
-------

632fe90 Simplified Twig_Node_Include

minor #1458 Add variables types in error message of twig_array_merge (inalgnu)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Add variables types in error message of twig_array_merge

Commits
-------

ea6676d add variables types in error message of twig_array_merge

Restore unused var

Get rid of naming clash

Static code analysis tool did not like this.
Looks like it was wrong before. Tests unchanged though.

Remove unused local vars

Remove unused var

Fix type hint

In PHPStorm the inferred type was 'A'

Fix type hint

PHPStorm wrongly inferred the type 'Unicode'

Simplified Twig_Node_Include

add variables types in error message of twig_array_merge

minor #1457 Fixed the `matches` operator code example (brandonkelly)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Fixed the `matches` operator code example

A customer just pointed out that your `matches` operator code example wasn't working for them, and it ended up being because its backslashes themselves needed to be escaped by additional backslashes. (I've verified this.)

While I was editing it, I also switched to forwardslash delimiters on the regex, since those are much more commonly used than curly brackets. (I hadn't even been aware that curly brackets are allowed.) To back that assertion up, [PHP's official preg_quote() docs](http://php.net/manual/en/function.preg-quote.php) state:

> The `/` is the most commonly used delimiter.

Commits
-------

33dcbc8 Fixed the `matches` operator code example

Fixed the `matches` operator code example

It wasn't working because the backslashes themselves needed to be escaped by additional backslashes. Also switched to using forwardslashes for the regex delimiters rather than curly brackets.

bug #1453 Fix for mb function overload mb_substr acting different (1emming)

This PR was squashed before being merged into the 1.16-dev branch (closes #1453).

Discussion
----------

Fix for mb function overload mb_substr acting different

When bug hunting https://github.com/fabpot/Twig/issues/1428 I ran into an issue.
I started out by changing my `php.ini` for the `cli` enviroment, I've added:
`mbstring.func_overload = 2`
Later I tested with adding and removing:
`mbstring.internal_encoding = 'UTF-8'`

It turns out `mb_substr` returns `false` and not `''` when doing something like this:

```php
$item = '';
$start = -1;
$length = 1;
$charset = 'UTF-8';
$a = mb_substr($item, $start, $length, $charset);
var_dump($a);
die;
```

I'm not sure this will fix 1428 since I don't have enough information for this, but this PR should at least fix one issue.

Commits
-------

214fe52 Fix for mb function overload mb_substr acting different

Fix for mb function overload mb_substr acting different

fixed some typos

minor #1333 Added disambiguation on using raw in expressions (OwlyCode)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Added disambiguation on using raw in expressions

I recently got in trouble using the ``raw`` filter in a ternary, leading to a value being escaped and me not expecting it. The code was :

```
{{ foo|striptags|length > 250 ? foo|striptags|slice(0, 250) ~ '...' : foo|raw }}
```

When foo's length was under 250 characters, the result of this expression was escaped. Here is the updated documentation to explain why.

Commits
-------

cec2b57 Added disambiguation on using raw in expressions

minor #1431 Typo in deprecated.rst (barryvdh)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Typo in deprecated.rst

Commits
-------

06ca7f9 Typo in deprecated.rst

bumped version to 1.16.1-DEV

Typo in deprecated.rst

prepared the 1.16.0 release

replaced deprecated divisibleby by divisible by in tests

replaced deprecated sameas by same as in tests

fixed some deprecation notices

fixed the deprecated docs about PEAR

fixed potential exception

minor #1426 Add 'PHP-twig for atom' package to IDEs support (zoltanradics)

This PR was squashed before being merged into the 1.16-dev branch (closes #1426).

Discussion
----------

Add 'PHP-twig for atom' package to IDEs support

Since Atom.io editor is growing, i would add this package to the list for people who would like to edit Twig in Atom.

Commits
-------

c90d469 Add 'PHP-twig for atom' package to IDEs support

Add 'PHP-twig for atom' package to IDEs support

minor #1422 Update slice.rst (1emming)

This PR was merged into the 1.16-dev branch.

Discussion
----------

Update slice.rst

Please state that Twig `slice` is  multi-byte safe since it uses `mb_substr` :)

Commits
-------

87165e7 Update slice.rst

Update slice.rst

added a note in the docs about the fact that installing the C ext is optional

feature #1390 fixed url_encode when raw is false and URL is an array (Tobion, fabpot)

This PR was merged into the 1.16-dev branch.

Discussion
----------

fixed url_encode when raw is false and URL is an array

alternative for #1388

Commits
-------

1930600 fixed url_encode when raw is false and URL is an array
49ba6b4 removed parameter from url_encode filter

fixed url_encode when raw is false and URL is an array

removed parameter from url_encode filter

the filter now only uses the more appropriate rawurlencode() which is also more consistent with the escape('url') filter

updated CHANGELOG

bug #1370 Fixed inheritance in a 'use'-hierarchy (hason)

This PR was merged into the 1.15-dev branch.

Discussion
----------

Fixed inheritance in a 'use'-hierarchy

Fixes #1365

Commits
-------

9e0907e Fixed inheritance in a 'use'-hierarchy

updated CHANGELOG

bug #1382 Added isSandbox check around the __toString check in Sandbox extension (Scott Smith, smitherz82)

This PR was merged into the 1.15-dev branch.

Discussion
----------

Added isSandbox check around the __toString check in Sandbox extension

The `__toString` policy check currently still happens when the sandbox is disabled

Commits
-------

3ce4202 Added test for sandbox __toString when not enabled
8dfa432 Added isSandbox check around the __toString check

minor #1403 Refactor namespace/shortname parsing into own method (anlutro)

This PR was merged into the 1.15-dev branch.

Discussion
----------

Refactor namespace/shortname parsing into own method

I've written a custom file loader in order for the namespace syntax to match across different resource loading classes. To achieve this I have to replace the loadTemplate method, which includes a lot of logic. While this logic may not be likely to change anytime soon, I'd feel better if the namespace parsing was extracted into its own method so I can replace just that bit.

Commits
-------

9dd3bb8 Refactor namespace/shortname parsing into own method

minor #1409 Update json_encode.rst (matthieuprat)

This PR was merged into the 1.15-dev branch.

Discussion
----------

Update json_encode.rst

Commits
-------

1b3ac0a Update json_encode.rst

minor #1413 [Doc]Put the return of the split filter in a variable (n1c01a5)

This PR was squashed before being merged into the 1.15-dev branch (closes #1413).

Discussion
----------

[Doc]Put the return of the split filter in a variable

Commits
-------

0e228c7 [Doc]Put the return of the split filter in a variable

[Doc]Put the return of the split filter in a variable

Update json_encode.rst

Refactor namespace/shortname parsing into own method

Allows you to more easily create a custom file loader with a custom
namespace syntax.

minor #1406 Improve the description of the 'same as' test (adamelso)

This PR was merged into the 1.15-dev branch.

Discussion
----------

Improve the description of the 'same as' test

This description of the `same as` test is based on the description in the docblock for `Twig_Node_Expression_Test_Sameas`. It reads better than the one currently used in the docs.

Commits
-------

e4b7db1 Improve the description of the 'same as' test

Improve the description of the 'same as' test

This description of the 'same as' test is based on the description in the docblock for Twig_Node_Expression_Test_Sameas. It reads better than the one currently used in the docs.

minor #1401 [Twig][Tests][Fixtures][filters] Added empty string tests for "first" and "last" filters (ureimers)

This PR was squashed before being merged into the 1.15-dev branch (closes #1401).

Discussion
----------

[Twig][Tests][Fixtures][filters] Added empty string tests for "first" and "last" filters

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | none
| License       | MIT

This was already fixed with with https://github.com/fabpot/Twig/commit/6003ae43b92cfa7e1ddec17262733355e0bf435a but to make sure there won't be any regression bugs in the future. This is an edge case but the first and the last element of an empty string are empty strings too.
Before the above mentioned PR this threw a notice `Uninitialized string offset: 0`.

Commits
-------

0db5c31 [Twig][Tests][Fixtures][filters] Added empty string tests for "first" and "last" filters

[Twig][Tests][Fixtures][filters] Added empty string tests for "first" and "last" filters

minor #1399 Fixed typo (WouterJ)

This PR was merged into the 1.15-dev branch.

Discussion
----------

Fixed typo

Commits
-------

49de7a6 Fixed typo

Fixed typo

added some missing tests

minor #1398 Remove unused constructor arguments (rybakit)

This PR was merged into the 1.15-dev branch.

Discussion
----------

Remove unused constructor arguments

There are no line & tag arguments in `Twig_Node_Module::__construct()`.

Commits
-------

5b5741a Remove unused constructor arguments