From 8f394e1d922e23384e0b3813684e37fb976932c4 Mon Sep 17 00:00:00 2001
From: Ian Woods <ianwoods1993@gmail.com>
Date: Tue, 23 Jun 2026 20:26:55 -0700
Subject: [PATCH] fix bugs

---
 Assets/_Project/Scripts/Combat/TowerCombat.cs | 9 ++++++---
 Assets/_Project/Scripts/UI/HUDController.cs   | 7 +++++--
 2 files changed, 11 insertions(+), 5 deletions(-)

diff --git a/Assets/_Project/Scripts/Combat/TowerCombat.cs b/Assets/_Project/Scripts/Combat/TowerCombat.cs
index 3498678..ebb8950 100644
--- a/Assets/_Project/Scripts/Combat/TowerCombat.cs
+++ b/Assets/_Project/Scripts/Combat/TowerCombat.cs
@@ -226,7 +226,6 @@ namespace TD.Combat
         private PlayerBuffManager GetOwnerBuffManager()
         {
             if (buffManagerResolved) return ownerBuffManager;
-            buffManagerResolved = true;
 
             var slot = towerInstance.Owner;
             if (slot == PlayerSlot.None) return null;
@@ -235,6 +234,7 @@ namespace TD.Combat
             if (matchState == null) return null;
 
             ownerBuffManager = matchState.GetComponent<PlayerBuffManager>();
+            buffManagerResolved = true;
             return ownerBuffManager;
         }
 
@@ -434,7 +434,10 @@ namespace TD.Combat
 
         private void HitEnemy(in CombatProfile p, EnemyHealth target, PlayerSlot owner)
         {
-            float effectiveDamage = p.Damage * GetOwnerBuffManager()?.GetMultiplier(BuffStat.Damage) ?? 1f;
+            var multiplier = GetOwnerBuffManager()?.GetMultiplier(BuffStat.Damage) ?? 1f;
+            float effectiveDamage = p.Damage * multiplier;
+            Debug.Log("multiplier: " + multiplier);
+            Debug.Log("Effective damage: " + effectiveDamage);
             target.TakeDamage(effectiveDamage, p.DamageType, owner);
             ApplyStatusEffect(p, target, owner);
         }
@@ -478,7 +481,7 @@ namespace TD.Combat
                 * (GetOwnerBuffManager()?.GetMultiplier(BuffStat.Damage) ?? 1f);
             proj.InitializeServer(
                 target,
-                p.Damage,
+                effectiveDamage,
                 p.DamageType,
                 p.TargetType,
                 p.SplashRadius,
diff --git a/Assets/_Project/Scripts/UI/HUDController.cs b/Assets/_Project/Scripts/UI/HUDController.cs
index 45930de..8dfe824 100644
--- a/Assets/_Project/Scripts/UI/HUDController.cs
+++ b/Assets/_Project/Scripts/UI/HUDController.cs
@@ -964,9 +964,12 @@ namespace TD.UI
 
         private VisualElement CreateBuffMenuButton(Key hotkey)
         {
+            // Key.None so HandleHotkeys doesn't register B here — BuilderInputController
+            // already fires ToggleBuffMenu on B globally. Registering it in both places
+            // causes a double-toggle on the same frame when the builder is selected.
             return CreateActionButton(
-                costText: "Buffs",
-                hotkey:   hotkey,
+                costText: "Buffs  [B]",
+                hotkey:   Key.None,
                 onClick:  () => ToggleBuffMenu());
         }