fix: retry Vault hook before disabling so addon-provided economies work

[tastybento]

Problem

BentoBox hooks Vault during its early-hook phase, before addons are enabled. If no economy plugin has registered a provider by then, that early hook fails and is discarded, so getPlugin().getVault() is empty.

When the economy is provided by an addon rather than a standalone plugin — e.g. InvSwitcher, which registers a per-world Vault Economy in its own onEnable() — Bank's onEnable checks getVault().isEmpty(), finds it empty, and disables itself with "Vault is required" even though a working per-world economy is available.

Fix

• Retry the Vault hook before giving up. If getVault() is empty, call getPlugin().getHooks().registerHook(new VaultHook()) once, then re-check. An economy registered by an addon that enabled earlier is now picked up.
• Add InvSwitcher as a softdepend so that, when present, InvSwitcher enables (and registers its economy provider) before Bank — making the ordering deterministic rather than relying on addon load order.

Pairs with BentoBoxWorld/BentoBox#2995 (which retries the hook after all addons enable). Either change alone helps; together they make addon-provided economies robust regardless of enable order.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[tastybento]

Problem

BentoBox hooks Vault during its early-hook phase, before addons are enabled. If no economy plugin has registered a provider by then, that early hook fails and is discarded, so getPlugin().getVault() is empty.

When the economy is provided by an addon rather than a standalone plugin — e.g. InvSwitcher, which registers a per-world Vault Economy in its own onEnable() — Bank's onEnable checks getVault().isEmpty(), finds it empty, and disables itself with "Vault is required" even though a working per-world economy is available.

Fix

• Retry the Vault hook before giving up. If getVault() is empty, call getPlugin().getHooks().registerHook(new VaultHook()) once, then re-check. An economy registered by an addon that enabled earlier is now picked up.
• Add InvSwitcher as a softdepend so that, when present, InvSwitcher enables (and registers its economy provider) before Bank — making the ordering deterministic rather than relying on addon load order.

Pairs with BentoBoxWorld/BentoBox#2995 (which retries the hook after all addons enable). Either change alone helps; together they make addon-provided economies robust regardless of enable order.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[tastybento]

Problem

BentoBox hooks Vault during its early-hook phase, before addons are enabled. If no economy plugin has registered a provider by then, that early hook fails and is discarded, so getPlugin().getVault() is empty.

When the economy is provided by an addon rather than a standalone plugin — e.g. InvSwitcher, which registers a per-world Vault Economy in its own onEnable() — Bank's onEnable checks getVault().isEmpty(), finds it empty, and disables itself with "Vault is required" even though a working per-world economy is available.

Fix

• Retry the Vault hook before giving up. If getVault() is empty, call getPlugin().getHooks().registerHook(new VaultHook()) once, then re-check. An economy registered by an addon that enabled earlier is now picked up.
• Add InvSwitcher as a softdepend so that, when present, InvSwitcher enables (and registers its economy provider) before Bank — making the ordering deterministic rather than relying on addon load order.

Pairs with BentoBoxWorld/BentoBox#2995 (which retries the hook after all addons enable). Either change alone helps; together they make addon-provided economies robust regardless of enable order.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[tastybento]

Problem

BentoBox hooks Vault during its early-hook phase, before addons are enabled. If no economy plugin has registered a provider by then, that early hook fails and is discarded, so getPlugin().getVault() is empty.

When the economy is provided by an addon rather than a standalone plugin — e.g. InvSwitcher, which registers a per-world Vault Economy in its own onEnable() — Bank's onEnable checks getVault().isEmpty(), finds it empty, and disables itself with "Vault is required" even though a working per-world economy is available.

Fix

• Retry the Vault hook before giving up. If getVault() is empty, call getPlugin().getHooks().registerHook(new VaultHook()) once, then re-check. An economy registered by an addon that enabled earlier is now picked up.
• Add InvSwitcher as a softdepend so that, when present, InvSwitcher enables (and registers its economy provider) before Bank — making the ordering deterministic rather than relying on addon load order.

Pairs with BentoBoxWorld/BentoBox#2995 (which retries the hook after all addons enable). Either change alone helps; together they make addon-provided economies robust regardless of enable order.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[tastybento]

Problem

BentoBox hooks Vault during its early-hook phase, before addons are enabled. If no economy plugin has registered a provider by then, that early hook fails and is discarded, so getPlugin().getVault() is empty.

When the economy is provided by an addon rather than a standalone plugin — e.g. InvSwitcher, which registers a per-world Vault Economy in its own onEnable() — Bank's onEnable checks getVault().isEmpty(), finds it empty, and disables itself with "Vault is required" even though a working per-world economy is available.

Fix

• Retry the Vault hook before giving up. If getVault() is empty, call getPlugin().getHooks().registerHook(new VaultHook()) once, then re-check. An economy registered by an addon that enabled earlier is now picked up.
• Add InvSwitcher as a softdepend so that, when present, InvSwitcher enables (and registers its economy provider) before Bank — making the ordering deterministic rather than relying on addon load order.

Pairs with BentoBoxWorld/BentoBox#2995 (which retries the hook after all addons enable). Either change alone helps; together they make addon-provided economies robust regardless of enable order.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[tastybento]

Problem

BentoBox hooks Vault during its early-hook phase, before addons are enabled. If no economy plugin has registered a provider by then, that early hook fails and is discarded, so getPlugin().getVault() is empty.

When the economy is provided by an addon rather than a standalone plugin — e.g. InvSwitcher, which registers a per-world Vault Economy in its own onEnable() — Bank's onEnable checks getVault().isEmpty(), finds it empty, and disables itself with "Vault is required" even though a working per-world economy is available.

Fix

• Retry the Vault hook before giving up. If getVault() is empty, call getPlugin().getHooks().registerHook(new VaultHook()) once, then re-check. An economy registered by an addon that enabled earlier is now picked up.
• Add InvSwitcher as a softdepend so that, when present, InvSwitcher enables (and registers its economy provider) before Bank — making the ordering deterministic rather than relying on addon load order.

Pairs with BentoBoxWorld/BentoBox#2995 (which retries the hook after all addons enable). Either change alone helps; together they make addon-provided economies robust regardless of enable order.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[tastybento]

Problem

BentoBox hooks Vault during its early-hook phase, before addons are enabled. If no economy plugin has registered a provider by then, that early hook fails and is discarded, so getPlugin().getVault() is empty.

When the economy is provided by an addon rather than a standalone plugin — e.g. InvSwitcher, which registers a per-world Vault Economy in its own onEnable() — Bank's onEnable checks getVault().isEmpty(), finds it empty, and disables itself with "Vault is required" even though a working per-world economy is available.

Fix

• Retry the Vault hook before giving up. If getVault() is empty, call getPlugin().getHooks().registerHook(new VaultHook()) once, then re-check. An economy registered by an addon that enabled earlier is now picked up.
• Add InvSwitcher as a softdepend so that, when present, InvSwitcher enables (and registers its economy provider) before Bank — making the ordering deterministic rather than relying on addon load order.

Pairs with BentoBoxWorld/BentoBox#2995 (which retries the hook after all addons enable). Either change alone helps; together they make addon-provided economies robust regardless of enable order.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[tastybento]

Problem

BentoBox hooks Vault during its early-hook phase, before addons are enabled. If no economy plugin has registered a provider by then, that early hook fails and is discarded, so getPlugin().getVault() is empty.

When the economy is provided by an addon rather than a standalone plugin — e.g. InvSwitcher, which registers a per-world Vault Economy in its own onEnable() — Bank's onEnable checks getVault().isEmpty(), finds it empty, and disables itself with "Vault is required" even though a working per-world economy is available.

Fix

• Retry the Vault hook before giving up. If getVault() is empty, call getPlugin().getHooks().registerHook(new VaultHook()) once, then re-check. An economy registered by an addon that enabled earlier is now picked up.
• Add InvSwitcher as a softdepend so that, when present, InvSwitcher enables (and registers its economy provider) before Bank — making the ordering deterministic rather than relying on addon load order.

Pairs with BentoBoxWorld/BentoBox#2995 (which retries the hook after all addons enable). Either change alone helps; together they make addon-provided economies robust regardless of enable order.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud