- Why are you distributing default templates in this modification? By doing this, you are overwriting the default vBulletin templates in the MASTER_STYLE, making it impossible to revert to the original templates, even after uninstalling this modification. Doing this is an absolute "Don't" in my view. On the other hand, this modification will probably stop working if any of the default templates used by this modification is ever updated in the default vBulletin package.
- Why are you processing only at the 'global_start' hook location? I strongly suggest to use more specific hook locations to avoid code being run on pages where they don't have any function. This is bad for performance.