From aff1b6eebed60dadad49615d8002b34cf4b588a0 Mon Sep 17 00:00:00 2001
From: Carlos Fernandez <cfern1990@gmail.com>
Date: Wed, 2 Jun 2021 17:07:03 -0400
Subject: [PATCH] fix: casted level display on roll repeats

---
 betterrolls5e/scripts/custom-roll.js | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/betterrolls5e/scripts/custom-roll.js b/betterrolls5e/scripts/custom-roll.js
index 96393430..0f4e1f30 100644
--- a/betterrolls5e/scripts/custom-roll.js
+++ b/betterrolls5e/scripts/custom-roll.js
@@ -635,6 +635,12 @@ export class CustomItemRoll {
 			placeTemplate = config.placeTemplate;
 		}
 
+		// Update item casted level property to match the slot level
+		// This ensure things like the property list showing the correct spell level
+		if (item && this.params.slotLevel) {
+			item.data.update({ 'data.castedLevel': this.params.slotLevel });
+		}
+
 		// Show Advantage/Normal/Disadvantage dialog if enabled
 		const hasAttack = this.fields.some(
 			(f) => ["attack", "check", "custom", "tool", "toolcheck"].includes(f[0]) && !f[1]?.rollState);
@@ -1145,9 +1151,7 @@ export class CustomItemRoll {
 			spellLevel = getProperty(actor, `data.data.spells.pact.level`) || spellLevel;
 		}
 
-		// Update params and item data temporarily
-		// We use data.update() so that changes are maintained even through clone() uses
-		item.data.update({ 'data.castedLevel': spellLevel });
+		// Update params temporarily
 		this.params.slotLevel = spellLevel;
 		this.params.consumeSpellLevel = consume;
 		return { lvl: spellLevel, consume, placeTemplate };