search

nwnxee / unified

Binaries available under the Releases tab on Github
https://nwnxee.github.io/unified
GNU General Public License v3.0
131 stars 92 forks source link

Tweaks: add `NWNX_TWEAKS_CAN_USE_ITEMS_WHILE_POLYMORPHED` #1780

Closed hendrikgit closed 1 month ago

hendrikgit commented 1 month ago

I'm not sure about the Hooks::Order

Daztek commented 1 month ago

Hooks::Order::Early should be ok.

Here's what I usually follow, but it can be fuzzy

The rule for ordering should be roughly:
Earliest for pure event notification hooks
Early for skippable events, or events that modify state in before/after
Late for things that provide alternative implementation in some cases, like the POS getters in creature
Latest for things that almost never call the original
Final for things that fully reimplement base game functions
hendrikgit commented 1 month ago

Changed to Hooks::Order::Early.