-
Notifications
You must be signed in to change notification settings - Fork 14
Closed
Description
The Model::$delegate feature is either a little half-baked and missing tests, or so simple that they're not needed, but either way it seems to be using the array<mixed> type which raises difficulties for static analysis. It's also complicated by the inclusion of a processed property that is just written right into the middle of the list, necessitating lots of careful runtime checking.
TODO:
- see if we can separate out
processed - convert to a
array<string, DelegateOptions>format, similar to what you see with things likeModel::$has_many.
Metadata
Metadata
Assignees
Labels
No labels