This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9795).
Discussion
----------
[Form] Fixed issue in BaseDateTimeTransformer when invalid timezone cause Trans...
...formation filed exception (closes#9403).
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | no
| Fixed tickets | #9403
| License | MIT
| Doc PR |
DateTimeZone object creation could rise an exception. So in BaseDateTimeTransformer constructor we need to check if timezone is valid. If this exception is catched and rethrowed as TransformationFailed exception we have an issue: user will see "Invalid value" error in form, but issue is actually with php configuration or form type configuration.
Replaces #9772.
Commits
-------
3ece630 Fixed issue in BaseDateTimeTransformer when invalid timezone cause Transformation filed exception (closes#9403).
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9714).
Discussion
----------
[HttpFoundation] BinaryFileResponse should also return 416 or 200 on some range-requets
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | yes
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
I read around in the docs and tried something with BinaryFileResponse ... There are two things I missed in the implementation here:
* If the range, provided in the query, does exceed the file-position, return an 416 (or 200 - I decided to take 416 here)
* If the range is logical invalid (f.e. to request from byte 50 to byte 30 ..) return 200 - OK
One decision I took in addition, is to provide a 200 code if the full file is requested. For me, it doesn't make sense here to return a 206 for just the complete file.
What I'm quite unsure about: Do we need some additional fields for these two options? I can remember reading something about a Content-Range for 416 but I was quite unsure what it should be ...
Commits
-------
d5dc317 [HttpFoundation] BinaryFileResponse should also return 416 or 200 on some range-requets
This PR was merged into the 2.5-dev branch.
Discussion
----------
[DependencyInjection] added support for inlining Configurators
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | -
| License | MIT
| Doc PR | -
This is one commit from #9432.
As mentioned in #3758 configurators can not be private (it's just ignored). This pull changes that and allows them to be inlined. It it also creates better code if a configurator is used multiple times for one service (i.e. to both inject it and configure the same service, or to configure multiple inlined services) but this should be very rare.
Commits
-------
4e9aa07 [DependencyInjection] added support for inlining Configurators
This PR was merged into the 2.3 branch.
Discussion
----------
[Routing] Remove usage of deprecated _scheme requirement
**This is exact the same commit as it was in #9585, which was not merged due to my fault. Sorry for the noise.**
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #8898, #8176
| License | MIT
| Doc PR |
I removed all usages of the deprecated _scheme requirement inside the Routing Component.
Most parts were pretty easy and after multiple refactorings I came up with the solution to have a Route::hasScheme() method and check against this method.
I also checked for performance and after trying in_array, arra_flip+isset and foreach, the last one was clearly the winner.
https://gist.github.com/Danez/7609898#file-test_performance-php
I also adjusted all tests that test '_scheme' to also check the new schemes-requirement.
Commits
-------
557dfaa Remove usage of deprecated _scheme in Routing Component
This PR was submitted for the 2.4-dev branch but it was merged into the 2.4 branch instead (closes#9488).
Discussion
----------
[Form] fix deprecated usage and clarify constructor defaults for number formatter
By using the real default one can see the actual value without having to look into the implementation what NULL stands for
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | -
| License | MIT
| Doc PR | n/a
Commits
-------
5a25989 [Form] fix deprecated usage and clarify constructor defaults for number formatter
This PR was submitted for the 2.4-dev branch but it was merged into the 2.4 branch instead (closes#9358).
Discussion
----------
Bumping dependency to ProxyManager to allow testing against the new 0.5.x versions
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | no (looks like an unrelated failure, though)
| Fixed tickets |
| License | MIT
| Doc PR |
Nothing to see here. Move along.
Commits
-------
5377992 Bumping dependency to ProxyManager to allow testing against the new 0.5.x branch changes
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9489).
Discussion
----------
[DependencyInjection] Add normalization to tag options
Currently, when using tags in XML, the options aren't normalized. This means that the following code is wrong:
<tag name="sonata_admin" manager-type="doctrine_phpcr" ... />
It should be `manager_name` to remove errors, but that's not following the XML rules. The solution is to use the same normalization as the configuration: replacing - with _.
To be BC, both options (with and without normalization) are kept
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | -
| License | MIT
| Doc PR | -
Commits
-------
06d64d8 Do normalization on tag options
This PR was merged into the 2.3 branch.
Discussion
----------
[Filesystem] Changed the mode for a target file in copy() to be write only
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #9789
| License | MIT
| Doc PR | -
Stream wrappers like S3 [do not support w+](http://docs.aws.amazon.com/aws-sdk-php/latest/class-Aws.S3.StreamWrapper.html) and we don't read here anyway (as pointed out in #9789).
Commits
-------
10e43b7 [Filesystem] Changed the mode for a target file in copy() to be write only.
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9758).
Discussion
----------
[Console] fixed TableHelper when cell value has new line
#9757
Commits
-------
2e14ba6 [Console] fixed TableHelper when cell value has new line
This PR was merged into the 2.3 branch.
Discussion
----------
[Routing] Fix router matching pattern against multiple hosts
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #8811, #6744
| License | MIT
| Doc PR |
When you had a pattern that matched on multiple host then only the first one was displayed as "almost matching". Fixed router matching against the same pattern on multiple hosts so now it shows every "almost match" on different hosts.
Commits
-------
f727b22 [Routing] Fix router matching pattern against multiple hosts
This PR was squashed before being merged into the 2.4 branch (closes#9768).
Discussion
----------
[FrameworkBundle] Fixed bug in XMLDescriptor
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Fixed bug in container:debug command where parameter value doesn't get printed.
Commits
-------
34d0b1d [FrameworkBundle] Fixed bug in XMLDescriptor
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9763).
Discussion
----------
Update validators.cs.xlf
fixed CZ translation.
there is not such a word as "null" in czech
Commits
-------
eac9032 Update validators.cs.xlf
This PR was merged into the 2.4 branch.
Discussion
----------
[Security] Add ExpressionLanguage to require-dev.
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Running the tests for Security-Core fails because ExpressionLanguage is missing.
Commits
-------
7f18d19 [Security] Add ExpressionLanguage to require-dev.
This PR was merged into the 2.3 branch.
Discussion
----------
[Validator] updated the Spanish translation of the validator strings
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Commits
-------
c9c21a5 [Validator] updated the Spanish translation of the validator strings
This PR was submitted for the 2.4-dev branch but it was merged into the 2.4 branch instead (closes#9756).
Discussion
----------
[Config] Removed duplicated condition
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | none
| License | MIT
| Doc PR | none
Removed unnecessary condition (looks like it's been copy/pasted from
line 84).
Commits
-------
5c9dd30 [Config] Removed duplicated condition
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | none
| License | MIT
| Doc PR | none
Removed unnecessary condition (looks like it's been copy/pasted from
line 84).
This PR was merged into the 2.3 branch.
Discussion
----------
[Validator] Fix mistype in lithuanian validator translations
Fix mistype which slipped through initial proof reading.
| Q | A
| ------------- | ---
| Fixed tickets | #9749
| License | MIT
Commits
-------
e2ef38c Fix mistype which slipped through initial proof reading.
Instead correctly use the array of schemes from the Route.
Also adjusted the dumpers to dump the correct data.
I extended the tests to not only test the deprecated behavior, but also
the new schemes-requirement.
This PR was submitted for the 2.4-dev branch but it was merged into the 2.4 branch instead (closes#9696).
Discussion
----------
[Security] added test for ExpressionVoter.php
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | none
| License | MIT
| Doc PR | none
Added the missing test for `ExpressioneVoter`,
Fixed small docblock.
Commits
-------
e9de00f added ExpressionVoterTest
This PR was submitted for the 2.2 branch but it was merged into the 2.3 branch instead (closes#9674).
Discussion
----------
[2.2][Form] rename validators.ua.xlf to validators.uk.xlf
| Q | A
| ------------- | ---
| Fixed tickets | --
| License | MIT
The ISO 639-1 code for Ukrainian is "uk".
Shall I open separate PRs for newer branches as well, or can this be merged into them?
Commits
-------
fe6995d renamed validators.ua.xlf to validators.uk.xlf
This PR was merged into the 2.3 branch.
Discussion
----------
[Validator]Fixed getting wrong msg when value is an object in Exception
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| License | MIT
Commits
-------
cdb5ae0 [Validator]Fixed getting wrong msg when value is an object in UnexpectedTypeException