Commit dc41f92
fix: bonus attack coefficient broken for two-handed weapon classes (#666)
Two bugs in the bonus attack calculation caused 7/11 classes to deal
drastically reduced damage:
1. StatsManager.BonusAttackCoefficient had EquipSlot reads swapped —
rightHandRarity read from EquipSlot.LH and leftHandRarity read
from EquipSlot.RH. Two-handed weapons store in EquipSlot.RH as
their primary slot, so the swapped reads made rightHandRarity=0,
causing BonusAttack.Coefficient to return 0.
2. BonusAttack.Coefficient early-returned the raw RarityMultiplier
when leftHandRarity was 0, skipping the 4.96 * JobBonusMultiplier
scaling. Two-handed classes got ~5.7x less bonus attack.
Affected classes: Berserker, Wizard, Archer, Heavy Gunner, Rune Blader,
Striker, Soul Binder.
Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>1 parent 289770c commit dc41f92
2 files changed
Lines changed: 4 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | | - | |
13 | | - | |
| 12 | + | |
| 13 | + | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
17 | 16 | | |
18 | 17 | | |
19 | 18 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
76 | | - | |
77 | | - | |
| 76 | + | |
| 77 | + | |
78 | 78 | | |
79 | 79 | | |
80 | 80 | | |
| |||
0 commit comments