Skip to content

fix: disable per-world money by default - #65

Merged
tastybento merged 1 commit into
developfrom
fix/money-default-off
Aug 4, 2026
Merged

fix: disable per-world money by default#65
tastybento merged 1 commit into
developfrom
fix/money-default-off

Conversation

@tastybento

Copy link
Copy Markdown
Member

Summary

  • options.money now defaults to false: InvSwitcher will not handle per-world money by default.
  • The per-world economy registers as the highest-priority Vault provider and takes over all economy transactions in managed worlds. On servers upgrading with another economy plugin installed (CMI, EssentialsX, etc.), this silently split the economy — payments from Vault-based plugins (e.g. Jobs, ChestShop) went into InvSwitcher's per-world balances while the server's economy plugin showed no change.
  • Servers that want per-world money can still opt in with options.money: true.
  • Version bumped to 1.19.3.

Test plan

  • mvn test — 128 tests, all passing (no test relied on the default; all set money explicitly).

🤖 Generated with Claude Code

https://claude.ai/code/session_01UM9jTon64M91vetv6rg577

InvSwitcher no longer handles per-world money by default. The economy
feature registers as the highest-priority Vault provider and takes over
all economy transactions in managed worlds, which surprises servers that
upgrade while running another economy plugin (e.g. CMI, EssentialsX).
Servers that want per-world money can opt in with options.money: true.

Bump version to 1.19.3.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UM9jTon64M91vetv6rg577
@tastybento
tastybento merged commit 6874212 into develop Aug 4, 2026
1 check passed
@tastybento tastybento mentioned this pull request Aug 4, 2026
@sonarqubecloud

sonarqubecloud Bot commented Aug 4, 2026

Copy link
Copy Markdown

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