Tagging Plugin v1.2.2 Review

[flash.tato]

Thank you for your review.

I understood your plugin review and i think your points are right (i could've made a single query instead of n-queries and avoided some redudant code by writing a function

Quote:UserCP action should probably check/verify postkey like MyBB does

You're right i forgot to do, i think.

Quote:Plugin doesn't seem to handle copy threads, split posts etc, but I'll say that handling all these would probably really be a pain.  [b]The merge post handler/b] may not always work (delayed moderation?)

Quote:$plugins->add_hook("moderation_start", "tagging_on_merge_posts");

Doesn't "tagging_on_merge_posts" do its job in merging posts? Maybe i don't understand what you mean that it is probably.

Ah and as last thing.

Thank you for your time, i think your points will be helpful for the future i like very much your analyzing abilities.