Merge branch 'feature/roll-item-refactoring' into feature/wfrp4

src/systems/demonlord.js

@@ -130,4 +130,8 @@ export class demonlord{
     getSpellUses(token,level,item) {
         return;
     }
+
+    rollItem(item) {
+        return item.roll()
+    }
 }

src/systems/dnd35e.js

@@ -195,4 +195,8 @@ export class dnd35e{
             maximum: item.maxCharges
         }
     }
+
+    rollItem(item) {
+        return item.roll()
+    }
 }

src/systems/dnd5e.js

@@ -198,4 +198,8 @@ export class dnd5e{
             maximum: token.actor.data.data.spells?.[`spell${level}`].max
         }
     }
+
+    rollItem(item) {
+        return item.roll()
+    }
 }

src/systems/pf2e.js

@@ -192,4 +192,8 @@ export class pf2e{
             maximum: spellbook.data.data.slots?.[`slot${level}`].max
         }
     }
+
+    rollItem(item) {
+        return item.roll()
+    }
 }

src/systems/tokenHelper.js

@@ -246,4 +246,8 @@ export class TokenHelper{
     getSpellUses(token,level,item) {
         return this.system.getSpellUses(token,level,item);
     }
+
+    rollItem(item) {
+        return this.system.rollItem(item);
+    }
 }

src/token.js

@@ -742,13 +742,7 @@ export class TokenControl{
 
             const item = items[itemNr];
             if (item != undefined) {
-                if (game.system.id == 'wfrp4e') {
+                tokenHelper.rollItem(item);
-                    console.log("rolling wfrp4e item macro")
-                    game.wfrp4e.utility.rollItemMacro(item.name, item.type, false);
-                }
-                else {
-                    item.roll();
-                }
             }
             
         }