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
Model::postSaveRelatedRecords has a lot of many-to-many complex logic mixed with normal to-many , making a clear separation of both will be easier to debug and make less checks during execution.
Model messages can be wrapped into a function that will clear up some code and add functionality.
Model::doLowUpdate could benefit a better seperation of DynamicUpdate from regular update all fields.
doLowUpdate probably does not need to add this portion in the dynamic update, if the field isn't set, no change has happened.
Model::postSaveRelatedRecords has a lot of many-to-many complex logic mixed with normal to-many , making a clear separation of both will be easier to debug and make less checks during execution.
Model messages can be wrapped into a function that will clear up some code and add functionality.
Model::doLowUpdate could benefit a better seperation of DynamicUpdate from regular update all fields.
doLowUpdate probably does not need to add this portion in the dynamic update, if the field isn't set, no change has happened.
Better Error reporting in models by adding model name and/or referenceModel to exception
Better Message reporting in models by adding model name and/or referenceModel to Message::MetaData
The text was updated successfully, but these errors were encountered: