php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Doc Bug #81675 Undocumented change in get_class_vars behaviour with static properties in 8.1
Submitted: 2021-11-30 10:56 UTC Modified: 2021-11-30 14:24 UTC
Votes:6
Avg. Score:4.2 ± 0.7
Reproduced:5 of 5 (100.0%)
Same Version:5 (100.0%)
Same OS:3 (60.0%)
From: diede at yoast dot com Assigned:
Status: Verified Package: Scripting Engine problem
PHP Version: 8.1.0 OS:
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: diede at yoast dot com
New email:
PHP Version: OS:

 

 [2021-11-30 10:56 UTC] diede at yoast dot com 
Description:
------------
I have a feeling that `get_class_vars()` received a bugfix. Previously it included the current value of a static property, but now returns the default/initial value instead. This now matches the documentation, but the behaviour of previous PHP versions didn't match the documentation.

I can't find anything in the changelogs, docs and RFCs. Here's where we looked:
- https://github.com/php/php-src/blob/php-8.1.0/UPGRADING
- https://www.php.net/manual/en/migration81.php
- https://www.php.net/manual/en/function.get-class-vars.php
- https://wiki.php.net/rfc#php_81
- https://github.com/php/php-src/blob/php-8.1.0/NEWS


This could at the very least be considered a breaking change and should be documented.


Test script:
---------------
<?php
// https://3v4l.org/RjVk0
class MyClass {
    public static $myProperty;
}

MyClass::$myProperty = 3;
var_dump( get_class_vars( MyClass::class ) );

Expected result:
----------------
// Output for 7.3.0 - 7.3.33, 7.4.0 - 7.4.26, 8.0.0 - 8.0.13
array(1) {
  ["myProperty"]=>
  int(3)
}

Actual result:
--------------
// Output for 8.1.0
array(1) {
  ["myProperty"]=>
  NULL
}

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2021-11-30 12:21 UTC] cmb@php.net
-Type: Bug +Type: Documentation Problem -Package: *General Issues +Package: Scripting Engine problem
 [2021-11-30 12:21 UTC] cmb@php.net 
3v4l.org runs without OPcache.  With OPcache enabled, you already
got the same result as with PHP 8.1.  This inconsistency[1] has
now been fixed[2]; I'm not sure whether this should be documented,
given that OPcache is likely always enabled in production
environments at least.

[1] <https://bugs.php.net/bug.php?id=80821>
[2] <https://github.com/php/php-src/commit/3eb97a456648c739533d92c81102cb919eab01c9>
 [2021-11-30 13:40 UTC] bugs dot php dot net_nospam at adviesenzo dot nl 
> I'm not sure whether this should be documented,
given that OPcache is likely always enabled in production
environments at least.

Well, considering it took us a not insignificant amount of time to pin down what was causing the difference in behaviour and that the change removes a difference between typical production vs develop environments which people may have run into and got stuck on trying to figure out before, I think documenting this would be useful, if for no other reason than to save other people running into the same/similar situation debug time.
 [2021-11-30 14:11 UTC] nikic@php.net 
As the behavior was there for quite a long time, adding a note for buggy behavior with PHP < 8.1 without opcache seems reasonable.
 [2021-11-30 14:24 UTC] cmb@php.net
-Status: Open +Status: Verified
 [2022-03-03 12:45 UTC] ghsdf6sdf at protonmail dot com 
Thanks 

https://altosaxo.net/products/pistol-annies-t-shirt-men 
https://altosaxo.net/products/makthaverskan-t-shirt-men 
https://altosaxo.net/products/cloud-nothings-t-shirt-men 
https://altosaxo.net/products/courteeners-t-shirt-men 
https://altosaxo.net/products/peach-tree-rascals-t-shirt-men
 [2022-03-03 12:46 UTC] hsf45y at protonmail dot com 
Thanks 

https://altosaxo.net/products/screaming-trees-t-shirt-men 
https://altosaxo.net/products/frenzal-rhomb-t-shirt-men 
https://altosaxo.net/products/despised-icon-t-shirt-men 
https://altosaxo.net/products/buffalo-daughter-t-shirt-men 
https://altosaxo.net/products/the-warlocks-t-shirt-men
 [2022-03-03 12:47 UTC] hfsd65y at protonmail dot com 
Thanks 

https://altosaxo.net/products/the-revivalists-t-shirt-men 
https://altosaxo.net/products/papooz-t-shirt-men 
https://altosaxo.net/products/los-lobos-t-shirt-men 
https://altosaxo.net/products/black-star-riders-t-shirt-men 
https://altosaxo.net/products/collective-soul-t-shirt-men
 [2022-09-22 10:07 UTC] pedder55655 at gmx dot net 
I can can confirm this. 

I use a static object for localisation and modify the static variables upon load-up with those from the user's preference. After upgrade to php 8.1, they all see the default language as my property assignments have no effect anymore.
 [2023-01-12 05:09 UTC] samira dot akhlaqi314 at gmail dot com 
A really good post, very thankful and hopeful that you will write many more posts like this one.		


(https://www.chipotlefeedback.me/)php.net
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Mon Dec 30 14:01:28 2024 UTC