php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #66568 Update reflection information for unserialize() function
Submitted: 2014-01-24 05:55 UTC Modified: 2014-04-13 00:48 UTC
Votes:1
Avg. Score:3.0 ± 0.0
Reproduced:0 of 0 (0.0%)
From: lisachenko dot it at gmail dot com Assigned: tyrael
Status: Closed Package: Reflection related
PHP Version: 5.6.0alpha1 OS: Windows
Private report: No CVE-ID:
 [2014-01-24 05:55 UTC] lisachenko dot it at gmail dot com
Description:
------------
For version PHP5.6 additional parameter $consumed was added to the unserialize() method, but it's marked as required, need to make it optional and add this parameter into official documentation.


Test script:
---------------
$ /c/php/php-5.6.0alpha1-nts-Win32-VC11-x86/php --rf unserialize


Expected result:
----------------
Function [ <internal:standard> function unserialize ] {

  - Parameters [2] {
    Parameter #0 [ <required> $variable_representation ]
    Parameter #1 [ <optional> &$consumed ]
  }
}

Actual result:
--------------
Function [ <internal:standard> function unserialize ] {

  - Parameters [2] {
    Parameter #0 [ <required> $variable_representation ]
    Parameter #1 [ <required> &$consumed ]
  }
}

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2014-01-24 05:57 UTC] lisachenko dot it at gmail dot com
-Summary: Update reflection information for unserialize() method +Summary: Update reflection information for unserialize() function
 [2014-01-24 05:57 UTC] lisachenko dot it at gmail dot com
Change "method" to "function" in the summary
 [2014-01-27 18:28 UTC] pajoye@php.net
Not related to windows only
 [2014-04-13 00:44 UTC] tyrael@php.net
-Status: Open +Status: Assigned -Assigned To: +Assigned To: tyrael
 [2014-04-13 00:47 UTC] tyrael@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=aefc8f22b35999624f3a579696cf68b389f41ae8
Log: Fix bug #66568: the second arg for unserialize is optional.
 [2014-04-13 00:47 UTC] tyrael@php.net
-Status: Assigned +Status: Closed
 [2014-04-13 00:47 UTC] tyrael@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=aefc8f22b35999624f3a579696cf68b389f41ae8
Log: Fix bug #66568: the second arg for unserialize is optional.
 [2014-04-13 00:48 UTC] tyrael@php.net
The fix for this bug has been committed.

Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.

 For Windows:

http://windows.php.net/snapshots/
 
Thank you for the report, and for helping us make PHP better.


 [2014-04-15 12:04 UTC] ab@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=aefc8f22b35999624f3a579696cf68b389f41ae8
Log: Fix bug #66568: the second arg for unserialize is optional.
 [2014-04-15 13:05 UTC] ab@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=aefc8f22b35999624f3a579696cf68b389f41ae8
Log: Fix bug #66568: the second arg for unserialize is optional.
 [2014-05-01 14:59 UTC] tyrael@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=aefc8f22b35999624f3a579696cf68b389f41ae8
Log: Fix bug #66568: the second arg for unserialize is optional.
 [2014-07-25 11:15 UTC] tyrael@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=a68ef0a117b0e7da4fe9ab1ee098c6e5e0553c4a
Log: Revert &quot;Fix bug #66568: the second arg for unserialize is optional.&quot; as we already reverted the introduction of the second arg.
 [2014-07-25 11:16 UTC] tyrael@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=a68ef0a117b0e7da4fe9ab1ee098c6e5e0553c4a
Log: Revert &quot;Fix bug #66568: the second arg for unserialize is optional.&quot; as we already reverted the introduction of the second arg.
 [2014-07-30 09:52 UTC] tyrael@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=a68ef0a117b0e7da4fe9ab1ee098c6e5e0553c4a
Log: Revert &quot;Fix bug #66568: the second arg for unserialize is optional.&quot; as we already reverted the introduction of the second arg.
 [2014-08-04 08:26 UTC] ab@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=a68ef0a117b0e7da4fe9ab1ee098c6e5e0553c4a
Log: Revert &quot;Fix bug #66568: the second arg for unserialize is optional.&quot; as we already reverted the introduction of the second arg.
 [2014-08-04 08:43 UTC] ab@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=a68ef0a117b0e7da4fe9ab1ee098c6e5e0553c4a
Log: Revert &quot;Fix bug #66568: the second arg for unserialize is optional.&quot; as we already reverted the introduction of the second arg.
 [2014-08-04 09:59 UTC] dmitry@php.net
Automatic comment on behalf of tyrael
Revision: http://git.php.net/?p=php-src.git;a=commit;h=a68ef0a117b0e7da4fe9ab1ee098c6e5e0553c4a
Log: Revert &quot;Fix bug #66568: the second arg for unserialize is optional.&quot; as we already reverted the introduction of the second arg.
 
PHP Copyright © 2001-2017 The PHP Group
All rights reserved.
Last updated: Tue Feb 21 23:01:36 2017 UTC