This PR was squashed before being merged into the 3.x branch.
Discussion
----------
Fix mistake in docs for `keys` filter
Commits
-------
deb37c30c4 Fix mistake in docs for `keys` filter
This PR was squashed before being merged into the 3.x branch.
Discussion
----------
[String] Add SpanishInflector support for singular and plural
This PR implement a new Inflector using Spanish (ISO Code = es) implemented in this PR symfony/symfony#58228.
* I sort the list of supported Inflectors to follow some rule (alphabet?)
* I just change invalid test from "it" to "qq" (qq doesn't exist as valid ISO code, but it is italian... don't be evil if anyone want to create the italian inflector 😅 )
Commits
-------
290a923a42 [String] Add SpanishInflector support for singular and plural
This PR was merged into the 3.x branch.
Discussion
----------
Remove duplicate test case
The test cases in line `7` and `9` were the same.
Commits
-------
d59cc8baa5 Remove duplicate test case
This PR was merged into the 3.x branch.
Discussion
----------
Add `$this` return type to Template::unwrap
See https://phpstan.org/writing-php-code/phpdoc-types#static-and-%24this
This way, PHPStan understands that the returned template is the exact same instance, and not just a Template.
Commits
-------
92e4989002 Add `$this` return type to Template::unwrap
This PR was squashed before being merged into the 3.x branch.
Discussion
----------
Add link to TwigQI in docs "You might also be interested in" section
As proposed by Fabien last Thursday at Evolve. Merci beaucoup!
Commits
-------
4a0568521a Add link to TwigQI in docs "You might also be interested in" section
This PR was merged into the 3.x branch.
Discussion
----------
Add return type to compiled macro
This makes it easier for TwigStan to analyze the return type.
Commits
-------
1e7c719e24 Add return type to compiled macro
This PR was merged into the 3.x branch.
Discussion
----------
do not drop none digit characters from generated variable names
I was wondering if it was really intended to cast the var name generated by the compiler to an int as done in #4398. Doesn't this create the risk of variable names being reused at some point?
Commits
-------
5b0209ffa3 do not drop none digit characters from generated variable names
This PR was merged into the 3.x branch.
Discussion
----------
Add return type to getDebugInfo
Commits
-------
0bb49dc40f Add return type to getDebugInfo
This PR was merged into the 3.x branch.
Discussion
----------
Rename Node classes related to variables
Whenever I work on Twig internals, it's always complicated to reason about variable names, probably because the class names are confusing. This PR is an attempt to find "better" and more explicit names.
This PR does the following renaming:
* `NameExpression` to `Variable\ContextVariable`
Represents the value of a context variable like `$context[VAR] ?? null`
* `AssignNameExpression` to `Variable\AssignContextVariable`
Represents a context variable assignment like in `$context[VAR] = `
* `TempNameExpression` to `Variable\LocalVariable`
Represents a "private" local variable like `$_l111`
Commits
-------
fc15e7ccbc Rename Node classes related to variables
This PR was squashed before being merged into the 3.x branch.
Discussion
----------
Documentation for types tag uses Twig types in examples instead of PHP
Specifically, "bool" => "boolean" and "int" => "number".
This aligns the `types` documentation with templates.rst. See #4362
Thanks, `@alexander`-schranz!
Commits
-------
f3e0a00cf0 Documentation for types tag uses Twig types in examples instead of PHP
This PR was squashed before being merged into the 3.x branch.
Discussion
----------
Rework macros handling
Commits
-------
f1481be00f Remove MacroAutoImportNodeVisitor
72693884c4 Simplify code
1105964873 Rework macros handling
This PR was merged into the 3.x branch.
Discussion
----------
reduce the deprecation noise when more than one argument is passed
When more than one argument is passed to parseArguments(), one deprecation is triggered for each argument. Since passing any argument is deprecated the number of triggered deprecations can be reduced for more than one argument being passed.
Commits
-------
26256fc488 reduce the deprecation noise when more than one argument is passed
When more than one argument is passed to parseArguments(), one deprecation
is triggered for each argument. Since passing any argument is deprecated
the number of triggered deprecations can be reduced for more than one
argument being passed.
This PR was merged into the 3.x branch.
Discussion
----------
Check reserved names in TempNameExpression
Commits
-------
66658a3c1d Check reserved names in TempNameExpression
This PR was merged into the 3.x branch.
Discussion
----------
Fix AssignNameExpression to forbid using names that won't work
Commits
-------
9af72e6323 Fix AssignNameExpression to forbid using names that won't work
This PR was merged into the 3.x branch.
Discussion
----------
Do not allow : as macro definition separator
Commits
-------
831e69b77e Do not allow : as macro definition separator
This PR was merged into the 3.x branch.
Discussion
----------
Add support for named arguments on macro calls and dot operator arguments
Closes#929Closes#3635
Commits
-------
223d36bbae Add support for named arguments on macro calls and dot operator arguments
Fabien Potencier
Nicolas Grekas
Ruud Kamphuis
Dennis Tobar
Jacob Dreesen
Jeroen Versteeg
Christian Flothmann