From 1b8ad316b9d1ada56e7acbc625f9cfbff8217424 Mon Sep 17 00:00:00 2001
From: sladecraven <sylvain@lahiette.com>
Date: Sun, 25 Sep 2022 21:21:28 +0200
Subject: [PATCH] Merge conteneur + fix hotbar

---
 module/rdd-hotbar-drop.js | 64 +++++++++++++++++++++------------------
 1 file changed, 34 insertions(+), 30 deletions(-)

diff --git a/module/rdd-hotbar-drop.js b/module/rdd-hotbar-drop.js
index 3900012c..4bef344f 100644
--- a/module/rdd-hotbar-drop.js
+++ b/module/rdd-hotbar-drop.js
@@ -2,6 +2,20 @@ import { Misc } from "./misc.js";
 
 export class RdDHotbar {
 
+  static async addToHotbar(item, slot) {
+    let command = `game.system.rdd.RdDHotbar.rollMacro("${item.name}", "${item.type}");`;
+    let macro = game.macros.contents.find(m => (m.name === item.name) && (m.command === command));
+    if (!macro) {
+      macro = await Macro.create({
+        name: item.name,
+        type: "script",
+        img: item.img,
+        command: command
+      }, { displaySheet: false })
+    }
+    await game.user.assignHotbarMacro(macro, slot);
+  }
+
   /**
    * Create a macro when dropping an entity on the hotbar
    * Item      - open roll dialog for item
@@ -10,7 +24,7 @@ export class RdDHotbar {
    */
   static initDropbar() {
 
-    Hooks.on("hotbarDrop", async (bar, documentData, slot) => {
+    Hooks.on("hotbarDrop", (bar, documentData, slot) => {
 
       // Create item macro if rollable item - weapon, spell, prayer, trait, or skill
       if (documentData.type == "Item") {
@@ -22,43 +36,33 @@ export class RdDHotbar {
         if (!item || (item.type != "arme" && item.type != "competence")) {
           return true
         }
-        let command = `game.system.rdd.RdDHotbar.rollMacro("${item.name}", "${item.type}");`;
-        let macro = game.macros.contents.find(m => (m.name === item.name) && (m.command === command));
-        if (!macro) {
-          macro =  await Macro.create({
-            name: item.name,
-            type: "script",
-            img: item.img,
-            command: command
-          }, { displaySheet: false })
-        }
-        game.user.assignHotbarMacro(macro, slot);
+        this.addToHotbar(item, slot)
         return false
       }
 
       return true
-  });
-}
+    })
+  }
 
   /** Roll macro */
   static rollMacro(itemName, itemType, bypassData) {
-  const speaker = ChatMessage.getSpeaker();
-  let actor;
-  if (speaker.token) actor = game.actors.tokens[speaker.token];
-  if (!actor) actor = game.actors.get(speaker.actor);
+    const speaker = ChatMessage.getSpeaker();
+    let actor;
+    if (speaker.token) actor = game.actors.tokens[speaker.token];
+    if (!actor) actor = game.actors.get(speaker.actor);
 
-  let item = actor?.items.find(it => it.name === itemName && it.type == itemType) ?? undefined;
-  if (!item) {
-    return ui.notifications.warn(`Impossible de trouver l'objet de cette macro`);
+    let item = actor?.items.find(it => it.name === itemName && it.type == itemType) ?? undefined;
+    if (!item) {
+      return ui.notifications.warn(`Impossible de trouver l'objet de cette macro`);
+    }
+
+    // Trigger the item roll
+    switch (item.type) {
+      case "arme":
+        return actor.rollArme(item);
+      case "competence":
+        return actor.rollCompetence(itemName);
+    }
   }
 
-  // Trigger the item roll
-  switch (item.type) {
-    case "arme":
-      return actor.rollArme(item);
-    case "competence":
-      return actor.rollCompetence(itemName);
-  }
-}
-
 }