This PR was merged into the 3.4 branch.
Discussion
----------
Use long array syntax
| Q | A
| ------------- | ---
| Branch? | 3.4
| Bug fix? | yes
| New feature? | no
| BC breaks? | no <!-- see https://symfony.com/bc -->
| Deprecations? | no
| Tests pass? | yes <!-- please add some, will be required by reviewers -->
| Fixed tickets | #... <!-- #-prefixed issue number(s), if any -->
| License | MIT
| Doc PR | symfony/symfony-docs#... <!-- required for new features -->
<!--
Write a short README entry for your feature/bugfix here (replace this comment block.)
This will help people understand your PR and can be used as a start of the Doc PR.
Additionally:
- 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.
-->
Commits
-------
2abb8a4 Use long array syntax
This PR was merged into the 3.4 branch.
Discussion
----------
Set controller without __invoke method from invokable class
| Q | A
| ------------- | ---
| Branch? | 3.4
| Bug fix? | no
| New feature? | no
| BC breaks? | no <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md files -->
| Tests pass? | yes <!-- please add some, will be required by reviewers -->
| Fixed tickets |
| License | MIT
| Doc PR |
Fixes one part of #25103
Commits
-------
cc68c5074e Set controller without __invoke method from invokable class
This PR was merged into the 2.7 branch.
Discussion
----------
[Routing] Revert throwing 405 on missed slash/scheme redirections
| Q | A
| ------------- | ---
| Branch? | 2.7
| Bug fix? | ye
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | -
| License | MIT
| Doc PR | -
This partially reverts an unreleased behavior: throwing a 405 when a non-safe verb prevents a slash/scheme redirection.
While this was correct, we are removing it from master to remove some complexity from the implementation of dumped router.
Better not make anyone rely on such nitpicking details that have little to no use in practice.
Commits
-------
391b01393d [Routing] Revert throwing 405 on missed slash/scheme redirections
This PR was merged into the 3.4 branch.
Discussion
----------
[WebProfilerBundle] fix test after ajax path updated
| Q | A
| ------------- | ---
| Branch? | 3.4
| Bug fix? | no
| New feature? | no <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks? | no <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md files -->
| Tests pass? | yes <!-- please add some, will be required by reviewers -->
| Fixed tickets | none
| License | MIT
| Doc PR | none
Fixing the WebProfilerBundle test after the merge in 3.4.
Commits
-------
77dfc90846 [WebProfilerBundle] fix test after ajax path updated
This PR was merged into the 2.7 branch.
Discussion
----------
Fix ArrayInput::toString() for InputArgument::IS_ARRAY args
| Q | A
| ------------- | ---
| Branch? | 2.7 up to 4.0
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #... <!-- #-prefixed issue number(s), if any -->
| License | MIT
| Doc PR | symfony/symfony-docs#... <!-- required for new features -->
Fix ArrayInput::toString() method to eliminate 'Array to string conversion' notice and get right string representation of InputArgument::IS_ARRAY argument. It need to be fixed in all branches up to master.
Commits
-------
f371fd8 Fix ArrayInput::toString() for InputArgument::IS_ARRAY args
This PR was merged into the 4.1-dev branch.
Discussion
----------
[Routing] Redirect from trailing slash to no-slash when possible
| Q | A
| ------------- | ---
| Branch? | master
| Bug fix? | no
| New feature? | yes
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #26207
| License | MIT
| Doc PR | -
Implemented as suggest by @Tobion in https://github.com/symfony/symfony/pull/26059#issuecomment-365071281
When a route for `/foo` exists but the request is for `/foo/`, we now redirect.
(this complements the flipped side redirection, which already exists.)
Commits
-------
69a4e94130 [Routing] Redirect from trailing slash to no-slash when possible
This PR was submitted for the master branch but it was squashed and merged into the 3.4 branch instead (closes#26177).
Discussion
----------
Update excluded_ajax_paths for sf4
| Q | A
| ------------- | ---
| Branch? | 4.0
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #25941
| License | MIT
| Doc PR | https://github.com/symfony/symfony-docs/pull/9267/files
This PR update `excluded_ajax_paths` from `vendor/symfony/web-profiler-bundle/DependencyInjection/Configuration.php` because
there is no neither `app.php` nor `app_dev.php` in symfony 4
We also need update this [Documentation](https://symfony.com/doc/current/reference/configuration/web_profiler.html)
Commits
-------
ce01097935 Update excluded_ajax_paths for sf4
This PR was merged into the 4.1-dev branch.
Discussion
----------
[FrameworkBundle] fix lowest supported Serializer version
| Q | A
| ------------- | ---
| Branch? | master
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
The `AbstractObjectNormalizer::setMaxDepthHandler()` method does not
exist before `symfony/serializer` 4.1.
Commits
-------
3b092a08ea fix lowest supported Serializer version
This PR was merged into the 4.0 branch.
Discussion
----------
[Security] Add missing use of Role
| Q | A
| ------------- | ---
| Branch? | 4.0 to master
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Same as [here](https://github.com/symfony/symfony/pull/26286) but this time for 4.0
Commits
-------
3cfe7d0 Add missing use of Role
This PR was submitted for the 3.3 branch but it was merged into the 3.4 branch instead (closes#26286).
Discussion
----------
[Security] Add missing use for RoleInterface
| Q | A
| ------------- | ---
| Branch? | 3.3
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Just a missing `use ...\RoleInterface` in the `UsernamePasswordToken` since 3.3.
Commits
-------
05359f3 Add missing use for RoleInterface
This PR was merged into the 4.1-dev branch.
Discussion
----------
[Console] fix console test
| Q | A
| ------------- | ---
| Branch? | master
| Bug fix? | no
| New feature? | no <!-- don't forget to update src/**/CHANGELOG.md files -->
| BC breaks? | no <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- don't forget to update UPGRADE-*.md files -->
| Tests pass? | yes <!-- please add some, will be required by reviewers -->
| Fixed tickets | none <!-- #-prefixed issue number(s), if any -->
| License | MIT
| Doc PR | none
It seems that tests are broken due to missing spaces.
Commits
-------
965e472 [Console] fix console test
This PR was merged into the 3.4 branch.
Discussion
----------
add container.autowiring.strict_mode to 3.4 docs
| Q | A
| ------------- | ---
| Branch? | 3.4
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
#26251 was a nice step, but IMO we need to add some docs to the 3.4 upgrade docs too as this is where people are probably looking first.
Commits
-------
4d973c2676 add container.autowiring.strict_mode to 3.4 docs
