You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jul 16, 2023. It is now read-only.
It makes little sense to have Ardent as an optional improvement (as a trait), given it would poses confusion to developers if a model has Ardent improvements, and another one don't.
However, if you're not using a relational database, you're probably not extending Eloquent directly - you're using an extension that implements, say, NoSQL. Thus, Ardent cannot be extended at the same time. In these cases, Ardent makes sense being used as a Trait.
It makes little sense to have Ardent as an optional improvement (as a trait), given it would poses confusion to developers if a model has Ardent improvements, and another one don't.
However, if you're not using a relational database, you're probably not extending Eloquent directly - you're using an extension that implements, say, NoSQL. Thus, Ardent cannot be extended at the same time. In these cases, Ardent makes sense being used as a Trait.
This would help users of Mongo + Ardent, that currently needs to reimplement some stuff: ardent-mongodb by @bmcmurray and laravel-mongo-ardent by @Fuitad, for instance.
The text was updated successfully, but these errors were encountered: