This PR was submitted for the 3.2 branch but it was merged into the 2.7 branch instead (closes#21485).
Discussion
----------
[Process] Non ASCII characters disappearing during the escapeshellarg
If the LC_CTYPE is not set at UTF-8, the escapeshellarg() function will remove every non-ascii characters.
As it's usual in europe to have directories with non-ascii chars in their name (ex : ~/Vidéos) the function should throw an exception if we're trying to submit it an argument containing non-ascii param and the LC_CTYPE is not set to use UTF-8
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? |
| Fixed tickets |
| License | MIT
| Doc PR |
I had this issue while using the lib ffmpeg and giving it a path like "~/Vidéos" the "é" chars was disappearing from the command giving a RuntimeException.
The problem was my LC_CTYPE that wasn't set properly, I believe an exception should be raised before the RuntimeException to warn the user of that behavior
Commits
-------
3779f3f [Process] Non ASCII characters disappearing during the escapeshellarg
This PR was merged into the 2.7 branch.
Discussion
----------
[FrameworkBundle][Console] JsonDescriptor: Respect original output
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | no
| Fixed tickets | N/A
| License | MIT
| Doc PR | N/A
I'm suggesting this one, because I recently pushed some changes to the descriptors, and of course, I'm not editing expected output fixtures by hand, but by dumping the real output to fixture files. But it's tiring to get false positive diffs when reviewing it. Descriptor tests already are painful enough 😅
This PR respects the way elements are actually output. If it's ok to you, I'll submit some other PRs to upper branches, because there are more issues regarding this (items order for instance).
If it causes too much troubles getting this in sync with upper branches, let's close this and never talk about it anymore 😄
Commits
-------
08dd70b [FrameworkBundle][Console] JsonDescriptor: Respect original output
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes#21495).
Discussion
----------
Add HEADER_FORWARDED to setTrustedHeaderName docs
| Q | A
| ------------- | ---
| Branch? | 3.2
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | -
| License | MIT
| Doc PR | -
<!--
- Bug fixes must be submitted against the lowest branch where they apply
(lowest branches are regularly merged to upper ones so they get the fixes too).
- Features and deprecations must be submitted against the master branch.
- Please fill in this template according to the PR you're about to submit.
- Replace this comment by a description of what your PR is solving.
-->
Commits
-------
b46276e Add HEADER_FORWARDED to setTrustedHeaderName docs
This PR was submitted for the master branch but it was merged into the 3.2 branch instead (closes#21493).
Discussion
----------
[DoctrineBridge] Remove dead code in DoctrineOrmExtension
| Q | A
| ------------- | ---
| Branch? | 3.2
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| License | MIT
After optimization in #20312 and #18359
Commits
-------
0cd8bf82bc Remove dead code
This PR was merged into the 2.7 branch.
Discussion
----------
Enable dump() in autoload-dev
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | -
| License | MIT
| Doc PR | -
I miss `dump()` badly when working on Symfony itself.
Commits
-------
89e0088 Enable dump() in autoload-dev
This PR was merged into the 2.8 branch.
Discussion
----------
[FrameworkBundle] add missing functional Serializer test case
| Q | A
| ------------- | ---
| Branch? | 2.8
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
This is needed to make the test introduced in #21370. It basically backports the functional test config as introduced by @dunglas in #20480.
Commits
-------
24243ac add missing functional Serializer test case
* 2.8:
[Console] Fix too strict test
[FrameworkBundle] Execute the PhpDocExtractor earlier
[validator] Updated croatian translation
ignore invalid cookies expires date format
[TwigBundle] Fix the name of the cache warming test class
[Console] Fix TableCell issues with decoration
Add missing pieces in the upgrade guide to 3.0
* 2.7:
[Console] Fix too strict test
[validator] Updated croatian translation
ignore invalid cookies expires date format
[TwigBundle] Fix the name of the cache warming test class
[Console] Fix TableCell issues with decoration
Add missing pieces in the upgrade guide to 3.0
This PR was merged into the 2.7 branch.
Discussion
----------
[Console] Fix too strict test
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | -
| License | MIT
| Doc PR | -
required to make #21474 green in cross versions tests
Commits
-------
ee4b3e2712 [Console] Fix too strict test
This PR was submitted for the master branch but it was merged into the 2.8 branch instead (closes#21370).
Discussion
----------
[FrameworkBundle] Execute the PhpDocExtractor earlier
| Q | A
| ------------- | ---
| Branch? | master
| Bug fix? | yes, but safer to apply on master
| New feature? | no
| BC breaks? | is changing a priority a bc break?
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | https://github.com/symfony/symfony/issues/21367
| License | MIT
| Doc PR |
Fixes https://github.com/symfony/symfony/issues/21367.
> I wonder if this is logical to execute the PhpDocExtractor after the ReflectionExtractor: when you use phpdocs it's because they are more precise than php type hints.
This causes an issue in NelmioApiDocBundle, for example you can't use int[] with a setter as the type mixed[] will be returned instead of int[].
>
> ~~Would you accept bumping its priority to -999?~~
This PR changes the priority of the `ReflectionExtractor` to `-1002` to make sure it is executed after the `PhpDocExtractor`.
Commits
-------
0425e0549b [FrameworkBundle] Execute the PhpDocExtractor earlier
This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes#21449).
Discussion
----------
[Validator] Added missing Croatian translation
| Q | A
| ------------- | ---
| Branch? | master
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Commits
-------
5cd7931c24 [validator] Updated croatian translation
This PR was merged into the 2.7 branch.
Discussion
----------
[BrowserKit] ignore invalid cookies expires date format
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #15656
| License | MIT
| Doc PR |
Commits
-------
f19788dd2e ignore invalid cookies expires date format
This PR was merged into the 3.2 branch.
Discussion
----------
[HttpKernel] DebugHandlersListener micro optimize
| Q | A
| ------------- | ---
| Branch? | 3.2 (lowest version introduced)
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| License | MIT
For ref.: [php_sapi_name()](http://php.net/manual/en/function.php-sapi-name.php) returns the const `PHP_SAPI`.
This a PR out of courtesy to see if the CS/Fixer/Rule that found this case should be added to the SF set by default.
Commits
-------
f9f862f Update DebugHandlersListener.php
This PR was merged into the 2.7 branch.
Discussion
----------
[TwigBundle] Fix the name of the cache warming test class
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | n/a
| License | MIT
| Doc PR | n/a
Rename the test class according to PSR-4.
Commits
-------
2bc4d3b2c1 [TwigBundle] Fix the name of the cache warming test class
This PR was merged into the 2.8 branch.
Discussion
----------
[FrameworkBundle] Remove useless checks in descriptors
| Q | A
| ------------- | ---
| Branch? | 2.8
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | n/a
| License | MIT
| Doc PR | n/a
As pointed out by @ogizanagi in https://github.com/symfony/symfony/pull/21315#discussion_r96476201, some code in FrameworkBundle's descriptors is useless because the bundle works only with DI 2.8.
Commits
-------
c759345781 [FrameworkBundle] Remove useless checks in descriptors
This PR was merged into the 2.7 branch.
Discussion
----------
Fix upgrade guide
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| License | MIT
This merges back the `UPGRADE-3.0.md` guide from master back up to 2.7. There were a lot of missing parts and a few duplications. I fear that also the other guides have the same issues.
Commits
-------
bd0c206a32 Add missing pieces in the upgrade guide to 3.0
* 3.1:
fixed typo
fixed composer.json
always check for all fields to be mapped
clarify exception when no args are configured
[PropertyAccess] Handle interfaces in the invalid argument exception
[DI] Fix defaults overriding empty strings in AutowirePass
[Debug] Workaround "null" $context
[Debug] Remove $context arg from handleError(), preparing for PHP 7.2
[Routing] Fix BC break in AnnotationClassLoader defaults attributes handling
Fix tests with ICU 57.1
Fix the condition checking the minimum ICU version
* 2.8:
always check for all fields to be mapped
clarify exception when no args are configured
[PropertyAccess] Handle interfaces in the invalid argument exception
[DI] Fix defaults overriding empty strings in AutowirePass
[Debug] Workaround "null" $context
[Debug] Remove $context arg from handleError(), preparing for PHP 7.2
[Routing] Fix BC break in AnnotationClassLoader defaults attributes handling
Fix tests with ICU 57.1
Fix the condition checking the minimum ICU version