Duplicated properties in serialisation payload

Description

Opening this as a reminder for the below.

@TimWolla: https://github.com/php/php-src/pull/22588#discussion_r3523424963

It is legal for properties to appear twice in serialization payload?! This probably should be deprecated and killed.

class TestDefault {
    public readonly int $prop = 1;
}

var_dump($s = serialize(new TestDefault));
var_dump(unserialize($s));

// Unserialization still bypasses normal readonly assignment semantics.
var_dump(unserialize("O:11:\"TestDefault\":0:{}"));
var_dump(unserialize("O:11:\"TestDefault\":1:{s:4:\"prop\";i:2;}"));
var_dump(unserialize("O:11:\"TestDefault\":2:{s:4:\"prop\";i:2;s:4:\"prop\";i:3;}"));

string(38) "O:11:"TestDefault":1:{s:4:"prop";i:1;}"
object(TestDefault)#1 (1) {
  ["prop"]=>
  int(1)
}
object(TestDefault)#1 (1) {
  ["prop"]=>
  int(1)
}
object(TestDefault)#1 (1) {
  ["prop"]=>
  int(2)
}
object(TestDefault)#1 (1) {
  ["prop"]=>
  int(3)
}

PHP Version

N/A

Operating System

No response

Inizia riproducendo gli esempi di serialize() e unserialize() nell’issue, confrontando le proprietà singole e duplicate nei campi readonly. Traccia i punti di ingresso della serializzazione e della unserializzazione responsabili del ripristino delle proprietà. Il lavoro è completato quando il comportamento previsto delle proprietà duplicate è stato concordato, compreso se debba essere contrassegnato come deprecated o rimosso, e il comportamento è coperto da test di regressione.