I don't understand why this is enforced in symfony/validator. It means I can't have a base class with a `final public static` method on it that I use as a common validator for all models. I have to define the static method on every model.
Discussion
Loading...
@ramsey The validator component merges metadata of the parent class with metadata of the child class. This condition is there to avoid duplicating constraint. It should not prevent your use case.
@stof I’m using addMethodMappings(), and I have step-debugged and found that it is preventing my use-case.
@ramsey is your step debugging taking into account the merging of constraints coming from parent classes in https://github.com/symfony/symfony/blob/b49a855dcfc018f496edec2cf1606629639461a8/src/Symfony/Component/Validator/Mapping/Factory/LazyLoadingMetadataFactory.php#L108 ?
@stof loadClassMetadata() returns false for this method and it never calls the static method that I want it to call.
@ramsey looks like a decision that was made over a decade ago. It sounds probably be re-evaluated in the context of modern PHP
https://github.com/symfony/validator/commit/6e6bd0f1928d1824ce91c209169024d53cf3ff6d
@nCrazed As I initially suspected: PEBKAC
bonfire.cafe
A space for Bonfire maintainers and contributors to communicate
Automatic federation enabled