Skip to content

Conversation

@jdarwood007
Copy link
Member

Sets up a dedicated hooks table for our integrations. Hooks have become very important to our code base to ensure our mod authors are able to tap into the code where needed without needing to alter the source code. As such we should build a more efficient setup for storing and maintaining those hooks.

  • Deprecates using modSettings to store integrations.
  • Deprecates integrate_ prefix to hook names
  • Adds compatibility with SMF 2.1 logic.
  • Sets up some special handling logic for handling the pre_include (and possibly others) logic.
  • Adds logic to the package manager to handle tagging all hooks registered by a mod to it to ensure removal on uninstall.

I have toyed with the idea of fully removing all integrate_ from all files. Not sure if it's relevant to this PR. But since we are now storing hooks in their table, we don't need to have the prefix. Looking for any feedback on that, I am thinking it should be done and let the IntegrationHooks logic handle restoring it for compatibility with SMF 2.1 where needed.

I need to do more testing here as this changes a lot and I have only done some basic mod tests.

Sets up a dedicated hooks table for our integrations.

Deprecates using modSettings to store integrations.
Deprecates integrate_ prefix to hook names
Adds compatibility with SMF 2.1 logic.
@jdarwood007 jdarwood007 marked this pull request as ready for review September 20, 2025 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants