Should we add support for property hooks in the package denormalization feature.

[nyamsprod]

Feature Request

Q	A
New Feature	yes
BC Break	yes

Issue

class UserIdentity
{ 
    public function __construct(
        private string $firstName,
        private string $lastName
    ) {
    }
 
    public string $fullName {
        get => $this->firstName . " " . $this->lastName;
        set {
            [$this->firstName, $this->lastName] = explode(' ', $value, 2);
        }
    }
}

• How should the deserialization feature works with this class ?

[nyamsprod]

Should the Deserialization feature of the package even try to work correctly with a class containing property hook ?

The current Deserialization mechanism is geared toward simple DTO. The moment you add Property hooks or any advanced feature in PHP to your class you are better of using a fully fledge Serializer package and there are many of them in PHP land. So should Property hook even be supported by the feature ?

[shadowhand]

@nyamsprod in my opinion, this package should not try to support advanced deserialization. As you said, there are many other packages available for those use cases. I appreciate how pure this package is and don't want to see it overlap with others.