From a852cc094e3e1023d601658ebc39c86c1c45a67e Mon Sep 17 00:00:00 2001
From: Harm Manders <harmmanders@gmail.com>
Date: Sun, 2 Jul 2023 22:37:32 +0200
Subject: [PATCH 1/8] convert versatile to options

---
 src/views/UserContent/Npcs/EditNpc.vue | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/src/views/UserContent/Npcs/EditNpc.vue b/src/views/UserContent/Npcs/EditNpc.vue
index 631ba120..421e7c13 100644
--- a/src/views/UserContent/Npcs/EditNpc.vue
+++ b/src/views/UserContent/Npcs/EditNpc.vue
@@ -168,6 +168,7 @@ export default {
 		copy({ result }) {
 			this.copy_dialog = false;
 			this.npc = { ...result };
+			this.npc = this.convertVersatileToOptions(this.npc);
 		},
 		reset() {
 			this.npc = {};
@@ -177,6 +178,19 @@ export default {
 			this.npc_copy = JSON.parse(JSON.stringify(this.npc));
 			this.unsaved_changes = false;
 		},
+		convertVersatileToOptions(npc) {
+			if (npc.actions.length > 0) {
+				for (const action of npc.actions) {
+					if (action.versatile === true) {
+						action.options = [action.versatile_one, action.versatile_two];
+						delete action.versatile;
+						delete action.versatile_one;
+						delete action.versatile_two;
+					}
+				}
+			}
+			return npc;
+		},
 		/**
 		 * Checks if a new NPC must be added, or an existing NPC must be saved.
 		 **/

From e3aaf6f997d0ee1f04d779b2254599b9968cef7d Mon Sep 17 00:00:00 2001
From: Harm Manders <harmmanders@gmail.com>
Date: Sun, 2 Jul 2023 22:37:48 +0200
Subject: [PATCH 2/8] fixed < issue in tempaltes

---
 src/components/combat/actions/Roll.vue        | 24 +++++--------------
 .../hk-action-rolls/hk-action-rolls-table.vue | 12 ++++------
 2 files changed, 11 insertions(+), 25 deletions(-)

diff --git a/src/components/combat/actions/Roll.vue b/src/components/combat/actions/Roll.vue
index 84aec002..048904cd 100644
--- a/src/components/combat/actions/Roll.vue
+++ b/src/components/combat/actions/Roll.vue
@@ -1,3 +1,4 @@
+<!-- eslint-disable vue/no-parsing-error -->
 <template>
 	<div v-if="current">
 		<h3 v-if="targeted.length === 0" class="red text-center">Select one or more targets</h3>
@@ -136,26 +137,13 @@
 													/>
 													{{ roll.dice_count || "" }}{{ roll.dice_type ? `d${roll.dice_type}` : ``
 													}}<template v-if="roll.fixed_val && roll.dice_count">
-														{{ (roll.fixed_val &lt; 0) ? `- ${Math.abs(roll.fixed_val)}` : `+ ${roll.fixed_val}`
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+														{{
+															roll.fixed_val < 0
+																? `- ${Math.abs(roll.fixed_val)}`
+																: `+ ${roll.fixed_val}`
 														}}) </template
 													><template v-else>{{ roll.fixed_val }})</template>
-													{{ roll_index+1 &lt; action.action_list[0].rolls.length ? "+" : "" }}
+													{{ roll_index + 1 < action.action_list[0].rolls.length ? "+" : "" }}
 													<q-tooltip anchor="top middle" self="center middle">
 														{{
 															action.action_list[0].type === "healing"
diff --git a/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue b/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
index b54e2a10..cf8acf7d 100644
--- a/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
+++ b/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
@@ -5,13 +5,11 @@
 				{{ calcAverage(data.row.dice_type, data.row.dice_count, data.row.fixed_val) }}
 				({{ data.row.dice_count || "" }}{{ data.row.dice_type ? `d${data.row.dice_type}` : `` }}
 				<template v-if="data.row.fixed_val && data.row.dice_count">
-					{{ 
-						(data.row.fixed_val &lt; 0) ? `- ${Math.abs(data.row.fixed_val)}` : `+ ${data.row.fixed_val}`
-
-
-
-
-
+					{{
+						// eslint-disable-next-line vue/no-parsing-error
+						data.row.fixed_val < 0
+							? `- ${Math.abs(data.row.fixed_val)}`
+							: `+ ${data.row.fixed_val}`
 					}})
 				</template>
 				<template v-else>{{ data.row.fixed_val }})</template>

From 12a85d2e61eb1273b8fff02d7b21ba751061828c Mon Sep 17 00:00:00 2001
From: Harm Manders <harmmanders@gmail.com>
Date: Sun, 2 Jul 2023 22:58:10 +0200
Subject: [PATCH 3/8] continue for loop if roll should be ignored by option

---
 src/mixins/dice.js | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mixins/dice.js b/src/mixins/dice.js
index ff24f865..778e83be 100644
--- a/src/mixins/dice.js
+++ b/src/mixins/dice.js
@@ -277,6 +277,7 @@ export const dice = {
 						const option = roll.options[config.option];
 						if (option.ignore) {
 							editableRoll = {};
+							continue;
 						} else {
 							editableRoll.damage_type = option.damage_type || editableRoll.damage_type;
 							editableRoll.dice_type = option.dice_type || editableRoll.dice_type;

From 28913aa8a5c05c05b6ffda1fa3e24d018591c96f Mon Sep 17 00:00:00 2001
From: Harm Manders <harmmanders@gmail.com>
Date: Sun, 2 Jul 2023 23:14:25 +0200
Subject: [PATCH 4/8] Also parse versatile rolls not only labels

---
 src/views/UserContent/Npcs/EditNpc.vue | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/src/views/UserContent/Npcs/EditNpc.vue b/src/views/UserContent/Npcs/EditNpc.vue
index 421e7c13..8e2150d6 100644
--- a/src/views/UserContent/Npcs/EditNpc.vue
+++ b/src/views/UserContent/Npcs/EditNpc.vue
@@ -182,10 +182,28 @@ export default {
 			if (npc.actions.length > 0) {
 				for (const action of npc.actions) {
 					if (action.versatile === true) {
+						const versTwoName = action.versatile_two;
 						action.options = [action.versatile_one, action.versatile_two];
 						delete action.versatile;
 						delete action.versatile_one;
 						delete action.versatile_two;
+
+						for (const sub_action of action.action_list) {
+							for (const roll of sub_action.rolls) {
+								roll.options = {
+									[versTwoName]: {
+										damage_type: roll.versatile_damage_type,
+										dice_count: roll.versatile_dice_count,
+										dice_type: roll.versatile_dice_type,
+										fixed_val: roll.versatile_fixed_val,
+									},
+								};
+								delete roll.versatile_damage_type;
+								delete roll.versatile_dice_count;
+								delete roll.versatile_dice_type;
+								delete roll.versatile_fixed_val;
+							}
+						}
 					}
 				}
 			}

From 72aa3bd7c8a443abb4ee3e461ba514dcfb635ca3 Mon Sep 17 00:00:00 2001
From: Harm Manders <harmmanders@gmail.com>
Date: Sun, 2 Jul 2023 23:32:42 +0200
Subject: [PATCH 5/8] visualize options in roll table

---
 .../hk-action-rolls/hk-action-rolls-table.vue | 46 +++++++++----------
 1 file changed, 23 insertions(+), 23 deletions(-)

diff --git a/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue b/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
index cf8acf7d..523c1d35 100644
--- a/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
+++ b/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
@@ -13,11 +13,8 @@
 					}})
 				</template>
 				<template v-else>{{ data.row.fixed_val }})</template>
-				<q-tooltip v-if="versatile" anchor="top middle" self="bottom middle">
-					{{ versatileOptions[0] || "Enter versatile option" }}
-				</q-tooltip>
 			</span>
-			<span v-if="versatile && versatileRoll(data.row)">
+			<span v-if="data.row.options && versatileRoll(data.row)">
 				|
 				{{
 					calcAverage(
@@ -27,9 +24,10 @@
 					)
 				}}
 				({{ versatileRoll(data.row) }})
-				<q-tooltip anchor="top middle" self="bottom middle">
-					{{ versatileOptions[1] || "Enter versatile option" }}
-				</q-tooltip>
+			</span>
+			<span v-if="Object.values(data.row.options).length > 1">
+				<span>| ...</span>
+				<q-tooltip anchor="top middle" self="bottom middle"> More then two Options </q-tooltip>
 			</span>
 		</div>
 
@@ -177,27 +175,29 @@ export default {
 				: parseInt(modifier);
 		},
 		versatileRoll(roll) {
-			if (!roll.versatile_dice_count && !roll.versatile_dice_type && !roll.versatile_fixed_val) {
+			console.log(roll);
+			if (!roll.options) {
 				return undefined;
-			} else {
-				let returnRoll = {};
+			}
 
-				returnRoll.dice_count = roll.versatile_dice_count
-					? roll.versatile_dice_count
-					: roll.dice_count;
-				returnRoll.dice_type = roll.versatile_dice_type ? roll.versatile_dice_type : roll.dice_type;
-				returnRoll.fixed_val = roll.versatile_fixed_val ? roll.versatile_fixed_val : roll.fixed_val;
+			const firstOption = Object.values(roll.options)[0];
 
-				let fixed;
-				if (returnRoll.fixed_val !== undefined) {
-					fixed =
-						returnRoll.fixed_val < 0
-							? ` - ${Math.abs(returnRoll.fixed_val)}`
-							: ` + ${returnRoll.fixed_val}`;
-				}
+			let returnRoll = {};
 
-				return `${returnRoll.dice_count}d${returnRoll.dice_type}${fixed}`;
+			returnRoll.dice_count = firstOption.dice_count ? firstOption.dice_count : roll.dice_count;
+			returnRoll.dice_type = firstOption.dice_type ? firstOption.dice_type : roll.dice_type;
+			returnRoll.fixed_val =
+				firstOption.fixed_val !== undefined ? firstOption.fixed_val : roll.fixed_val;
+
+			let fixed;
+			if (returnRoll.fixed_val !== undefined) {
+				fixed =
+					returnRoll.fixed_val < 0
+						? ` - ${Math.abs(returnRoll.fixed_val)}`
+						: ` + ${returnRoll.fixed_val}`;
 			}
+
+			return `${returnRoll.dice_count}d${returnRoll.dice_type}${fixed}`;
 		},
 		scalingDesc(tiers, scaling, level) {
 			return spellScalingDescription(tiers, scaling, level);

From dc4d5af540d39493f71922fac69163714ada1d29 Mon Sep 17 00:00:00 2001
From: Unknown <keyroos@gmail.com>
Date: Fri, 7 Jul 2023 11:22:40 +0200
Subject: [PATCH 6/8] For update on options input

---
 src/components/npcs/Actions.vue | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/components/npcs/Actions.vue b/src/components/npcs/Actions.vue
index 3a4f0288..6390e769 100644
--- a/src/components/npcs/Actions.vue
+++ b/src/components/npcs/Actions.vue
@@ -349,6 +349,7 @@
 												class="mb-4"
 												@new-value="addOption"
 												@remove="removeOption($event, category, ability_index)"
+												@input="$forceUpdate()"
 											>
 												<hk-popover slot="append" header="Action options">
 													<i class="fas fa-info-circle" aria-hidden="true" />

From 0d2c790925d8d890b08cd0b649b55fab2e79bc63 Mon Sep 17 00:00:00 2001
From: Unknown <keyroos@gmail.com>
Date: Fri, 7 Jul 2023 11:23:11 +0200
Subject: [PATCH 7/8] Fix roll average calculation

---
 .../hk-action-rolls/hk-action-rolls-table.vue | 24 ++++++++-----------
 1 file changed, 10 insertions(+), 14 deletions(-)

diff --git a/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue b/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
index 523c1d35..d578fb2c 100644
--- a/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
+++ b/src/components/hk-components/hk-action-rolls/hk-action-rolls-table.vue
@@ -15,19 +15,11 @@
 				<template v-else>{{ data.row.fixed_val }})</template>
 			</span>
 			<span v-if="data.row.options && versatileRoll(data.row)">
-				|
-				{{
-					calcAverage(
-						data.row.versatile_dice_type || data.row.dice_type,
-						data.row.versatile_dice_count || data.row.dice_count,
-						data.row.versatile_fixed_val || data.row.fixed_val
-					)
-				}}
-				({{ versatileRoll(data.row) }})
+				| {{ versatileRoll(data.row) }}
 			</span>
 			<span v-if="Object.values(data.row.options).length > 1">
 				<span>| ...</span>
-				<q-tooltip anchor="top middle" self="bottom middle"> More then two Options </q-tooltip>
+				<q-tooltip anchor="top middle" self="bottom middle">More than two Options</q-tooltip>
 			</span>
 		</div>
 
@@ -175,14 +167,12 @@ export default {
 				: parseInt(modifier);
 		},
 		versatileRoll(roll) {
-			console.log(roll);
 			if (!roll.options) {
 				return undefined;
 			}
 
 			const firstOption = Object.values(roll.options)[0];
-
-			let returnRoll = {};
+			const returnRoll = {};
 
 			returnRoll.dice_count = firstOption.dice_count ? firstOption.dice_count : roll.dice_count;
 			returnRoll.dice_type = firstOption.dice_type ? firstOption.dice_type : roll.dice_type;
@@ -197,7 +187,13 @@ export default {
 						: ` + ${returnRoll.fixed_val}`;
 			}
 
-			return `${returnRoll.dice_count}d${returnRoll.dice_type}${fixed}`;
+			const average = this.calcAverage(
+				returnRoll.dice_type,
+				returnRoll.dice_count,
+				returnRoll.fixed_val
+			);
+
+			return `${average} (${returnRoll.dice_count}d${returnRoll.dice_type}${fixed})`;
 		},
 		scalingDesc(tiers, scaling, level) {
 			return spellScalingDescription(tiers, scaling, level);

From f65b4542e05a5ae3630b180b87da0ddd233ee6c6 Mon Sep 17 00:00:00 2001
From: Unknown <keyroos@gmail.com>
Date: Fri, 14 Jul 2023 17:30:47 +0200
Subject: [PATCH 8/8] Added to hit bonus on monster attacks

---
 package-lock.json                      |  4 ++--
 package.json                           |  2 +-
 src/components/combat/actions/Roll.vue | 10 ++++++++++
 3 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/package-lock.json b/package-lock.json
index adbfe039..d73d396e 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,12 +1,12 @@
 {
 	"name": "harmless_key",
-	"version": "2.17.0",
+	"version": "2.18.0",
 	"lockfileVersion": 2,
 	"requires": true,
 	"packages": {
 		"": {
 			"name": "harmless_key",
-			"version": "2.17.0",
+			"version": "2.18.0",
 			"dependencies": {
 				"@octokit/rest": "^19.0.5",
 				"@quasar/extras": "^1.0.0",
diff --git a/package.json b/package.json
index 3382e53f..5b8da149 100644
--- a/package.json
+++ b/package.json
@@ -1,5 +1,5 @@
 {
-	"version": "2.17.0",
+	"version": "2.18.0",
 	"name": "harmless_key",
 	"description": "A Dungeons and Dragons Combat Tracker",
 	"productName": "Harmless Key",
diff --git a/src/components/combat/actions/Roll.vue b/src/components/combat/actions/Roll.vue
index 048904cd..9af3c831 100644
--- a/src/components/combat/actions/Roll.vue
+++ b/src/components/combat/actions/Roll.vue
@@ -121,6 +121,14 @@
 											"
 										>
 											<!-- Rolls -->
+											<span v-if="!isNil(action.action_list[0].attack_bonus)">
+												{{
+													action.action_list[0].attack_bonus < 0
+														? `-${Math.abs(action.action_list[0].attack_bonus)}`
+														: `+${action.action_list[0].attack_bonus}`
+												}}
+												to hit
+											</span>
 											<span v-if="action.action_list[0].rolls">
 												<span
 													v-for="(roll, roll_index) in action.action_list[0].rolls"
@@ -298,6 +306,7 @@ import { setHP } from "src/mixins/HpManipulations.js";
 import { damage_type_icons } from "src/utils/generalConstants";
 import { runEncounter } from "src/mixins/runEncounter.js";
 import Projectiles from "./Projectiles";
+import { isNil } from "lodash";
 
 export default {
 	name: "Roll",
@@ -321,6 +330,7 @@ export default {
 			],
 			rollObject: {},
 			projectile_dialog: false,
+			isNil: isNil,
 		};
 	},
 	computed: {