Paradaux
← IssuesPAR-127Done
0

/business reload should actually re-read config values (not just raw config)

The existing /business reload calls message.reload() + business.reloadConfig() but the config POJOs (FirmConfiguration, BalanceTaxConfiguration) read their values once in their constructors into final fields, so services keep stale values. Give those POJOs a reload() that re-reads from plugin.getConfig(), make their fields non-final, and call them from ReloadCommand so firm.owned-limit/create-cooldown and the balance-tax brackets actually update live.

Comments

No comments yet.

Activity

  • ParadauxIO linked a commit — Commit 5b10ca3 — Release business 2.2.1 (PAR-127: /business reload refreshes config values)Jun 7, 2026, 4:06 PM
  • ParadauxIO linked a commit — Commit e54ae61 — Make /business reload actually refresh config values (PAR-127)Jun 7, 2026, 4:06 PM
  • ParadauxIO changed status to Status → DoneJun 7, 2026, 4:06 PM
  • ParadauxIO linked a pull request — PR #8 merged — Release: business 2.2.1Jun 7, 2026, 4:06 PM
  • ParadauxIO linked a pull request — PR #8 open — Release: business 2.2.1Jun 7, 2026, 4:06 PM
  • ParadauxIO linked a commit — Commit 5b10ca3 — Release business 2.2.1 (PAR-127: /business reload refreshes config values)Jun 7, 2026, 4:06 PM
  • tesks changed status to Status → Pending ReleaseJun 7, 2026, 4:04 PM
  • ParadauxIO linked a commit — Commit e54ae61 — Make /business reload actually refresh config values (PAR-127)Jun 7, 2026, 4:04 PM
  • tesks created the issueJun 7, 2026, 3:58 PM