[Fix] Beastform Fixes (#1288)

* Fixed Beastforms getting stuck

* Made the wildcard image select dialog scrollable

* Beastform button shifts to cancelBeastform

module/applications/dialogs/imageSelectDialog.mjs

@@ -12,7 +12,7 @@ export default class ImageSelectDialog extends HandlebarsApplicationMixin(Applic
         tag: 'form',
         classes: ['daggerheart', 'dialog', 'dh-style', 'image-select'],
         position: {
-            width: 600,
+            width: 612,
             height: 'auto'
         },
         window: {
@@ -30,7 +30,10 @@ export default class ImageSelectDialog extends HandlebarsApplicationMixin(Applic
 
     /** @override */
     static PARTS = {
-        main: { template: 'systems/daggerheart/templates/dialogs/image-select/main.hbs' },
+        main: {
+            template: 'systems/daggerheart/templates/dialogs/image-select/main.hbs',
+            scrollable: ['.images-container']
+        },
         footer: { template: 'systems/daggerheart/templates/dialogs/image-select/footer.hbs' }
     };
 

module/applications/sheets/actors/character.mjs

@@ -31,6 +31,7 @@ export default class CharacterSheet extends DHBaseActorSheet {
             toggleEquipItem: CharacterSheet.#toggleEquipItem,
             toggleResourceDice: CharacterSheet.#toggleResourceDice,
             handleResourceDice: CharacterSheet.#handleResourceDice,
+            cancelBeastform: CharacterSheet.#cancelBeastform,
             useDowntime: this.useDowntime
         },
         window: {
@@ -219,6 +220,8 @@ export default class CharacterSheet extends DHBaseActorSheet {
             }
         };
 
+        context.beastformActive = this.document.effects.find(x => x.type === 'beastform');
+
         const homebrewCurrency = game.settings.get(CONFIG.DH.id, CONFIG.DH.SETTINGS.gameSettings.Homebrew).currency;
         if (homebrewCurrency.enabled) {
             context.inventory.currency = homebrewCurrency;
@@ -848,6 +851,15 @@ export default class CharacterSheet extends DHBaseActorSheet {
         });
     }
 
+    /**
+     *
+     */
+    static async #cancelBeastform(_, target) {
+        const item = await getDocFromElement(target);
+        if (!item) return;
+        game.system.api.fields.ActionFields.BeastformField.handleActiveTransformations.call(item);
+    }
+
     /**
      *  Open the downtime application.
      * @type {ApplicationClickAction}

module/applications/sheets/api/application-mixin.mjs

@@ -420,19 +420,6 @@ export default function DHApplicationMixin(Base) {
             ];
 
             if (usable) {
-                options.unshift({
-                    name: 'DAGGERHEART.APPLICATIONS.ContextMenu.cancelBeastform',
-                    icon: 'fa-solid fa-ban',
-                    condition: target => {
-                        const doc = getDocFromElementSync(target);
-                        return doc && doc.system?.actions?.some(a => a.type === 'beastform');
-                    },
-                    callback: async target =>
-                        game.system.api.fields.ActionFields.BeastformField.handleActiveTransformations.call(
-                            await getDocFromElement(target)
-                        )
-                });
-
                 options.unshift({
                     name: 'DAGGERHEART.GENERAL.damage',
                     icon: 'fa-solid fa-explosion',

module/data/fields/action/beastformField.mjs

@@ -41,9 +41,6 @@ export default class BeastformField extends fields.SchemaField {
      * @param {object} config    Object that contains workflow datas. Usually made from Action Fields prepareConfig methods.
      */
     static async execute(config) {
-        // Should not be useful anymore here
-        await BeastformField.handleActiveTransformations.call(this);
-
         const { selected, evolved, hybrid } = await BeastformDialog.configure(config, this.item);
         if (!selected) return false;
 

module/helpers/utils.mjs

@@ -257,9 +257,10 @@ export const updateActorTokens = async (actor, update) => {
     /* Update the tokens in all scenes belonging to Actor */
     for (let token of actor.getDependentTokens()) {
         const tokenActor = token.baseActor ?? token.actor;
-        if (tokenActor?.id === actor.id) {
+        if (token.id && tokenActor?.id === actor.id) {
             await token.update({
-                ...update
+                ...update,
+                _id: token.id
             });
         }
     }