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

Implemented __sleep in yii\db\Connection to support serializing the c… #10149

Closed

Conversation

SamMousa
Copy link
Contributor

…onnection object without exceptions.

@samdark
Copy link
Member

samdark commented Nov 12, 2015

Can it be unserialized like that?

@SamMousa
Copy link
Contributor Author

Yes, I think so ^^.

@cebe
Copy link
Member

cebe commented Nov 12, 2015

could you add the unserialization to the test?

@SamMousa
Copy link
Contributor Author

Yes, will do tomorrow.

@SamMousa
Copy link
Contributor Author

Done.

@samdark
Copy link
Member

samdark commented Nov 15, 2015

Great. A line for the changelog file is needed. Then we'll merge it.

@SamMousa
Copy link
Contributor Author

I think this is mergable, it only fails on PHP7 but logs suggest that is PHP compilation error, not a source code error.

@cebe cebe self-assigned this Nov 16, 2015
@cebe cebe added this to the 2.0.7 milestone Nov 16, 2015
@cebe cebe closed this in 603a956 Nov 16, 2015
@cebe
Copy link
Member

cebe commented Nov 16, 2015

Merged, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants