From 0f1ac406df4a4e866ea089798d131ef73af827eb Mon Sep 17 00:00:00 2001
From: WBHarry <williambjrklund@gmail.com>
Date: Thu, 2 Apr 2026 12:22:31 +0200
Subject: [PATCH] Fixed adding a new damage instance to an action default to
 prof, which doesn't work for adversaries

---
 .../applications/sheets-configs/action-config.mjs |  1 -
 module/data/fields/action/damageField.mjs         |  4 ++--
 templates/actionTypes/effect.hbs                  | 15 ---------------
 3 files changed, 2 insertions(+), 18 deletions(-)

diff --git a/module/applications/sheets-configs/action-config.mjs b/module/applications/sheets-configs/action-config.mjs
index e15b0391..e073cfa6 100644
--- a/module/applications/sheets-configs/action-config.mjs
+++ b/module/applications/sheets-configs/action-config.mjs
@@ -21,7 +21,6 @@ export default class DHActionConfig extends DHActionBaseConfig {
 
     static async addEffect(_event) {
         if (!this.action.effects) return;
-        const effectData = this._addEffectData.bind(this)();
         const data = this.action.toObject();
 
         const created = await this.action.item.createEmbeddedDocuments('ActiveEffect', [
diff --git a/module/data/fields/action/damageField.mjs b/module/data/fields/action/damageField.mjs
index d41d2d4a..5d40a470 100644
--- a/module/data/fields/action/damageField.mjs
+++ b/module/data/fields/action/damageField.mjs
@@ -232,7 +232,7 @@ export class DHActionDiceData extends foundry.abstract.DataModel {
         return {
             multiplier: new fields.StringField({
                 choices: CONFIG.DH.GENERAL.multiplierTypes,
-                initial: 'prof',
+                initial: 'flat',
                 label: 'DAGGERHEART.ACTIONS.Config.damage.multiplier',
                 nullable: false,
                 required: true
@@ -244,7 +244,7 @@ export class DHActionDiceData extends foundry.abstract.DataModel {
             }),
             dice: new fields.StringField({
                 choices: CONFIG.DH.GENERAL.diceTypes,
-                initial: 'd6',
+                initial: CONFIG.DH.GENERAL.diceTypes.d6,
                 label: 'DAGGERHEART.GENERAL.Dice.single',
                 nullable: false,
                 required: true
diff --git a/templates/actionTypes/effect.hbs b/templates/actionTypes/effect.hbs
index dd6a7974..8c5e6e68 100644
--- a/templates/actionTypes/effect.hbs
+++ b/templates/actionTypes/effect.hbs
@@ -4,21 +4,6 @@
         <a><i class="fa-solid fa-plus icon-button" data-action="addEffect"></i></a>
     </legend>
     <ul class="items-list">
-        {{!-- {{#each source as | effect index |}}
-            <div class="inventory-item" data-effect-id="{{effect._id}}" data-action="editEffect">
-                {{#with (@root.getEffectDetails effect._id) as | details |}}
-                        <img class="item-img" src="{{img}}">
-                    <div class="item-label">
-                        <span class="item-name">{{name}}</span>
-                    </div>
-                {{/with}}
-                <input type="hidden" name="effects.{{index}}._id" value="{{effect._id}}">
-                {{#if @root.source.save.trait}}{{formInput ../fields.onSave value=effect.onSave name=(concat "effects." index ".onSave") dataset=(object tooltip=(localize "DAGGERHEART.UI.Tooltip.appliedEvenIfSuccessful") tooltipDirection="UP")}}{{/if}}
-                <div class="controls">
-                    <a data-tooltip="{{localize "CONTROLS.CommonDelete"}}" data-action="removeEffect" data-index="{{index}}"><i class="fas fa-trash"></i></a>
-                </div>
-            </div>
-        {{/each}} --}}
         {{#each source as | effect index |}}
             <div class="inventory-item single-img" data-effect-id="{{effect._id}}" data-action="editEffect">
                 <div class="inventory-item-header">