Skip to content

Changes

Started 8 hr 20 min ago
Took 49 sec on Builder

Summary

  1. Fix incorrect Tricky Trials potion durations via UpgradeManager migration (commit: 366a512) (details)
Commit 366a512b5fa83c69bcbcee51852f9dcd42976abc by nossr50
Fix incorrect Tricky Trials potion durations via UpgradeManager migration

Warriorrrr's commit (fb6aebcaa) corrected the default potions.yml shipped
with the plugin (splash: 2500->3600, lingering: 3000->900), but existing
server installs still had the old incorrect values.

This adds a one-time UpgradeManager migration (FIX_TRICKY_TRIALS_SPLASH_POTION_DURATIONS)
that auto-patches existing potions.yml on first server startup after updating.
The migration only corrects exact-match bad values; user-customized durations
are intentionally left untouched. Absent potion keys are silently skipped.

- Add UpgradeType.FIX_TRICKY_TRIALS_SPLASH_POTION_DURATIONS
- Implement PotionConfig.fixTrickyTrialsPotionDurations() + patchTrickyTrialsDurations()
- Gate migration in PotionConfig.loadPotions() via shouldUpgrade()
- Add 13 unit tests covering all migration paths in PotionConfigTest
(commit: 366a512)
The file was modifiedsrc/main/java/com/gmail/nossr50/config/skills/alchemy/PotionConfig.java (diff)
The file was modifiedChangelog.txt (diff)
The file was modifiedsrc/test/java/com/gmail/nossr50/config/skills/alchemy/PotionConfigTest.java (diff)
The file was modifiedsrc/main/java/com/gmail/nossr50/datatypes/database/UpgradeType.java (diff)