Version bump

- fixed string and boolean evaluators within mappings not behaving correctly #217
- added right click listener for PF1e item sheets #216

Author: Aedif

CHANGELOG.md

@@ -1,3 +1,8 @@
+# 6.0.6
+
+- Fixed string and boolean evaluators within mappings not behaving correctly (#217)
+- Added Art Select Right-click listener for PF1e item sheet images (#216)
+
 # 6.0.5
 
 - Fixed a bug causing unresponsive Token HUD button

token-variants/module.json

@@ -2,12 +2,12 @@
   "id": "token-variants",
   "title": "Token Variant Art",
   "description": "Searches a customizable list of directories and displays art variants for tokens/actors through pop-ups and a new Token HUD button. Variants can be individually shared with players allowing them to switch out their token art on the fly.",
-  "version": "6.0.5",
+  "version": "6.0.6",
   "compatibility": {
     "minimum": 13,
     "verified": 13
   },
-  "download": "https://github.com/Aedif/TokenVariants/releases/download/6.0.5/token-variants.zip",
+  "download": "https://github.com/Aedif/TokenVariants/releases/download/6.0.6/token-variants.zip",
   "url": "https://github.com/Aedif/TokenVariants",
   "manifest": "https://raw.githubusercontent.com/Aedif/TokenVariants/master/token-variants/module.json",
   "authors": [

token-variants/scripts/hooks/artSelectButtonHooks.js

@@ -10,8 +10,9 @@ export function registerArtSelectButtonHooks() {
   // Insert right-click listeners to open up ArtSelect forms from various contexts
   if (TVA_CONFIG.permissions.portrait_right_click[game.user.role]) {
     registerHook(feature_id, 'renderActorSheet', _modActorSheet);
-    registerHook(feature_id, 'renderItemSheet5e', _modItemSheet);
+    if (game.system.id === 'dnd5e') registerHook(feature_id, 'renderItemSheet5e', _modItemSheet);
     registerHook(feature_id, 'renderItemActionSheet', _modItemSheet);
+    if (game.system.id === 'pf1') registerHook(feature_id, 'renderItemSheetPF', _modItemSheetPF);
     registerHook(feature_id, 'renderJournalSheet', _modJournalSheet);
   } else {
     ['renderActorSheet', 'renderItemSheet5e', 'renderItemActionSheet', 'renderJournalSheet'].forEach((name) =>
@@ -118,6 +119,17 @@ function _modItemSheet(itemSheet, html, options) {
   });
 }
 
+function _modItemSheetPF(itemSheet, html, options) {
+  html[0]?.querySelector('.profile')?.addEventListener('contextmenu', () => {
+    const item = itemSheet.document;
+    if (!item) return;
+    showArtSelect(item.name, {
+      searchType: SEARCH_TYPE.ITEM,
+      callback: (imgSrc) => item.update({ img: imgSrc }),
+    });
+  });
+}
+
 function _modMacroConfig(macroConfig, html, options) {
   const img = html.querySelector('.sheet-header > img');
   img?.addEventListener('contextmenu', () => {

token-variants/scripts/hooks/effectMappingHooks.js

@@ -863,9 +863,9 @@ function getPropertyValue(token, property) {
 
   if (property.startsWith('"') && property.endsWith('"')) {
     val = property.substring(1, property.length - 1);
-    if (val === 'true') val = true;
-    else if (val === 'false') val = true;
-    return { val, boolean: true };
+    if (val === 'true') return { val: true, boolean: true };
+    else if (val === 'false') return { val: false, boolean: true };
+    return { val };
   } else if (property.endsWith('%')) {
     val = Number(property.substring(0, property.length - 1));
     return { val, percentage: true };
@@ -903,8 +903,8 @@ export function evaluateComparator(token, expression) {
 
     if (val1.percentage) val2.val = (val2.val / val2.maxVal) * 100;
     else if (val2.percentage) val1.val = (val1.val / val1.maxVal) * 100;
-    else if (val1.boolean) val2.val = foundry.utils.isEmpty(val2.val) ? false : Boolean(val2.val);
-    else if (val2.boolean) val1.val = foundry.utils.isEmpty(val1.val) ? false : Boolean(val1.val);
+    else if (val1.boolean && !val2.boolean) val2.val = foundry.utils.isEmpty(val2.val) ? false : Boolean(val2.val);
+    else if (val2.boolean && !val1.boolean) val1.val = foundry.utils.isEmpty(val1.val) ? false : Boolean(val1.val);
 
     let passed = false;
     if (sign === '=') {