This PR was merged into the 2.5-dev branch.
Discussion
----------
[Form] Errors now reference the field they were added to and the violation/exception that caused them
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #9472, #9582
| License | MIT
| Doc PR | -
Commits
-------
c8a0ee6 [Form] Errors now reference the field they were added to and the violation/exception that caused them
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9991).
Discussion
----------
Fix for a couple of undefined classes in docblocks
These are the 2 that I've come across, but there may be some more.
Commits
-------
3918d6d Fix for a couple of undefined classes in docblocks
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9979).
Discussion
----------
[Doctrine Bridge][Validator] Fix for null values in assosiated properties when using UniqueEntityValidator
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #7382
| License | MIT
| Doc PR |
This fixes issue with UniqueEntityValidator when ignoreNull = false and associated property contains null value:
```
Warning: ReflectionProperty::getValue() expects parameter 1 to be object, null given in [...]/symfony2/vendor/doctrine/orm/lib/Doctrine/ORM/Mapping/ClassMetadataInfo.php line 670
```
Here's what docs say about ignoreNull:
> If this option is set to true, then the constraint will allow multiple entities to have a null value for a field without failing validation. If set to false, only one null value is allowed - if a second entity also has a null value, validation would fail.
Commits
-------
bf368bc [Doctrine Bridge][Validator] Fix for null values in assosiated properties when using UniqueEntityValidator
This PR was merged into the 2.3 branch.
Discussion
----------
[TwigBridge] Update min. version of Twig
| Q | A
| ------------- | ---
| Bug fix? | kinda
| New feature? | no
| BC breaks? | no
| Tests pass? | yes
| License | MIT
The minimal version of Twig must be 1.12 because of usage the method `Twig_SimpleFunction` that was added in that version.
Commits
-------
c003291 [TwigBridge] Update min. version of Twig
This PR was merged into the 2.5-dev branch.
Discussion
----------
Upgrade File for 2.5
Added upgrade info for #9601, as this pr might break code which overwrites this method and also to respect symfony/symfony-docs#3439.
Commits
-------
fefcf41 Added upgrade info for #9601
This PR was merged into the 2.3 branch.
Discussion
----------
[CssSelector] fixed numeric attribute issue
This PR adds a cast from number to string when parsing a numeric attribute value.
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #9968
Commits
-------
613535a [CssSelector] fixed numeric attribute issue
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9747).
Discussion
----------
[DoctrineBridge] Fix: Add type detection. Needed by pdo_dblib
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets |
| License | MIT
| Doc PR |
Commits
-------
75f5cc3 Add type detection. Needed by pdo_dblib
This PR was squashed before being merged into the 2.5-dev branch (closes#9776).
Discussion
----------
[Console] Added the possibility to set a different default command
I am not quite sure if this is the best approach to solve the issue but the solution I provide works. Let me know your suggestions to improve it.
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | yes
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #8058
| License | MIT
| Doc PR | symfony/symfony-docs#3426
Commits
-------
418de05 [Console] Added the possibility to set a different default command
This PR was merged into the 2.5-dev branch.
Discussion
----------
added feedback to the cache:clear command
simplified version of #9463
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #9463
| License | MIT
| Doc PR | n/a
"Especially in production it is sometimes critical to know exactly when your cache is cleared or warmed up. Currently cache:clear provides no feedback whats however. As sites get bigger, so become their cache files. Removing old cache files may sometimes even take minutes. Without any feedback from the cache:clear you do not know the current status.
That's why i added more feedback to the cache:clear command that it makes it possible to see when your cache is warmed up and ready to go."
Commits
-------
f2261da [FrameworkBundle] simplified code
a1f6411 [FrameworkBundle] added feedback in cache:clear
* 2.4:
udpated LICENSE year
update year on licenses
rundown and typo fix
[Process] Fix#9861 : Revert TTY mode
[Form] Update minimal requirement in composer.json
Fix Empty translations with Qt files
[Console] Fixed command name guessing if an alternative is an alias.
Update UPGRADE-2.3.md to account for #9388
[WebProfilerBundle] Fixed profiler toolbar icons for XHTML.
[BrowserKit] Throw exception on invalid cookie expiration timestamp
[Propel1Bridge][ModelChoiceList] add exception message for invalid classes
This PR was merged into the 2.3 branch.
Discussion
----------
[All] update license year to 2014
| Q | A |
| --- | --- |
| Bug fix? | n |
| New feature? | n |
| BC breaks? | n |
| Deprecations? | n |
| Tests pass? | y |
| Fixed tickets | na |
| License | MIT |
| Doc PR | na |
Sent using [Gush](https://github.com/cordoval/gush)
sorry about the previous PR now closed, forgot Gush had branched off of master and not 2.3
Commits
-------
8ace5f9 update year on licenses
This PR was merged into the 2.3 branch.
Discussion
----------
[Validator] rundown with some typo fixes and consistency
| Q | A |
| --- | --- |
| Bug fix? | no |
| New feature? | no |
| BC breaks? | no |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | na |
| License | MIT |
| Doc PR | na |
Sent using [Gush](https://github.com/cordoval/gush)
Commits
-------
dcd23df rundown and typo fix
This PR was merged into the 2.5-dev branch.
Discussion
----------
[HttpFoundation] JsonResponse::setEncodingOptions accepts also integer
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | yes
| BC breaks? | no
| Tests pass? | yes
| License | MIT
Now you can also set encoding options like:
```php
$response->setEncodingOptions(JSON_UNESCAPED_UNICODE | $response->getEncodingOptions());
Commits
-------
f8bc3b2 [HttpFoundation] JsonResponse::setEncodingOptions accepts also integer
This PR was merged into the 2.5-dev branch.
Discussion
----------
[OptionsResolver] Allow giving a callback as an allowedValue to OptionsResolver
I recently had to use an option which was an array and could contain some one or multiple values from a list. As it could contain all possible combinations, it was not possible to validate it with a list of allowed values.
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | yes
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | --
| License | MIT
| Doc PR | https://github.com/symfony/symfony-docs/pull/3437
Commits
-------
07d1d30 Allow giving a callback as an allowedValue to OptionsResolver
This PR was merged into the 2.3 branch.
Discussion
----------
[Process] Fix#9861 : Revert TTY mode
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #9861
| License | MIT
This revert the TTY mode support for the process component
Commits
-------
160b1cf [Process] Fix#9861 : Revert TTY mode
This PR was merged into the 2.3 branch.
Discussion
----------
[Form] Update minimal requirement in composer.json
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Tests pass? | yes
| Fixed tickets | #9959
| License | MIT
Minimal requirement for PropertyAccess component needs to be `~2.3` as Form component depends on new method `PropertyAccess::createPropertyAccessor()`.
Commits
-------
88153b3 [Form] Update minimal requirement in composer.json
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9952).
Discussion
----------
[Translator] Fix Empty translations with Qt files
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | n/a
| Fixed tickets | no
| License | MIT
| Doc PR |
When you use ts files, QtLinguist, generate empty string for all translations, even if there are not translated yet.
It could be a good idea to generate an entry in the MessageCatalogue only if translation value is not empty.
Commits
-------
1c220be [Translator] Fix Empty translations with Qt files
This PR was merged into the 2.4 branch.
Discussion
----------
[Console] Fixed command name guessing if an alternative is an alias
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #9953
| License | MIT
| Doc PR | -
Commits
-------
ade448c [Console] Fixed command name guessing if an alternative is an alias.
This PR was submitted for the 2.3-dev branch but it was merged into the 2.3 branch instead (closes#9804).
Discussion
----------
[Form] Update UPGRADE-2.3.md to account for #9388
Added documentation for how to correctly pre-fill a form using the form's `data` option.
The "old" and also wrong way of doing it broke with Symfony 2.3.7 (with https://github.com/symfony/symfony/pull/9388 to be precise) and this short documentation should help others to fix the problem and do it right.
Commits
-------
5e06535 [Form] Update UPGRADE-2.3.md to account for #9388
This PR was squashed before being merged into the 2.3 branch (closes#9948).
Discussion
----------
[WebProfilerBundle] Fixed profiler toolbar icons for XHTML.
| Q | A
| ------------- | ---
| Bug fix? | yes
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | n/a
| Fixed tickets | n/a
| License | MIT
| Doc PR | n/a
Profiler page itself is entirely autonomic, but toolbar is not - unfortunately currently it doesn't work in XHTML documents :(.
#9877 backported to `2.3` branch.
Commits
-------
296c4d1 [WebProfilerBundle] Fixed profiler toolbar icons for XHTML.
This PR was merged into the 2.3 branch.
Discussion
----------
Propel1 exception message
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| BC breaks? | no
| Deprecations? | no
| Tests pass? | yes
| Fixed tickets | #9932
| License | MIT
| Doc PR | none
This add exception message for `ModelChoiceList` and `ModelType` when `class` parameter is not provided or invalid
Commits
-------
047492f [Propel1Bridge][ModelChoiceList] add exception message for invalid classes