From d5acbea1393bbaeeed337630c115d9bd11e20db4 Mon Sep 17 00:00:00 2001
From: DameryDad <74715860+DameryDad@users.noreply.github.com>
Date: Mon, 24 Jul 2023 09:47:14 +0100
Subject: [PATCH] Fix char sheet version check for 5e characters

* If a 5e (or other D&D version) character was loaded or transmogrified, the system would crash on checking the sheet version.  Fixed.
---
 AttackMaster/2.1.0/attackMaster.js | 6 +++---
 AttackMaster/attackMaster.js       | 6 +++---
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/AttackMaster/2.1.0/attackMaster.js b/AttackMaster/2.1.0/attackMaster.js
index 1b71185892..682fc39dcf 100644
--- a/AttackMaster/2.1.0/attackMaster.js
+++ b/AttackMaster/2.1.0/attackMaster.js
@@ -1884,7 +1884,7 @@ var attackMaster = (function() {
 			return new Promise(resolve => {
 
 				try {
-					var errFlag = doCheckAC( [tokenID], findTheGM(), [], true );
+					var errFlag = doCheckAC( [tokenID,'quiet'], findTheGM(), [], true );
 				} catch (e) {
 					log('AttackMaster checkACvars: JavaScript '+e.name+': '+e.message+' while checking AC for tokenID '+tokenID);
 					sendDebug('AttackMaster checkACvars: JavaScript '+e.name+': '+e.message+' while checking AC for tokenID '+tokenID);
@@ -6273,7 +6273,7 @@ var attackMaster = (function() {
 		
 		// set rogue activity percentages
 		
-		let csVersion = attrLookup(charCS,fields.csVersion).match(/(\d+)\.?(\d*)/);
+		let csVersion = String(attrLookup(charCS,fields.csVersion) || 4.17).match(/(\d+)\.?(\d*)/);
 		let modTag = (csVersion[1] >= 4 && (!csVersion[2] || csVersion[2] >= 17)) ? fields.Armor_mod_417 : fields.Armor_mod_416;
 		
 		setAttr( charCS, [fields.Pick_Pockets[0]+modTag,fields.Pick_Pockets[1]], acValues.armour.data.ppa );
@@ -6287,7 +6287,7 @@ var attackMaster = (function() {
 		setAttr( charCS, [fields.Legend_Lore[0]+modTag,fields.Legend_Lore[1]], acValues.armour.data.iba );
 		setAttr( charCS, fields.Armor_name, acValues.armour.data.racname);
 
-		if (!silent || ((ac != prevAC) && !magicItem)) {
+		if ((silentCmd !== 'quiet') && (!silent || ((ac != prevAC) && !magicItem))) {
 			makeACDisplay( args, senderId, ac, dmgAdj, acValues, armourMsgs );
 		} else {
 			sendWait(senderId,0);
diff --git a/AttackMaster/attackMaster.js b/AttackMaster/attackMaster.js
index 1b71185892..682fc39dcf 100644
--- a/AttackMaster/attackMaster.js
+++ b/AttackMaster/attackMaster.js
@@ -1884,7 +1884,7 @@ var attackMaster = (function() {
 			return new Promise(resolve => {
 
 				try {
-					var errFlag = doCheckAC( [tokenID], findTheGM(), [], true );
+					var errFlag = doCheckAC( [tokenID,'quiet'], findTheGM(), [], true );
 				} catch (e) {
 					log('AttackMaster checkACvars: JavaScript '+e.name+': '+e.message+' while checking AC for tokenID '+tokenID);
 					sendDebug('AttackMaster checkACvars: JavaScript '+e.name+': '+e.message+' while checking AC for tokenID '+tokenID);
@@ -6273,7 +6273,7 @@ var attackMaster = (function() {
 		
 		// set rogue activity percentages
 		
-		let csVersion = attrLookup(charCS,fields.csVersion).match(/(\d+)\.?(\d*)/);
+		let csVersion = String(attrLookup(charCS,fields.csVersion) || 4.17).match(/(\d+)\.?(\d*)/);
 		let modTag = (csVersion[1] >= 4 && (!csVersion[2] || csVersion[2] >= 17)) ? fields.Armor_mod_417 : fields.Armor_mod_416;
 		
 		setAttr( charCS, [fields.Pick_Pockets[0]+modTag,fields.Pick_Pockets[1]], acValues.armour.data.ppa );
@@ -6287,7 +6287,7 @@ var attackMaster = (function() {
 		setAttr( charCS, [fields.Legend_Lore[0]+modTag,fields.Legend_Lore[1]], acValues.armour.data.iba );
 		setAttr( charCS, fields.Armor_name, acValues.armour.data.racname);
 
-		if (!silent || ((ac != prevAC) && !magicItem)) {
+		if ((silentCmd !== 'quiet') && (!silent || ((ac != prevAC) && !magicItem))) {
 			makeACDisplay( args, senderId, ac, dmgAdj, acValues, armourMsgs );
 		} else {
 			sendWait(senderId,0);