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
Having a BaseModel class is pretty ugly. It forces developers to make all their models extend this class, forcing them to commit to a class hierarchy and limiting their design choices.
The current purpose of BaseModel is to define the getValidationRules() function, which is used by CrudController. This purpose should be accomplished with a different solution, such as an interface or a trait.
Thoughts on this?
The text was updated successfully, but these errors were encountered:
Having a
BaseModel
class is pretty ugly. It forces developers to make all their models extend this class, forcing them to commit to a class hierarchy and limiting their design choices.The current purpose of
BaseModel
is to define thegetValidationRules()
function, which is used byCrudController
. This purpose should be accomplished with a different solution, such as an interface or a trait.Thoughts on this?
The text was updated successfully, but these errors were encountered: