Skip to content

[HttpFoundation][bugfix] $bags should always be initialized #22138

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from

Conversation

MacDada
Copy link
Contributor

@MacDada MacDada commented Mar 24, 2017

Q A
Branch? 2.7
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #21990
License MIT

@MacDada MacDada force-pushed the MockArraySessionStorage_fix branch from 1accd05 to e86f2df Compare March 24, 2017 03:36
@MacDada MacDada changed the title [HttpFoundation] Bugfix: $bags should be always initialized [HttpFoundation][bugfix] $bags should be always initialized Mar 24, 2017
@MacDada
Copy link
Contributor Author

MacDada commented Mar 24, 2017

I guess the failing tests in HttpKernel are not my fault.

@MacDada MacDada force-pushed the MockArraySessionStorage_fix branch from e86f2df to f1a1068 Compare March 24, 2017 03:55
@jakzal
Copy link
Contributor

jakzal commented Mar 24, 2017

👍

@MacDada MacDada changed the title [HttpFoundation][bugfix] $bags should be always initialized [HttpFoundation][bugfix] $bags should always be initialized Mar 24, 2017
@fabpot
Copy link
Member

fabpot commented Mar 24, 2017

Thank you @MacDada.

fabpot added a commit that referenced this pull request Mar 24, 2017
…d (MacDada)

This PR was squashed before being merged into the 2.7 branch (closes #22138).

Discussion
----------

[HttpFoundation][bugfix] $bags should always be initialized

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

Commits
-------

d984c73 [HttpFoundation][bugfix]  should always be initialized
@fabpot fabpot closed this Mar 24, 2017
This was referenced Apr 4, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants