Add framework-bundle
minor #11077 [TwigBundle] [Tests] Add framework-bundle (clemens-tolboom)

This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #11077).

Discussion
----------

[TwigBundle] [Tests] Add framework-bundle

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

I'm unable to run the test from within TwigBundle

```bash
src/Symfony/Bundle/TwigBundle
composer install
phpunit
```
gives
```
PHP Fatal error:  Class 'Symfony\Bundle\FrameworkBundle\Templating\TemplateReference' not found
```

Adding the FrameworkBundle fixes this but is that what is needed?

I believe `require` and `require-dev` should use `dev-master`
- [x] require(-dev) is not up to par with symfony master.

Commits
-------

a12471d Add framework-bundle
[Filesystem] Fix test suite on OSX
[HttpFoundation] Fixed Request::getPort returns incorrect value under IPv6

Fixed issue with Request::getPort method returning an incorrect value when the HTTP_HOST header is a IPv6 address.
minor #11136 [Filesystem] Fix test suite on OSX (romainneutron)

This PR was merged into the 2.3 branch.

Discussion
----------

[Filesystem] Fix test suite on OSX

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

Reviewing PR's, I realized the filesystem test suite fails on my setup (OSX, PHP 5.5.13 with posix ext). `posix_getgrgid` returns false, so some tests are failing.
This solves this issue.

Be aware that the patched method has been moved in FilesystemTestCase in recent branches

Commits
-------

e26f08e [Filesystem] Fix test suite on OSX
bug #11144 [HttpFoundation] Fixed Request::getPort returns incorrect value under IPv6 (kicken)

This PR was merged into the 2.3 branch.

Discussion
----------

[HttpFoundation] Fixed Request::getPort returns incorrect value under IPv6

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

Commits
-------

2a0e8e3 [HttpFoundation] Fixed Request::getPort returns incorrect value under IPv6
[Console] Fixed notice in DialogHelper
bug #11169 [Console] Fixed notice in DialogHelper (florianv)

This PR was merged into the 2.3 branch.

Discussion
----------

[Console] Fixed notice in DialogHelper

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

When pressing escape in a question, a notice is shown because the sequence is only 2 characters long.

Commits
-------

ff6c65e [Console] Fixed notice in DialogHelper
Small comment update according to PSR-2

See [PSR-2](http://www.php-fig.org/psr/psr-2/) paragraph 5.2

> There MUST be a comment such as `// no break` when fall-through is intentional in a non-empty case body.

Related to #11181
minor #11182 Small comment update according to PSR-2 (apfelbox)

This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #11182).

Discussion
----------

Small comment update according to PSR-2

See [PSR-2](http://www.php-fig.org/psr/psr-2/) paragraph 5.2

> There MUST be a comment such as `// no break` when fall-through is intentional in a non-empty case body.

Related to #11181

Commits
-------

31b1dff Small comment update according to PSR-2
don't disable constructor calls to mockups of classes that extend internal PHP classes
Added missing `break` statement

Added missing `break` statement in `AbstractFindAdapter` - a fall through doesn't seem to be intended here.
bug #11186 Added missing `break` statement (apfelbox)

This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #11186).

Discussion
----------

Added missing `break` statement

Added a missing `break` statement in `AbstractFindAdapter` - a fall-through doesn't seem to be intended here.

Commits
-------

5af2802 Added missing `break` statement
minor #11187 [Tests] don't disable constructor calls to mockups of classes that extend intern... (xabbuh)

This PR was merged into the 2.3 branch.

Discussion
----------

[Tests] don't disable constructor calls to mockups of classes that extend intern...

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

Fixes the tests for the 2.3 branch as reported by @stof in #11176.

Commits
-------

2c726b8 don't disable constructor calls to mockups of classes that extend internal PHP classes
Merge branch '2.3' into 2.4

* 2.3:
  Added missing `break` statement
  don't disable constructor calls to mockups of classes that extend internal PHP classes
  Small comment update according to PSR-2
  [Console] Fixed notice in DialogHelper
  [HttpFoundation] Fixed Request::getPort returns incorrect value under IPv6
  [Filesystem] Fix test suite on OSX
  Add framework-bundle

Conflicts:
	src/Symfony/Component/Filesystem/Tests/FilesystemTest.php
	src/Symfony/Component/HttpFoundation/Tests/Session/Storage/Handler/MongoDbSessionHandlerTest.php
fixed previous merge
Merge branch '2.4' into 2.5

* 2.4:
  fixed previous merge
  Added missing `break` statement
  don't disable constructor calls to mockups of classes that extend internal PHP classes
  Small comment update according to PSR-2
  [Console] Fixed notice in DialogHelper
  [HttpFoundation] Fixed Request::getPort returns incorrect value under IPv6
  [Filesystem] Fix test suite on OSX
  Add framework-bundle

Conflicts:
	src/Symfony/Bundle/TwigBundle/composer.json
special handling for the JsonDescriptor to work around changes in PHP's JSON pretty printer
don't disable constructor calls to mockups of classes that extend internal PHP classes
minor #11191 [Tests] fix tests due to recent changes in PHP's behavior (xabbuh)

This PR was merged into the 2.4 branch.

Discussion
----------

[Tests] fix tests due to recent changes in PHP's behavior

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

The drawback of this solution is that the pretty printed string produced by ``json_encode()`` isn't validated anymore. Instead, the tests only ensures that the represented object structure is as expected. I guess that's fair enough since the goal of the test is not to test PHP's pretty printing feature.

Commits
-------

bc8042d don't disable constructor calls to mockups of classes that extend internal PHP classes
f4a3c7a special handling for the JsonDescriptor to work around changes in PHP's JSON pretty printer