From c0b717878db607b55ed76b885b90fcec419b1b52 Mon Sep 17 00:00:00 2001
From: Brues <5278969+brues-code@users.noreply.github.com>
Date: Sun, 5 Jul 2026 23:48:02 -0500
Subject: [PATCH] Always set OnClick handler for action buttons

Ensure the action button OnClick handler is always assigned. Previously SetScript("OnClick", ButtonClick) only ran when HookScript was missing; now HookScript is still added only if absent, but the OnClick script is set unconditionally so existing frames won't miss the click handler. Change is limited to modules/actionbar.lua.
---
 modules/actionbar.lua | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/modules/actionbar.lua b/modules/actionbar.lua
index 659f7581..0f5a751d 100644
--- a/modules/actionbar.lua
+++ b/modules/actionbar.lua
@@ -1013,10 +1013,8 @@ pfUI:RegisterModule("actionbar", function ()
     -- no button available, create a new one
     if not exists then
       -- prepare the button for vanilla
-      if not f.HookScript then
-        f.HookScript = HookScript
-        f:SetScript("OnClick", ButtonClick)
-      end
+      if not f.HookScript then f.HookScript = HookScript end
+      f:SetScript("OnClick", ButtonClick)
 
       if bar ~= 11 then
         f:RegisterForDrag("LeftButton", "RightButton")