promoted property in serialized object behaves unexpected
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C
- Sterne
- 40.4k
- Forks
- 8.2k
- Ø Merge
- 2 T. 13 Std.
- Gemergte PRs (30 T.)
- 96
Beschreibung
Description
The following code:
<?php
class Foo {
public string $bar = 'a';
public ?string $baz = null;
public function __construct(public readonly ?string $x = null) {
}
}
$fooSerialized = 'O:3:"Foo":1:{s:3:"bar";s:1:"a";}a';
$fooUnserialized = unserialize($fooSerialized);
var_dump($fooUnserialized->bar); // works because exists in serialized string
var_dump($fooUnserialized->baz); // works because regular typed property, even if not existing in serialized string
var_dump($fooUnserialized->x); // Doesn't work
Resulted in this output:
string(1) "a"
NULL
Fatal error: Uncaught Error: Typed property Foo::$x must not be accessed before initialization in /in/4d6XG:18
Stack trace:
#0 {main}
thrown in /in/4d6XG on line 18
Process exited with code 255.
But I expected this output instead:
string(1) "a"
NULL
NULL
See also https://3v4l.org/4d6XG#v8.2.6 (all versions)
PHP Version
PHP 8.2.6
Operating System
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, den bereitgestellten Reproducer für PHP 8.2.6 auszuführen, und vergleiche die Behandlung von promoted readonly properties mit der von regulären typisierten Properties während unserialize(). Verfolge das dabei relevante Serialisierungsverhalten und ergänze oder aktualisiere die Abdeckung für den Fall einer fehlenden Property; als abgeschlossen gilt die Aufgabe, wenn der reproduzierte Zugriff NULL ohne einen Initialisierungsfehler ergibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- php
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100