From a development standpoint, wouldn't it make more sense to have this mod use the thread table instead of the datastore? That would eliminate the one query that's currently being added to each showthread pageload. After all, you only really ever care about the thread that you're currently viewing. If you wanted to get a global snapshot of all threads with bans a select where thread.btu != NULL query could be used.
I'll likely end up installing this if such a modification is made. Or, if I have some extra time, I'd be more than willing to try making it given your permission (this would be for the 3.x version of the plugin).
|