From f050a988f98a9ae6138a8c46f35c67709f082115 Mon Sep 17 00:00:00 2001 From: "centron\\schwoerer" Date: Thu, 20 Nov 2025 10:16:56 +0100 Subject: [PATCH] fix(gowlers-tracking-ledger): use correct Foundry combat hooks for v11 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Change combatStart hook to createCombat (correct hook for Foundry v11) - Change combatEnd hook to deleteCombat (correct hook when combat is deleted) - Add debug logging to confirm combat hooks are registered - Update version to 0.1.10 Fixes issue where combat end was never being detected, preventing encounter status from being updated to 'finished' and XP from being linked to encounters. The console logs showed onCombatEnd was never firing because the hooks had incorrect names for Foundry v11. 🤖 Generated with Claude Code Co-Authored-By: Claude --- .../scripts/gowlers-tracking-ledger.js | 8 +++++--- src/macros_new/gowlers-tracking-ledger/module.json | 2 +- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/src/macros_new/gowlers-tracking-ledger/foundry.gowlershome.dyndns.org/modules/gowlers-tracking-ledger/scripts/gowlers-tracking-ledger.js b/src/macros_new/gowlers-tracking-ledger/foundry.gowlershome.dyndns.org/modules/gowlers-tracking-ledger/scripts/gowlers-tracking-ledger.js index 5c87af10..696030ab 100644 --- a/src/macros_new/gowlers-tracking-ledger/foundry.gowlershome.dyndns.org/modules/gowlers-tracking-ledger/scripts/gowlers-tracking-ledger.js +++ b/src/macros_new/gowlers-tracking-ledger/foundry.gowlershome.dyndns.org/modules/gowlers-tracking-ledger/scripts/gowlers-tracking-ledger.js @@ -1,6 +1,6 @@ const MODULE_ID = "gowlers-tracking-ledger"; -const MODULE_VERSION = "0.1.9"; +const MODULE_VERSION = "0.1.10"; const TRACK_SETTING = "actorSettings"; const FLAG_SCOPE = "world"; const MAX_HISTORY_ROWS = 100; @@ -146,10 +146,12 @@ async function initializeModule() { ); // Track combat encounters - Hooks.on("combatStart", (combat) => onCombatStart(combat)); - Hooks.on("combatEnd", (combat) => onCombatEnd(combat)); + Hooks.on("createCombat", (combat) => onCombatStart(combat)); + Hooks.on("deleteCombat", (combat) => onCombatEnd(combat)); Hooks.on("updateCombat", (combat) => onCombatUpdate(combat)); + console.log("[GowlersTracking] Combat hooks registered: createCombat, deleteCombat, updateCombat"); + const api = { initialized: true, openConfig: () => new TrackingLedgerConfig().render(true), diff --git a/src/macros_new/gowlers-tracking-ledger/module.json b/src/macros_new/gowlers-tracking-ledger/module.json index d2d9f7eb..a3623fc6 100644 --- a/src/macros_new/gowlers-tracking-ledger/module.json +++ b/src/macros_new/gowlers-tracking-ledger/module.json @@ -3,7 +3,7 @@ "type": "module", "title": "Gowler's Tracking Ledger", "description": "Adds HP/XP/Currency log buttons to PF1 sheets and opens the tracking dialog preloaded with the actor's logs.", - "version": "0.1.9", + "version": "0.1.10", "authors": [ { "name": "Gowler", "url": "https://foundryvtt.com" } ],