From 16e931179fe4b722ff7d80fcd20b3c7ebc564168 Mon Sep 17 00:00:00 2001
From: Dapoulp <74197441+Dapoulp@users.noreply.github.com>
Date: Fri, 15 Aug 2025 21:43:33 +0200
Subject: [PATCH] Fix/attack type migrate data (#949)

* Temp ActionField attack type missing

* Move missing attack type to getModel

* Move attack type fix to actor migratedata
---
 module/data/fields/actionField.mjs | 1 -
 module/documents/actor.mjs         | 8 ++++++++
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/module/data/fields/actionField.mjs b/module/data/fields/actionField.mjs
index d2fdc6c8..04425d55 100644
--- a/module/data/fields/actionField.mjs
+++ b/module/data/fields/actionField.mjs
@@ -82,7 +82,6 @@ export class ActionsField extends MappingField {
  */
 export class ActionField extends foundry.data.fields.ObjectField {
     getModel(value) {
-        if (value && !value.type) value.type = 'attack';
         return game.system.api.models.actions.actionsTypes[value.type] ?? null;
     }
 
diff --git a/module/documents/actor.mjs b/module/documents/actor.mjs
index 05c4cbaa..84dd6d18 100644
--- a/module/documents/actor.mjs
+++ b/module/documents/actor.mjs
@@ -25,6 +25,14 @@ export default class DhpActor extends Actor {
 
     /* -------------------------------------------- */
 
+    /** @inheritDoc */
+    static migrateData(source) {
+        if(source.system?.attack && !source.system.attack.type) source.system.attack.type = "attack";
+        return super.migrateData(source);
+    }
+
+    /* -------------------------------------------- */
+
     /**@inheritdoc */
     static getDefaultArtwork(actorData) {
         const { type } = actorData;