This PR was merged into the 1.x branch.
Discussion
----------
Added pssibility to reset the profiler
In order to implement symfony/symfony#18244 properly, I needed the ability to reset a Twig profile.
Commits
-------
ea922bb6 Added pssibility to reset the profiler.
This PR was merged into the 1.x branch.
Discussion
----------
Ensure that the "use" TokenParser returns a node instance
Alternative to #2539, fixes#2538.
Commits
-------
421d8722 ensured that the use TokenParser returns a node instance
This PR was squashed before being merged into the 1.x branch (closes#2544).
Discussion
----------
simplified tests as min version if now 5.3.3
fixes#2526
Commits
-------
a3548a6e fixed CS
90a07d16 simplified tests as min version if now 5.3.3
This PR was submitted for the 2.x branch but it was squashed and merged into the 1.x branch instead (closes#2536).
Discussion
----------
show how to use a variable as hash map key
currently the documentation does not explain how it's possible to use a variable as key in a hash map
Commits
-------
ef6c9505 show how to use a variable as hash map key
This PR was merged into the 1.x branch.
Discussion
----------
Throw exception on circular reference detection
Out-the-box this isn't common... but into Symfony framework this kind of issue can happen frequently when we're overriding a template from third-party bundle (see main issue https://github.com/symfony/symfony/issues/17557)
```twig
{# app/Resources/AcmeBlogBundle/views/Blog/index.html.twig #}
{% extends '@AcmeBlog/Blog/index.html.twig' %}
{% block title 'New Title' %}
```
This exception avoids inifite loop or maximum execution time for this case, by detecting a recursive loading when a template extends from itself, thinking to override the parent template which has the same namespace + template name.
Commits
-------
074e1fbe Throw exception on circular reference detection
This PR was submitted for the 2.x branch but it was merged into the 1.x branch instead (closes#2531).
Discussion
----------
Update .travis.yml to deal with missing xdebug config
lets see if this fixes the nightly test
Commits
-------
1097685b Update .travis.yml
This PR was submitted for the 2.x branch but it was merged into the 1.x branch instead (closes#2515).
Discussion
----------
Update internals.rst
Missing an opening paren
Commits
-------
b8967bd3 Update internals.rst
This PR was merged into the 1.x branch.
Discussion
----------
Fix deprecation when using Twig_Profiler_Dumper_Html
Commits
-------
aebec7d8 fixed deprecation when using Twig_Profiler_Dumper_Html
This PR was submitted for the 2.x branch but it was merged into the 1.x branch instead (closes#2512).
Discussion
----------
Issue in documentation for twig number formatter
The explaination below my change is correct. Just the output is not correct
Commits
-------
54f0f7ae Create number_format.rst
This PR was merged into the 1.x branch.
Discussion
----------
Add support for runtime loaders to IntegrationTestCase
Adds the ability to provide runtime loaders in `IntegrationTestCase`. Otherwise, there's no way to test extensions that use them.
Fixes#2492
Commits
-------
ff96b34a Add support for runtime loaders to IntegrationTestCase
This PR was merged into the 1.x branch.
Discussion
----------
Move class_exists() at the bottom of files
Fixes#2503Closes#2506
Commits
-------
95e912b0 Move class_exists() at the bottom of files
This PR was merged into the 1.x branch.
Discussion
----------
Use class_exists instead of require
I think this is required for https://github.com/symfony/symfony/pull/23073 to pass. It's probably very similar to https://github.com/symfony/symfony/pull/22657.
A simple reproducer:
```php
<?php
use Twig\Node\Node;
require_once __DIR__ . '/vendor/autoload.php';
new Node();
new \Twig_Node();
```
[Without this patch](https://travis-ci.org/symfony/symfony/jobs/239712676#L2100):
```php
PHP Fatal error: Cannot declare class Twig_Node, because the name is already in use in vendor/twig/twig/lib/Twig/Node.php on line 20
```
With, everything works fine, whatever the two calls order.
Commits
-------
2c174e40 Use class_exists instead of require
This PR was merged into the 1.x branch.
Discussion
----------
Use class_exists instead of require to play nice with inlining
Should fix#2495
Commits
-------
dfe0dc2b Use class_exists instead of require to play nice with inlining
This PR was merged into the 1.x branch.
Discussion
----------
[Doc] Fixed documentation about 'Definition vs Runtime'
Commits
-------
c0b5eb9f [Doc] Fixed documentation about 'Definition vs Runtime'
Fabien Potencier
Alexander M. Turek
Tobias Olry
Grégoire Pineau
Yonel Ceruto
SpacePossum
Steven Pribilinskiy
simsim2006
Edi Modrić
Nicolas Grekas
Maxime Steinhausser