Skip to content

fix: CMI and Vault registration issue - #576

Closed
Exanthiax wants to merge 1 commit into
masterfrom
fix/cmi_vault_integration
Closed

fix: CMI and Vault registration issue#576
Exanthiax wants to merge 1 commit into
masterfrom
fix/cmi_vault_integration

Conversation

@Exanthiax

Copy link
Copy Markdown
Member

No description provided.

The Vault economy integration was resolved once, synchronously, during
plugin onEnable. If the actual economy provider (e.g. CMI) registers
with Vault's ServicesManager after that point - which can happen since
plugin enable order between two plugins that both merely soft-depend
on Vault isn't guaranteed - the lookup returned null and no economy
integration was ever registered. EconomyManager then failed silently
on every balance operation for the rest of the server's uptime.

Add a ServiceRegisterEvent/ServiceUnregisterEvent listener that keeps
the registered Vault integration in sync with whatever Economy
provider Vault currently has, instead of relying on a single lookup
at boot.
@Exanthiax Exanthiax closed this Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant