Merge pull request 'Version 12.0.6 - Le bazar d'Astrobazzarh' (#708) from VincentVk/foundryvtt-reve-de-dragon:v11 into v11

Reviewed-on: #708

changelog.md

@@ -1,5 +1,17 @@
 # 12.0
-## 12.0.6 - Les souvenirs d'Astrobazzarh
+
+## 12.0.6 - Le bazar d'Astrobazzarh
+- Corrections de l'inventaire en bazar:
+  - un problème pouvait survenir en déplaçant les objets 
+    l'inventaire, qui fait qu'un conteneur se retrouve récursivement dans son
+    propre contenu, ce qui empêche d'ouvrir la feuille d'acteur.
+  - un objet non-conteneur pouvait dans certains cas avoir un pseudo contenu
+  - un objet pouvait être considéré comme contenu, sans être présent dans un
+    conteneur (et donc non affiché)
+  - vider les conteneurs supprime correctement toutes les informations liées
+    aux conteneurs/contenus
+- Les messages pour les tirages dans le compendium utilisent le "roll mode" 
+  courant pour leur visibilité
 - Fix: restaurer la compatibilité Foundry 11
 
 ## 12.0.5 - Les mauvais jours d'Astrobazzarh

module/actor/base-actor.js

@@ -557,15 +557,15 @@ export class RdDBaseActor extends Actor {
   /* -------------------------------------------- */
   /** Ajoute un item dans un conteneur, sur la base de leurs ID */
   async ajouterDansConteneur(item, conteneur, onAjouterDansConteneur) {
-    if (!conteneur) {
+    if (conteneur?.isConteneur()) {
-      // TODO: afficher
-      item.estContenu = false;
-    }
-    else if (conteneur.isConteneur()) {
       item.estContenu = true;
       const nouveauContenu = [...conteneur.system.contenu, item.id];
       await conteneur.update({ 'system.contenu': nouveauContenu });
-      onAjouterDansConteneur(item.id, conteneur.id);
+      onAjouterDansConteneur(item.id, conteneur.id)
+    }
+    else {
+      item.estContenu = false;
+      await conteneur?.update({ 'system.-=contenu': undefined })
     }
   }
 
@@ -583,8 +583,13 @@ export class RdDBaseActor extends Actor {
           if (item.estContenu) {
             item.estContenu = undefined;
           }
-          if (item.type == 'conteneur' && item.system.contenu.length > 0) {
+          if (item.system.contenu != undefined) {
-            corrections.push({ _id: item.id, 'system.contenu': [] });
+            if (item.type == 'conteneur') {
+              corrections.push({ _id: item.id, 'system.contenu': [] });
+            }
+            else {
+              corrections.push({ _id: item.id, 'system.-=contenu': undefined });
+            }
           }
         }
         if (corrections.length > 0) {
@@ -619,15 +624,21 @@ export class RdDBaseActor extends Actor {
   }
 
   /* -------------------------------------------- */
-  /** Supprime un item d'un conteneur, sur la base
+  /**
-   * de leurs ID */
+   * Supprime un item d'un conteneur, sur la base de leurs ID
+   */
   async enleverDeConteneur(item, conteneur, onEnleverDeConteneur) {
-    if (conteneur?.isConteneur()) {
+    if (conteneur) {
-      item.estContenu = false;
+      if (conteneur.isConteneur()) {
-      const contenu = conteneur.system.contenu.filter(id => id != item.id);
+        const contenu = conteneur.system.contenu.filter(id => id != item.id);
-      await conteneur.update({ 'system.contenu': contenu });
+        await conteneur.update({ 'system.contenu': contenu });
-      onEnleverDeConteneur();
+        onEnleverDeConteneur();
+      }
+      else {
+        await conteneur.update({ 'system.-=contenu': undefined })
+      }
     }
+    item.estContenu = false;
   }
 
   /* -------------------------------------------- */

module/rdd-utility.js

@@ -358,13 +358,15 @@ export class RdDUtility {
     let objetVersConteneur = {};
     // Attribution des objets aux conteneurs
     for (let conteneur of conteneurs) {
-      conteneur.subItems = [];
+      if (conteneur.isConteneur()) {
-      for (let id of conteneur.system.contenu ?? []) {
+        conteneur.subItems = [];
-        let objet = inventaires.find(objet => (id == objet._id));
+        for (let id of conteneur.system.contenu ?? []) {
-        if (objet) {
+          let objet = inventaires.find(objet => (id == objet._id));
-          objet.estContenu = true; // Permet de filtrer ce qui est porté dans le template
+          if (objet) {
-          objetVersConteneur[id] = conteneur._id;
+            objet.estContenu = true;
-          conteneur.subItems.push(objet);
+            objetVersConteneur[id] = conteneur._id;
+            conteneur.subItems.push(objet);
+          }
         }
       }
     }

module/settings/system-compendiums.js

@@ -1,3 +1,4 @@
+import { ChatUtility } from "../chat-utility.js";
 import { HIDE_DICE, SYSTEM_RDD } from "../constants.js";
 import { RdDItem } from "../item.js";
 import { Misc } from "../misc.js";
@@ -290,7 +291,7 @@ export class CompendiumTableHelpers {
       sound: CONFIG.sounds.dice,
       content: flavorContent
     };
-    ChatMessage.create(messageData, { rollMode: "gmroll" });
+    await ChatUtility.createChatWithRollMode(game.user.id, messageData)
   }
 
   /* -------------------------------------------- */
@@ -306,7 +307,7 @@ export class CompendiumTableHelpers {
       whisper: game.user.id,
       content: flavorContent
     };
-    ChatMessage.create(messageData, { rollMode: "gmroll" });
+    await ChatUtility.createChatWithRollMode(game.user.id, messageData)
   }
 
 }

system.json

@@ -1,8 +1,8 @@
 {
   "id": "foundryvtt-reve-de-dragon",
   "title": "Rêve de Dragon",
-  "version": "12.0.5",
+  "version": "12.0.6",
-  "download": "https://www.uberwald.me/gitea/public/foundryvtt-reve-de-dragon/archive/foundryvtt-reve-de-dragon-12.0.5.zip",
+  "download": "https://www.uberwald.me/gitea/public/foundryvtt-reve-de-dragon/archive/foundryvtt-reve-de-dragon-12.0.6.zip",
   "manifest": "https://www.uberwald.me/gitea/public/foundryvtt-reve-de-dragon/raw/v11/system.json",
   "changelog": "https://www.uberwald.me/gitea/public/foundryvtt-reve-de-dragon/raw/branch/v11/changelog.md",
   "compatibility": {