Commit Graph

31097 Commits

Author SHA1 Message Date
Nicolas Grekas
d3449e6a24 Merge branch '3.4'
* 3.4:
  bug #22814 [FrameworkBundle] FC with EventDispatcher 4.0 (xabbuh)
  [PhpUnitBridge] remove unused use statement
  do not used deprecated validator test case class
  do not mock a deprecated interface
  [DI] Added missing deprecation in changelog
  [Ldap] add a changelog file
  [Security][Serializer][DI] Add new arguments typehints in preparation for 4.0
  [MonologBridge] Fix the Monlog ServerLogHandler from Hanging on Windows
  [DependencyInjection] Fix dumping of RewindableGenerator with empty IteratorArgument
  [DI][Serializer] Fix missing de(normalizer|coder) autoconfig
  Use 0.0.0.0 as the server log host default.
2017-05-21 10:58:35 +02:00
Nicolas Grekas
b6fced6261 Merge branch '3.3' into 3.4
* 3.3:
  [PhpUnitBridge] remove unused use statement
  do not mock a deprecated interface
  [DI] Added missing deprecation in changelog
  [Ldap] add a changelog file
  [Security][Serializer][DI] Add new arguments typehints in preparation for 4.0
  [MonologBridge] Fix the Monlog ServerLogHandler from Hanging on Windows
  [DependencyInjection] Fix dumping of RewindableGenerator with empty IteratorArgument
  [DI][Serializer] Fix missing de(normalizer|coder) autoconfig
  Use 0.0.0.0 as the server log host default.
2017-05-21 10:57:11 +02:00
Nicolas Grekas
023cf9e014 Merge branch '3.2' into 3.3
* 3.2:
  do not mock a deprecated interface
  [DI] Added missing deprecation in changelog
  [Ldap] add a changelog file
2017-05-21 10:56:50 +02:00
Nicolas Grekas
455b09bd3d minor #22802 [Ldap] add a changelog file (xabbuh)
This PR was merged into the 3.2 branch.

Discussion
----------

[Ldap] add a changelog file

| 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        |

Commits
-------

5bdda6c [Ldap] add a changelog file
2017-05-21 10:51:01 +02:00
Nicolas Grekas
a0364bc3eb minor #22757 [Security][Serializer][DI] Add new arguments typehints in preparation for 4.0 (ogizanagi)
This PR was merged into the 3.3 branch.

Discussion
----------

[Security][Serializer][DI] Add new arguments typehints in preparation for 4.0

| Q             | A
| ------------- | ---
| Branch?       | 3.3
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | https://github.com/symfony/symfony/pull/22743#pullrequestreview-38950305
| License       | MIT
| Doc PR        | N/A

See https://github.com/symfony/symfony/pull/22743#pullrequestreview-38950305 discussion for the motivations.

Commits
-------

b973b30 [Security][Serializer][DI] Add new arguments typehints in preparation for 4.0
2017-05-21 10:47:05 +02:00
Nicolas Grekas
c268ebafc3 bug #22781 [DI][Serializer] Fix missing de(normalizer|coder) autoconfig (ogizanagi)
This PR was merged into the 3.3 branch.

Discussion
----------

[DI][Serializer] Fix missing de(normalizer|coder) autoconfig

| Q             | A
| ------------- | ---
| Branch?       | 3.3
| Bug fix?      | yes(ish)
| New feature?  | no (at least complete the 3.3 feature)
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | N/A
| License       | MIT
| Doc PR        | N/A

Just migrating an app to `3.3.0-RC1`, I get surprised it wasn't already registered :)

Commits
-------

3c3a276 [DI][Serializer] Fix missing de(normalizer|coder) autoconfig
2017-05-21 10:45:16 +02:00
Nicolas Grekas
5c40e1264c bug #22790 [DependencyInjection] Fix dumping of RewindableGenerator with empty IteratorArgument (meyerbaptiste)
This PR was merged into the 3.3 branch.

Discussion
----------

[DependencyInjection] Fix dumping of RewindableGenerator with empty IteratorArgument

| Q             | A
| ------------- | ---
| Branch?       | 3.3
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #22780
| License       | MIT
| Doc PR        | N/A

According with https://github.com/symfony/symfony/issues/22780#issuecomment-302747395, when an `IteratorArgument` is empty, the closure generated by the `PhpDumper` should be `function () { return new EmptyIterator();` instead of `function () {}`, which is an invalid traversable for the `RewindableGenerator`.

Commits
-------

c2db0c1 [DependencyInjection] Fix dumping of RewindableGenerator with empty IteratorArgument
2017-05-21 10:44:23 +02:00
Nicolas Grekas
9b4a0c7af4 bug #22814 [FrameworkBundle] FC with EventDispatcher 4.0 (xabbuh)
This PR was merged into the 4.0-dev branch.

Discussion
----------

[FrameworkBundle] FC with EventDispatcher 4.0

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | https://github.com/symfony/symfony/pull/22796#issuecomment-302875580
| License       | MIT
| Doc PR        |

Commits
-------

b7c76f7 [FrameworkBundle] FC with EventDispatcher 4.0
2017-05-21 10:41:40 +02:00
Nicolas Grekas
8529e02879 minor #22810 [DI] Added missing deprecation in changelog (ro0NL)
This PR was merged into the 3.2 branch.

Discussion
----------

[DI] Added missing deprecation in changelog

| Q             | A
| ------------- | ---
| Branch?       | 3.2
| 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#... <!--highly recommended for new features-->

See #20113

Commits
-------

b3d58c5 [DI] Added missing deprecation in changelog
2017-05-21 10:40:13 +02:00
Nicolas Grekas
44237bcab5 minor #22813 [Security] do not mock a deprecated interface (xabbuh)
This PR was merged into the 3.2 branch.

Discussion
----------

[Security] do not mock a deprecated interface

| Q             | A
| ------------- | ---
| Branch?       | 3.2
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | https://github.com/symfony/symfony/pull/22797#issuecomment-302875662
| License       | MIT
| Doc PR        |

Commits
-------

a920274 do not mock a deprecated interface
2017-05-21 10:36:20 +02:00
Nicolas Grekas
339c6532d1 bug #22814 [FrameworkBundle] FC with EventDispatcher 4.0 (xabbuh)
This PR was merged into the 4.0-dev branch.

Discussion
----------

[FrameworkBundle] FC with EventDispatcher 4.0

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | https://github.com/symfony/symfony/pull/22796#issuecomment-302875580
| License       | MIT
| Doc PR        |

Commits
-------

b7c76f7 [FrameworkBundle] FC with EventDispatcher 4.0
2017-05-21 10:35:28 +02:00
Nicolas Grekas
9ef942b842 minor #22815 do not used deprecated validator test case class (xabbuh)
This PR was merged into the 3.4 branch.

Discussion
----------

do not used deprecated validator test case class

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | https://github.com/symfony/symfony/pull/22795#issuecomment-302875647
| License       | MIT
| Doc PR        |

Commits
-------

f20885e do not used deprecated validator test case class
2017-05-21 10:32:33 +02:00
Nicolas Grekas
297fabaced minor #22816 [PhpUnitBridge] remove unused use statement (xabbuh)
This PR was merged into the 3.3 branch.

Discussion
----------

[PhpUnitBridge] remove unused use statement

| Q             | A
| ------------- | ---
| Branch?       | 3.3
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | https://github.com/symfony/symfony/pull/22792#discussion_r117606399
| License       | MIT
| Doc PR        |

Commits
-------

c36ea03 [PhpUnitBridge] remove unused use statement
2017-05-21 10:26:22 +02:00
Christian Flothmann
c36ea03e68 [PhpUnitBridge] remove unused use statement 2017-05-21 10:03:01 +02:00
Christian Flothmann
f20885eefe do not used deprecated validator test case class 2017-05-21 09:57:21 +02:00
Christian Flothmann
b7c76f7147 [FrameworkBundle] FC with EventDispatcher 4.0 2017-05-21 09:51:07 +02:00
Christian Flothmann
a920274709 do not mock a deprecated interface 2017-05-21 09:39:50 +02:00
Roland Franssen
b3d58c54a4 [DI] Added missing deprecation in changelog 2017-05-20 22:16:10 +02:00
Nicolas Grekas
da61abdf20 feature #22794 [ExpressionLanguage] remove deprecated features (xabbuh)
This PR was merged into the 4.0-dev branch.

Discussion
----------

[ExpressionLanguage] remove deprecated features

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | yes
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

Commits
-------

37ac945 [ExpressionLanguage] remove deprecated features
2017-05-20 16:09:16 +02:00
Christian Flothmann
37ac9457c9 [ExpressionLanguage] remove deprecated features 2017-05-20 16:03:23 +02:00
Christian Flothmann
5bdda6ce74 [Ldap] add a changelog file 2017-05-20 15:43:53 +02:00
Maxime Steinhausser
b973b3072a [Security][Serializer][DI] Add new arguments typehints in preparation for 4.0 2017-05-20 12:13:18 +02:00
Nicolas Grekas
5ed9cb316e feature #22779 [4.0][BC Break] Removed BC layers for ControllerResolver::getArguments() (iltar)
This PR was squashed before being merged into the 4.0-dev branch (closes #22779).

Discussion
----------

[4.0][BC Break] Removed BC layers for ControllerResolver::getArguments()

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | yes
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | ~
| License       | MIT
| Doc PR        | ~

Removes the Backwards Compatibility layer for the `ControllerResolver` that depends on the `ArgumentValueResolver`. ~~There's still 1 bit left in the `HttpKernel`, but I don't quite know how this is solved in the best way:~~

```php
    public function __construct(EventDispatcherInterface $dispatcher, ControllerResolverInterface $resolver, RequestStack $requestStack = null, ArgumentResolverInterface $argumentResolver = null)
    {
        $this->dispatcher = $dispatcher;
        $this->resolver = $resolver;
        $this->requestStack = $requestStack ?: new RequestStack();
        $this->argumentResolver = $argumentResolver;

        if (null === $this->argumentResolver) {
            @trigger_error(sprintf('As of 3.1 an %s is used to resolve arguments. In 4.0 the $argumentResolver becomes the %s if no other is provided instead of using the $resolver argument.', ArgumentResolverInterface::class, ArgumentResolver::class), E_USER_DEPRECATED);
            // fallback in case of deprecations
            $this->argumentResolver = $resolver;
        }
    }
```

~~The 4th argument is now mandatory, but I can't make it mandatory without switching it with the request stack.~~
 - ~~I can make both mandatory~~
 - ~~I can make it `?RequestStack`~~
 - ~~I can switch the arguments~~

~~Each of those area a BC break but for the request stack or the switch, there is no BC layer yet (could be done in 3.4).~~

Commits
-------

64ac6e5 [4.0][BC Break] Removed BC layers for ControllerResolver::getArguments()
2017-05-20 12:00:07 +02:00
Iltar van der Berg
64ac6e5d1f [4.0][BC Break] Removed BC layers for ControllerResolver::getArguments() 2017-05-20 12:00:05 +02:00
Nicolas Grekas
d1aede93aa minor #22744 [Serializer] Remove a now useless call to method_exists (dunglas)
This PR was merged into the 4.0-dev branch.

Discussion
----------

[Serializer] Remove a now useless call to method_exists

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no <!-- don't forget updating src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | no <!-- don't forget updating UPGRADE-*.md files -->
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a
<!--
- 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
-------

6425f85 [Serializer] Remove a now useless call to method_exists
2017-05-20 11:49:19 +02:00
Nicolas Grekas
d25d340e36 minor #22747 [DependencyInjection] Remove dead code with PHP 7+ (dunglas)
This PR was merged into the 4.0-dev branch.

Discussion
----------

[DependencyInjection] Remove dead code with PHP 7+

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no <!-- don't forget updating src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | no <!-- don't forget updating UPGRADE-*.md files -->
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

Commits
-------

372e96e [DependencyInjection] Remove dead code with PHP 7+
2017-05-20 11:47:39 +02:00
Fabien Potencier
159f3c59d5 bug #22787 [MonologBridge] Fix the Monlog ServerLogHandler from Hanging on Windows (ChadSikorra)
This PR was squashed before being merged into the 3.3 branch (closes #22787).

Discussion
----------

[MonologBridge] Fix the Monlog ServerLogHandler from Hanging on Windows

| Q             | A
| ------------- | ---
| Branch?       | 3.3
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | -
| Fixed tickets | #22712
| License       | MIT

This resolves the issue discussed in https://github.com/symfony/symfony/issues/22712. This works on both Windows and Linux. Specifically it removes the additional hanging that was caused on Windows when attempting to write/close a TCP socket that's not open on the other end in asynchronous mode.

Commits
-------

be60aa401f [MonologBridge] Fix the Monlog ServerLogHandler from Hanging on Windows
2017-05-20 09:30:50 +02:00
Chad Sikorra
be60aa401f [MonologBridge] Fix the Monlog ServerLogHandler from Hanging on Windows 2017-05-20 09:30:49 +02:00
meyerbaptiste
c2db0c14e1 [DependencyInjection] Fix dumping of RewindableGenerator with empty IteratorArgument 2017-05-19 23:56:58 +02:00
Fabien Potencier
79bc3275db feature #22782 [Debug][VarDumper] Remove the symfony_debug C extension (nicolas-grekas)
This PR was merged into the 4.0-dev branch.

Discussion
----------

[Debug][VarDumper] Remove the symfony_debug C extension

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | yes
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

Commits
-------

f9c51a6098 [Debug][VarDumper] Remove the symfony_debug C extension
2017-05-19 22:56:51 +02:00
Fabien Potencier
a91a7de71e feature #22771 [Workflow] Removed deprecated features (lyrixx)
This PR was squashed before being merged into the 4.0-dev branch (closes #22771).

Discussion
----------

[Workflow] Removed deprecated features

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | yes
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

---

Note: all commits are atomic

Commits
-------

3bed8f6167 [Workflow] The `type` option of the `framework.workflows.*` configuration entries is `state_machine`
fd2577755d [Workflow] Removed  FrameworkBundle/DependencyInjection/Compiler/ValidateWorkflowsPass
1ccbe0bae2 [Workflow] Removed class name support in `WorkflowRegistry::add()` as second parameter
2017-05-19 22:47:20 +02:00
Fabien Potencier
649ad8cf55 bug #22768 Use 0.0.0.0 as the server log command host default. (ChadSikorra)
This PR was merged into the 3.3 branch.

Discussion
----------

Use 0.0.0.0 as the server log command host default.

| Q             | A
| ------------- | ---
| Branch?       | 3.3
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | -
| Fixed tickets | -
| License       | MIT

This fixes the `server:log` command on Windows when run with default settings. The current setting `0` has no meaning on Windows, and `0.0.0.0` is the standard "listen on all addresses/interfaces", so should be the most compatible. It works on both Linux and Windows for me this way.

I mentioned this in another PR where the logger config has the same issue on Windows:

https://github.com/symfony/symfony-standard/pull/1077

Commits
-------

3fcc810dd7 Use 0.0.0.0 as the server log host default.
2017-05-19 21:45:04 +02:00
Nicolas Grekas
f9c51a6098 [Debug][VarDumper] Remove the symfony_debug C extension 2017-05-19 19:05:40 +02:00
Maxime Steinhausser
3c3a276374 [DI][Serializer] Fix missing de(normalizer|coder) autoconfig 2017-05-19 18:52:57 +02:00
Grégoire Pineau
3bed8f6167 [Workflow] The type option of the framework.workflows.* configuration entries is state_machine 2017-05-19 17:59:15 +02:00
Grégoire Pineau
fd2577755d [Workflow] Removed FrameworkBundle/DependencyInjection/Compiler/ValidateWorkflowsPass 2017-05-19 17:59:15 +02:00
Grégoire Pineau
1ccbe0bae2 [Workflow] Removed class name support in WorkflowRegistry::add() as second parameter 2017-05-19 17:59:15 +02:00
Nicolas Grekas
31f74cad43 Merge branch '3.4'
* 3.4:
  typo
  CI fixes
  Improved how profiler errors are dispalyed on small screens
  bumped Symfony version to 3.3.0
  updated VERSION for 3.3.0-RC1
  updated CHANGELOG for 3.3.0-RC1
2017-05-19 14:18:36 +02:00
Nicolas Grekas
e9e19e7f4d Merge branch '3.3' into 3.4
* 3.3:
  typo
  CI fixes
  Improved how profiler errors are dispalyed on small screens
  bumped Symfony version to 3.3.0
  updated VERSION for 3.3.0-RC1
  updated CHANGELOG for 3.3.0-RC1
2017-05-19 14:13:29 +02:00
Nicolas Grekas
7938fdeceb Merge branch '3.2' into 3.3
* 3.2:
  typo
  CI fixes
  Improved how profiler errors are dispalyed on small screens
2017-05-19 14:13:17 +02:00
Nicolas Grekas
3aaec6f3a6 Merge branch '2.8' into 3.2
* 2.8:
  typo
2017-05-19 14:11:31 +02:00
Nicolas Grekas
fd7b084ece Merge branch '2.7' into 2.8
* 2.7:
  typo
2017-05-19 14:11:19 +02:00
Nicolas Grekas
d740342a28 typo 2017-05-19 14:11:07 +02:00
Nicolas Grekas
f154e67000 Merge branch '2.8' into 3.2
* 2.8:
  CI fixes
  Improved how profiler errors are dispalyed on small screens
2017-05-19 14:10:04 +02:00
Nicolas Grekas
f7b3252cc1 Merge branch '2.7' into 2.8
* 2.7:
  CI fixes
2017-05-19 13:49:58 +02:00
Chad Sikorra
3fcc810dd7 Use 0.0.0.0 as the server log host default. 2017-05-18 21:05:32 -05:00
Fabien Potencier
8514e22399 feature #22741 [Serializer] Remove deprecated DoctrineCache support (dunglas)
This PR was merged into the 4.0-dev branch.

Discussion
----------

[Serializer] Remove deprecated DoctrineCache support

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | no <!-- don't forget updating src/**/CHANGELOG.md files -->
| BC breaks?    | no
| Deprecations? | no <!-- don't forget updating UPGRADE-*.md files -->
| Tests pass?   | yes
| Fixed tickets | n/a
| License       | MIT
| Doc PR        | n/a

Commits
-------

be951b7096 [Serializer] Remove deprecated DoctrineCache support
2017-05-18 20:07:01 +02:00
Fabien Potencier
80e2e1f1ab bug #22752 Improved how profiler errors are displayed on small screens (javiereguiluz)
This PR was merged into the 2.8 branch.

Discussion
----------

Improved how profiler errors are displayed on small screens

| 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 change only affects the profiler menu when displaying it on small screens and collapsed:

| BEFORE | AFTER
| --- | ---
| ![before](https://cloud.githubusercontent.com/assets/73419/26194914/483ad578-3bba-11e7-85bb-94f00876b89a.png) | ![after](https://cloud.githubusercontent.com/assets/73419/26194916/492598d8-3bba-11e7-8d7c-6f37a3b00098.png)

In action:

![profiler-menu-errors](https://cloud.githubusercontent.com/assets/73419/26194969/749e7a7a-3bba-11e7-9464-d3c75fa51315.gif)

Commits
-------

3d6d80d843 Improved how profiler errors are dispalyed on small screens
2017-05-18 19:33:07 +02:00
Fabien Potencier
2f7f587de6 minor #22754 CI fixes (nicolas-grekas)
This PR was merged into the 2.7 branch.

Discussion
----------

CI fixes

| 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        | -

Commits
-------

a34b8ce2df CI fixes
2017-05-18 19:29:36 +02:00
Nicolas Grekas
a34b8ce2df CI fixes 2017-05-18 17:56:45 +02:00