Skip to content
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

[HttpFoundation] Delete not existing session handler proxy member #16607

Merged
merged 1 commit into from
Nov 23, 2015
Merged

[HttpFoundation] Delete not existing session handler proxy member #16607

merged 1 commit into from
Nov 23, 2015

Conversation

belka-ew
Copy link
Contributor

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

open() and close() SessionHandlerProxy method suse $this->active
that was removed from the parent AbstractProxy class after 2.8.

open() and close() SessionHandlerProxy method suse $this->active
that was removed from the parent AbstractProxy class after 2.8.
@Tobion
Copy link
Member

Tobion commented Nov 20, 2015

👍

Status: Reviewed

@fabpot
Copy link
Member

fabpot commented Nov 23, 2015

Thank you @belka-ew.

@fabpot fabpot merged commit b06b93f into symfony:master Nov 23, 2015
fabpot added a commit that referenced this pull request Nov 23, 2015
… member (belka-ew)

This PR was merged into the 3.0-dev branch.

Discussion
----------

[HttpFoundation] Delete not existing session handler proxy member

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

open() and close() SessionHandlerProxy method suse $this->active
that was removed from the parent AbstractProxy class after 2.8.

Commits
-------

b06b93f [HttpFoundation] Remove not existing class member
@fabpot fabpot mentioned this pull request Nov 30, 2015
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.

None yet

4 participants