R
ragtek
Guest
Wouldn't it be better if there's only 1 responsible method for deleting posts?
If i delete an post via the direct link in the postbit, the code runs postmodel::deletePost
if i delete it via inlinemod option, it deletes the posts direct in XenForo_Model_InlineMod_Post::deletePosts
if only 1 method is responsible for deleting(i'm only talking about deleting, not permission check etc..) , it would be IMO easier for 3rd party devs to change the behaviour..
Now i need to extend XenForo_Model_InlineMod_Post and the post model to change the behaviour
If i delete an post via the direct link in the postbit, the code runs postmodel::deletePost
if i delete it via inlinemod option, it deletes the posts direct in XenForo_Model_InlineMod_Post::deletePosts
if only 1 method is responsible for deleting(i'm only talking about deleting, not permission check etc..) , it would be IMO easier for 3rd party devs to change the behaviour..
Now i need to extend XenForo_Model_InlineMod_Post and the post model to change the behaviour
Upvote
0