Opt-in support for property hooks in `IsDatabaseModel`

[gglnx]

Description

I've got a trait to generate a public ID (Stripe-style with prefix, e.g. ws_9mJrX3hPMFMRPx3X):

trait HasPublicId
{
    #[IsBindingValue]
    public PublicId $publicId {
        get {
            $this->publicId ??= PublicId::create(self::idPrefix());

            return $this->publicId;
        }
        set(string|PublicId $value) {
            $this->publicId = new PublicId($value);
        }
    }

    /**
     * @return non-empty-lowercase-string
     */
    public static function idPrefix(): string
    {
        $classReflector = new ClassReflector(self::class);
        $objectIdPrefix = $classReflector->getAttribute(PublicIdPrefix::class)
            ?? throw new MissingAttributeException(PublicIdPrefix::class);

        return $objectIdPrefix->prefix;
    }
}

Currently IsDatabaseModel / ModelInspector ignores it. Any way to add an option for an opt-in via an attribute?

[brendt]

You mean it should be persisted to the database, right?

[gglnx]

Yes :)

[brendt]

Gotcha, and I reckon you missed a check here? Just making sure I properly understand the example

        set(string|PublicId $value) {
            if (!$value instanceof PublicId) {$value = new PublicId($value);}

            $this->publicId = $value;
        }

[gglnx]

Oh, yeah, that would be even better. Currently the check lives in PublicId::__constructor(). Thanks!

[brendt]

Could you try it out with this branch? #2215

composer require tempest/framework:dev-persist-hooked-properties

[gglnx]

I tested it and inspect($this)->getPropertyValues(); in IsDatabaseModel still don't include my property hook. So inserting fails.