~xaav/tsep/symfony

From Revision 7650 to 7631

expand all

collapse all

Rev		Summary	Authors	Date	Diff	Files
7650		bug #14708 [TwigBridge] use proper class to fetch asset vers... bug #14708 [TwigBridge] use proper class to fetch asset version strategy property (xabbuh) This PR was merged into the 2.7 branch. Discussion ---------- [TwigBridge] use proper class to fetch asset version strategy property | Q             | A | ------------- | --- | Bug fix?      | yes | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | #14692 | License       | MIT | Doc PR        | Commits ------- 01b7dd6 [TwigBridge] use proper class to fetch asset version strategy property	Fabien Potencier	9 years ago
7649		minor #14707 [FrameworkBundle][TwigBundle] add changed debug... minor #14707 [FrameworkBundle][TwigBundle] add changed debug command names to upgrade file (xabbuh) This PR was merged into the 2.7 branch. Discussion ---------- [FrameworkBundle][TwigBundle] add changed debug command names to upgrade file | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | | License       | MIT | Doc PR        | Thanks to @timglabisch for noticing this in https://github.com/symfony/symfony/pull/11627#issuecomment-66521130. Commits ------- 4443c4d add changed debug command names to upgrade file	Fabien Potencier	9 years ago
7648		minor #14662 added supported format in commands supporting -... minor #14662 added supported format in commands supporting --format (fabpot) This PR was merged into the 2.7 branch. Discussion ---------- added supported format in commands supporting --format | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | - | License       | MIT | Doc PR        | - replaces #14084 Commits ------- ea874b6 added supported format in commands supporting --format c4bf217 deprecated the --xml option for commands	Fabien Potencier	9 years ago
7647		bug #14576 [DoctrineBridge][Form] Fix BC break in DoctrineTy... bug #14576 [DoctrineBridge][Form] Fix BC break in DoctrineType (malarzm) This PR was squashed before being merged into the 2.7 branch (closes #14576). Discussion ---------- [DoctrineBridge][Form] Fix BC break in DoctrineType | Q             | A | ------------- | --- | Bug fix?      | yes | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | #14568 | License       | MIT | Doc PR        | n/a Commits ------- 3172d73 [DoctrineBridge][Form] Fix BC break in DoctrineType	Fabien Potencier	9 years ago
7646		bug #14551 [2.7][Form] Fixed ChoiceType with legacy ChoiceLi... bug #14551 [2.7][Form] Fixed ChoiceType with legacy ChoiceList (xelaris) This PR was merged into the 2.7 branch. Discussion ---------- [2.7][Form] Fixed ChoiceType with legacy ChoiceList | Q             | A | ------------- | --- | Bug fix?      | yes | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | #14382 | License       | MIT | Doc PR        | The "Backwards compatibility" condition doesn't grap (e.g. when passing a `SimpleChoiceList` as `choice_list` on `ChoiceType`), as the default value for the `ChoiceType` option `preferred_choices` is `array()` instead of `null`. So I changed the condition from `null === $preferredChoices` to `empty($preferredChoices)`. Then there was an issue with accessing `attr` in `form_div_layout.html.twig`, since the deprecated `Symfony\Component\Form\Extension\Core\View\ChoiceView` doesn't provide an `attr` attribute. Since the docblocks of `Symfony\Component\Form\ChoiceList\View\ChoiceListView` state `$choices` and `$preferredChoices` to be instances of `Symfony\Component\Form\ChoiceList\View\ChoiceView` instead of `Symfony\Component\Form\Extension\Core\View\ChoiceView` I fixed the template issue by mapping the deprecated `ChoiceView` objects to the new one with an empty `attr`. @webmozart Could you have a look at it, please? Without this PR the following example would render numeric values as labels: ```php         $formBuilder->add('choices', 'choice', array(             'choice_list' => new SimpleChoiceList(array(                 'creditcard' => 'Credit card payment',                 'cash' => 'Cash payment'             ))         )); ``` Commits ------- a98e484 [Form] Fix ChoiceType with legacy ChoiceList	Fabien Potencier	9 years ago
7645		bug #14648 [Console] Fix first choice was invalid when using... bug #14648 [Console] Fix first choice was invalid when using value (ogizanagi) This PR was merged into the 2.7 branch. Discussion ---------- [Console] Fix first choice was invalid when using value | Q             | A | ------------- | --- | Bug fix?      | yes | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | - | License       | MIT | Doc PR        | - This PR solves the following issues encountered using question helper and choices questions: - First choice was not selectable by value. - ChoiceList with associative choices with mixed string and int keys has same issue with first choice. - Fix inconsistency by always returning values as strings. First point exemple:  Last two points are mainly edge cases. Indeed, if a QuestionChoice has something like : ```php array(     '0' => 'No environment',     '1' => 'My environment 1',     'env_2' => 'My environment 2',     3 => 'My environment 3', ); ``` as choices, you will not be able to select the first choice and get an `InvalidArgumentException`: ``` There were 2 errors: 1) Symfony\Component\Console\Tests\Helper\QuestionHelperTest::testChoiceFromChoicelistWithMixedKeys with data set #0 ('0', '0') InvalidArgumentException: Value "0" is invalid 2) Symfony\Component\Console\Tests\Helper\QuestionHelperTest::testChoiceFromChoicelistWithMixedKeys with data set #1 ('No environment', '0') InvalidArgumentException: Value "No environment" is invalid ``` Moreover, even if you were able to select by value (`No environment`), you'll get an integer instead of a string: ``` Failed asserting that '0' is identical to 0. ``` For more consistency, the solution is to always return a string. The issue does not exist in 2.6, as the `QuestionChoice::getDefaultValidator` handled things differently. Commits ------- 03e4ab6 [Console] Fix first choice was invalid when using value	Fabien Potencier	9 years ago
7644		minor #14595 [FrameworkBundle] Applied new styles to transla... minor #14595 [FrameworkBundle] Applied new styles to translation commands (ogizanagi) This PR was merged into the 2.7 branch. Discussion ---------- [FrameworkBundle] Applied new styles to translation commands | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | Partially #14138 | License       | MIT | Doc PR        | - ## Translation debug _Only the table layout and deprecation note is changed_   ## Translation update ### Before:  ### After:     Commits ------- 812fedc [FrameworkBundle] Applied new styles to translation commands	Fabien Potencier	9 years ago
7643		minor #14665 [HttpKernel] trigger a deprecation warning when... minor #14665 [HttpKernel] trigger a deprecation warning when using the ContainerAwareHttpKernel (xabbuh) This PR was merged into the 2.7 branch. Discussion ---------- [HttpKernel] trigger a deprecation warning when using the ContainerAwareHttpKernel | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | #13280 | License       | MIT | Doc PR        | I agree that we should not trigger the deprecation warning when the `ContainerAwareHttpKernel` is used in the framework code. However, developers using this class in their own code should receive a warning to be able to prepare their applications for Symfony 3.0. Commits ------- 030731a [HttpKernel] trigger a deprecation warning when using the ContainerAwareHttpKernel	Fabien Potencier	9 years ago
7642		minor #14666 [UPGRADE] Mention the CSS strategy and it's eff... minor #14666 [UPGRADE] Mention the CSS strategy and it's effects (WouterJ) This PR was squashed before being merged into the 2.7 branch (closes #14666). Discussion ---------- [UPGRADE] Mention the CSS strategy and it's effects The CSS strategy, which is automatically applied to Twig CSS files as of Symfony 2.7, results in different behaviour when outputting reserved characters like `#123456`. This has to be documented in the UPGRADE file. See https://github.com/javiereguiluz/EasyAdminBundle/pull/305 for an example. | Q | A | --- | --- | Fixed tickets | - | License | MIT Commits ------- 8b891ff [UPGRADE] Mention the CSS strategy and it's effects	Fabien Potencier	9 years ago
7641		minor #14688 [Routing] clarify deprecation message (xabbuh) minor #14688 [Routing] clarify deprecation message (xabbuh) This PR was merged into the 2.7 branch. Discussion ---------- [Routing] clarify deprecation message | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | | License       | MIT | Doc PR        | as reported by @javiereguiluz in https://github.com/symfony/symfony/commit/7784b29e5afa4aed419b55e857cf9313627aa408#commitcomment-11229119 Commits ------- 8a99645 [Routing] clarify deprecation message	Fabien Potencier	9 years ago
7640		minor #14638 [Console] Fix QuestionHelperTest (ogizanagi) minor #14638 [Console] Fix QuestionHelperTest (ogizanagi) This PR was merged into the 2.7 branch. Discussion ---------- [Console] Fix QuestionHelperTest | Q             | A | ------------- | --- | Bug fix?      | yes | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | - | License       | MIT | Doc PR        | - This PR fixes the following issues in `QuestionHelperTest`: - When using the `InputInterface` mock with interactive option, asking a ChoiceQuestion without specifying max attempts nor a default value led to an infinite loop when invalid values are submitted. In case there is something wrong in tests, it wasn't obvious to understand what's happening, as the process was constantly waiting for inputs. - `testAmbiguousChoiceFromChoicelist` test was wrong as the autocompleter filled the value and the test was never triggered (+max attempt issue). Commits ------- 6a0308d [Console] Fix QuestionHelperTest	Fabien Potencier	9 years ago
7639		minor #14581 [Security] Removed unnecessary statement (MacDa... minor #14581 [Security] Removed unnecessary statement (MacDada) This PR was merged into the 2.7 branch. Discussion ---------- [Security] Removed unnecessary statement | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | ~ | License       | MIT | Doc PR        | ~ Removed unnecessary statement from `PersistentTokenBasedRememberMeServices.php`. `$series` comes from `$cookieParts` and `$this->tokenProvider->loadTokenBySeries($series);` is supposed to find the token with that value. Doing `$persistentToken->getSeries();` should give us exactly the same value, so it is an unnecessary statement. Why? * We don't need it? We won't miss it when it's gone. * It confuses a code reader who starts guessing why would that be needed (at least I did and lost time because of that). Unless… It actually is needed, as we want `TokenProviderInterface` implementations to have a possibility to give a `PersistentTokenInterface` with a different series value than asked… I can make a PR to the testing class so that such requirement is checked upon. I don't believe that this is BC, as this behaviour isn't documented anywhere and no existing (known to me) implementations return different series than the asked ones (and current tests pass successfully). Commits ------- c7a91f1 Removed unnecessary statement from PersistentTokenBasedRememberMeServices.php	Fabien Potencier	9 years ago
7638		bug #14645 [WebProfilerBundle] Fix compatiblity with HttpKer... bug #14645 [WebProfilerBundle] Fix compatiblity with HttpKernel < 2.7 (GromNaN) This PR was merged into the 2.7 branch. Discussion ---------- [WebProfilerBundle] Fix compatiblity with HttpKernel < 2.7 The WebProfilerBundle version 2.7 is supposed to be compatible with HttpKernel 2.4, but the template of the config profile uses a property `symfonyState` that has been added in Symfony 2.7 by #13626. > Twig_Error_Runtime: Method "symfonyState" for object "Symfony\Component\HttpKernel\DataCollector\ConfigDataCollector" does not exist in "@WebProfiler/Collector/config.html.twig" at line 12 This fix makes the property optional. | Q             | A | ------------- | --- | Bug fix?      | yes | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | https://github.com/silexphp/Silex-WebProfiler/pull/63 | License       | MIT Commits ------- 700fcf1 Fix WebProfilerBundle compatiblity with HttpKernel < 2.7	Fabien Potencier	9 years ago
7637		fixed CS fixed CS	Fabien Potencier	9 years ago
7636		Merge branch '2.6' into 2.7 Merge branch '2.6' into 2.7 * 2.6:   fixed CS   fixed CS Conflicts:         src/Symfony/Component/Security/Http/Tests/Firewall/RememberMeListenerTest.php	Fabien Potencier	9 years ago
7635		minor #14591 [FrameworkBundle] Applied new styles to the con... minor #14591 [FrameworkBundle] Applied new styles to the config:debug & config:dump-reference commands (ogizanagi) This PR was merged into the 2.7 branch. Discussion ---------- [FrameworkBundle] Applied new styles to the config:debug & config:dump-reference commands | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | Partially #14138 | License       | MIT | Doc PR        | - Min dependency on `symfony/console` is changed to `~2.7` in the FrameworkBundle, as it will be a requirement in order to update each command provided in this bundle. Commits ------- e56a619 [FrameworkBundle] Applied new styles to the config:debug & config:dump-reference commands	Fabien Potencier	9 years ago
7634		Merge branch '2.6' into 2.7 Merge branch '2.6' into 2.7 * 2.6:   [DebugBundle] Allow alternative destination for dumps   [DebugBundle] Use output mechanism of dumpers instead of echoing   [DebugBundle] Always collect dumps   Fix tests in HHVM   CS: Pre incrementation/decrementation should be used if possible Conflicts:         src/Symfony/Component/Finder/Expression/Glob.php	Fabien Potencier	9 years ago
7633		minor #14642 [Validator] Deprecated PHP7-incompatible constr... minor #14642 [Validator] Deprecated PHP7-incompatible constraints and related validators (nicolas-grekas) This PR was merged into the 2.7 branch. Discussion ---------- [Validator] Deprecated PHP7-incompatible constraints and related validators | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | yes | Tests pass?   | yes | Fixed tickets | - | License       | MIT | Doc PR        | - Commits ------- a3fc6b9 [Validator] Deprecated PHP7-incompatible constraints and related validators	Fabien Potencier	9 years ago
7632		bug #14600 [Console] SymfonyStyle: fix block rpadding when e... bug #14600 [Console] SymfonyStyle: fix block rpadding when escaping '<' (ogizanagi) This PR was merged into the 2.7 branch. Discussion ---------- [Console] SymfonyStyle: fix block rpadding when escaping '<' | Q             | A | ------------- | --- | Bug fix?      | yes | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | - | License       | MIT | Doc PR        | -  Commits ------- 80fc341 [Console] SymfonyStyle: fix block rpadding when escaping '<'	Nicolas Grekas	9 years ago
7631		minor #14620 [Console] fix code style (xabbuh) minor #14620 [Console] fix code style (xabbuh) This PR was merged into the 2.7 branch. Discussion ---------- [Console] fix code style | Q             | A | ------------- | --- | Bug fix?      | no | New feature?  | no | BC breaks?    | no | Deprecations? | no | Tests pass?   | yes | Fixed tickets | | License       | MIT | Doc PR        | Prevent fabbot from complaining when the `SymfonyStyle` class or the `StyleInterface` are modified (like, for example, [here](http://fabbot.io/report/symfony/symfony/14600/a144667d5f3889e832f68f85dc1ad6be4505167f)). Commits ------- 314901e [Console] fix code style	Nicolas Grekas	9 years ago

• Older »

Loggerhead is a web-based interface for Breezy
Version: 2.0.1