(WIP) feat: ✨ use active effects to sync conditions [fix #378]

.changeset/ten-papayas-camp.md

@@ -0,0 +1,5 @@
+---
+"foundry-types": minor
+---
+
+Move conditions to ActiveEffects

src/actor/actor-document.js

@@ -127,22 +127,29 @@ export class ForbiddenLandsActor extends Actor {
 	}
 
 	toggleCondition(conditionName) {
-		const conditionValue = this.conditions[conditionName].value;
-		const conditionLabel = this.conditions[conditionName].label;
-		const effect = this.effects.find(
-			(condition) => condition.getFlag("core", "statusId") === conditionName,
+		const statusEffect = CONFIG.statusEffects.find(
+			(it) => it.id === conditionName,
 		);
-		if (CONFIG.fbl.conditions.includes(conditionName)) {
-			this.update({
-				[`system.condition.${conditionName}.value`]: !conditionValue,
-			});
-			if (conditionValue && effect)
-				this.deleteEmbeddedDocuments("ActiveEffect", [effect.id]);
-			else if (!conditionValue && !effect)
-				this.createEmbeddedDocuments("ActiveEffect", {
-					...CONFIG.fbl.activeEffects[conditionName],
-					label: localize(conditionLabel),
-				});
+		const currentEffect = Array.from(this.effects?.values()).find(
+			(it) => it.icon === statusEffect.icon,
+		);
+		if (currentEffect) {
+			this.deleteEmbeddedDocuments("ActiveEffect", [currentEffect.id]);
+		} else {
+			this.createEmbeddedDocuments("ActiveEffect", [
+				{
+					label: game.i18n.localize(statusEffect.label),
+					icon: statusEffect.icon,
+					changes: statusEffect.changes,
+					id: this.uuid,
+					statuses: statusEffect.statuses,
+					flags: {
+						core: {
+							statusId: statusEffect.id,
+						},
+					},
+				},
+			]);
 		}
 	}
 

src/actor/actor-sheet.js

@@ -20,6 +20,8 @@ export class ForbiddenLandsActorSheet extends ActorSheet {
 		data.carriedStates = this.#getCarriedStates();
 		data.gear = this.#filterGear(data.items);
 		data.system.useHealthAndResolve = this.useHealthAndResolve;
+		data.system.condition = this.actor.system.condition;
+		data.statuses = this.actor.statuses;
 
 		return data;
 	}

src/forbidden-lands.ts

@@ -28,6 +28,7 @@ import registerSettings from "@system/core/settings.js";
 import { registerSheets } from "@system/core/sheets.js";
 import localizeString from "@utils/localize-string.js";
 import { YearZeroRollManager } from "foundry-year-zero-roller";
+import { ForbiddenLandsTokenHUD } from "@system/core/hud.js";
 
 /**
  * We use this label to remove the debug option in production builds.
@@ -48,6 +49,7 @@ Hooks.once("init", () => {
 		config: FBL,
 		roll: FBLRollHandler.createRoll,
 	};
+
 	CONFIG.Actor.documentClass = ForbiddenLandsActor;
 	CONFIG.Combat.documentClass = FBLCombat;
 	// @ts-expect-error - PF2 types Internal Type Error
@@ -57,6 +59,7 @@ Hooks.once("init", () => {
 	CONFIG.JournalEntry.documentClass = ForbiddenLandsJournalEntry;
 	// @ts-expect-error - PF2 types Internal Type Error
 	CONFIG.ui.combat = FBLCombatTracker;
+	CONFIG.statusEffects = FBL.statusEffects;
 	CONFIG.fbl = FBL;
 	CONFIG.fbl.adventureSites.utilities = utilities;
 	CONFIG.fbl.adventureSites.generate = (path: string, adventureSite: unknown) =>
@@ -107,3 +110,7 @@ Hooks.once("ready", () => {
 			});
 		});
 });
+
+Hooks.on("canvasReady", (canvas) => {
+	canvas.hud.token = new ForbiddenLandsTokenHUD();
+});