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
The plugin currently uses front-end hooks, but it is possible threads or post get inserted or updated out of the scope of the front-end, so ideally data-handler hooks should be used.
I suppose the same could be true about other non-post hooks. For easy of communication I share next a comparison of best practice hooks compared to the current ones being used :
I didn't check how this plugin handles content deletion so I won't suggest that here. Also, this might not be urgent so probably an improvement for later versions.
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
Uh oh!
There was an error while loading. Please reload this page.
-
The plugin currently uses front-end hooks, but it is possible threads or post get inserted or updated out of the scope of the front-end, so ideally data-handler hooks should be used.
I suppose the same could be true about other non-post hooks. For easy of communication I share next a comparison of best practice hooks compared to the current ones being used :
newthread_do_newthread_end=>datahandler_post_insert_thread_endxmlhttp_update_post=>datahandler_post_update_endeditpost_do_editpost_end=>datahandler_post_update_endnewreply_do_newreply_end=>datahandler_post_insert_post_endmember_do_register_end=>datahandler_user_insert_endI didn't check how this plugin handles content deletion so I won't suggest that here. Also, this might not be urgent so probably an improvement for later versions.
Beta Was this translation helpful? Give feedback.
All reactions