Paradaux
Issues

PAR-127

0

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

DoneUnassignedBusiness

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.

Resources

Comments

No comments yet.

Activity

  • ParadauxIO linked a commit — Commit 5b10ca3 — Release business 2.2.1 (PAR-127: /business reload refreshes config values)
  • ParadauxIO linked a commit — Commit e54ae61 — Make /business reload actually refresh config values (PAR-127)
  • ParadauxIO changed status to Status → Done
  • ParadauxIO linked a pull request — PR #8 merged — Release: business 2.2.1
  • ParadauxIO linked a pull request — PR #8 open — Release: business 2.2.1
  • ParadauxIO linked a commit — Commit 5b10ca3 — Release business 2.2.1 (PAR-127: /business reload refreshes config values)
  • tesks changed status to Status → Pending Release
  • ParadauxIO linked a commit — Commit e54ae61 — Make /business reload actually refresh config values (PAR-127)
  • tesks created the issue