-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Zend: Deprecate __sleep() and __wakeup() #19435
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
base: master
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -110,6 +110,10 @@ $db = MySQLPDOTest::factory(); | |
$db->exec('DROP TABLE IF EXISTS test_stmt_fetchserialize_fetch_class'); | ||
?> | ||
--EXPECTF-- | ||
Deprecated: The __sleep() serialization hook has been deprecated. Implement __serialize() instead (or in addition, if support for old PHP versions is necessary) in %s on line %d | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Can these tests not be rewritten to use the new methods? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. No, because those tests are for the deprecated |
||
|
||
Deprecated: The __wakeup() serialization hook has been deprecated. Implement __unserialize() instead (or in addition, if support for old PHP versions is necessary) in %s on line %d | ||
|
||
Deprecated: %s implements the Serializable interface, which is deprecated. Implement __serialize() and __unserialize() instead (or in addition, if support for old PHP versions is necessary) in %s on line %d | ||
Creating an object, serializing it and writing it to DB... | ||
myclass::singleton(Creating object) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Preexisting issue: This test is missing the closing PHP tag (or rather: All of the lazy object tests appear to be). /cc @arnaud-lb