module/actor.js

@@ -826,21 +826,6 @@ export class RdDActor extends Actor {
   }
 
   /* -------------------------------------------- */
-  validateConteneur(itemId, conteneurId) {
-    const dest = this.getObjet(conteneurId);
-    const item = this.getObjet(itemId);
-    const destData = Misc.data(dest);
-    if (destData?.type != 'conteneur') {
-      ui.notifications.warn(`Impossible de déplacer un objet (${item.name}) vers un ${destData.type} qui n'est pas un conteneur (${dest.name}) !`);
-      return false;
-    }
-    if (this._isConteneurContenu(item, conteneurId)) {
-      ui.notifications.warn(`Impossible de déplacer un conteneur parent (${item.name}) dans un de ses contenus ${dest.name} !`);
-      return false; // Loop detected !
-    }
-    return true;
-  }
-
   _isConteneurContenu(item, conteneurId) {
     if (Misc.data(item)?.type == 'conteneur') { // Si c'est un conteneur, il faut vérifier qu'on ne le déplace pas vers un sous-conteneur lui appartenant
       for (let id of Misc.templateData(item).contenu) {
@@ -870,41 +855,6 @@ export class RdDActor extends Actor {
       + Number(tplData.encombrement) /* TODO? Number(tplData.quantite) -- on pourrait avoir plusieurs conteneurs...*/
   }
 
-  /* -------------------------------------------- */
-  isConteneurSuffisant(itemId, conteneurId) {
-    if (!conteneurId) {
-      // on peut toujours vider son sac
-      return true;
-    }
-    return this.validateConteneur(itemId, conteneurId) && this.testConteneurCapacite(itemId, conteneurId);
-  }
-
-  /* -------------------------------------------- */
-  /** Teste si le conteneur de destination a suffisament de capacité
-   * pour recevoir le nouvel objet
-   */
-  testConteneurCapacite(itemId, conteneurId) {
-    let conteneur = this.getObjet(conteneurId);  // recup conteneur (ou undefined)
-    //console.log("Conteneur trouvé : ", conteneur);
-    if (conteneur?.type != "conteneur") {
-      return true;
-    }
-
-    const tplConteneur = Misc.templateData(conteneur);
-    // Calculer le total actuel des contenus
-    let encContenu = this.getRecursiveEnc(conteneur) - Number(tplConteneur.encombrement);
-    let nouvelObjet = this.getObjet(itemId);
-    let newEnc = this.getRecursiveEnc(nouvelObjet); // Calculer le total actuel du nouvel objet
-    //console.log( currentEnc, newEnc, tplConteneur.capacite, conteneur.name);
-    if (nouvelObjet && ((encContenu + newEnc) > Number(tplConteneur.capacite))) {
-      ui.notifications.warn(
-        `Le conteneur ${conteneur.name} a une capacité de ${tplConteneur.capacite}, et contient déjà ${encContenu}.
-        Impossible d'y ranger: ${nouvelObjet.name} d'encombrement ${newEnc}!`);
-      return false;
-    }
-    return true;
-  }
-
   /* -------------------------------------------- */
   buildSubConteneurObjetList(conteneurId, deleteList) {
     let conteneur = this.getObjet(conteneurId);
@@ -932,9 +882,8 @@ export class RdDActor extends Actor {
   /* -------------------------------------------- */
   /** Supprime un item d'un conteneur, sur la base 
    * de leurs ID */
-  async enleverDeConteneur(itemId, conteneurId) {
-    let conteneur = this.getObjet(conteneurId);
-    if (conteneur?.type == 'conteneur') { // Si présent
+  async enleverDeConteneur(itemId, conteneur) {
+    if (conteneur?.isConteneur()) { // Si présent
       let data2use = duplicate(Misc.data(conteneur));
       //console.log("Suppression du conteneur1", conteneurId, itemId,  data2use.data.contenu);
       let contenu = data2use.data.contenu;
@@ -950,9 +899,8 @@ export class RdDActor extends Actor {
   /* -------------------------------------------- */
   /** Ajoute un item dans un conteneur, sur la base 
    * de leurs ID */
-  async ajouterAConteneur(itemId, conteneurId) {
-    let conteneur = this.getObjet(conteneurId);
-    if (conteneur?.type == 'conteneur') {
+  async ajouterDansConteneur(itemId, conteneur) {
+    if (conteneur?.isConteneur()) {
       let data2use = duplicate(Misc.data(conteneur));
       data2use.data.contenu.push(itemId);
       await this.updateEmbeddedDocuments('Item', [data2use]);
@@ -976,39 +924,71 @@ export class RdDActor extends Actor {
   /* -------------------------------------------- */
   async processItemDropEvent(event, objetVersConteneur) {
     let dragData = JSON.parse(event.dataTransfer.getData("text/plain"));
-    console.log(dragData, this.id);
-    let objetId = dragData.id || dragData.data._id;
-    console.log("DRAG", dragData);
+    console.log("DRAG", this.id, dragData);
+    let itemId = dragData.id || dragData.data._id;
     if (dragData.type == 'Item') {
-      if (dragData.actorId) {
-        if (dragData.actorId != this.id) {
+      if (dragData.actorId && dragData.actorId != this.id) {
         console.log("Moving objects", dragData);
-          this.moveItemsBetweenActors(objetId, dragData.actorId);
+        this.moveItemsBetweenActors(itemId, dragData.actorId);
         return false;
       }
-        else {
+      if (dragData.actorId == this.id) {
         // rangement
         const destId = $(event.target).parents(".item").attr("data-item-id");
-          const srcId = objetVersConteneur[objetId];
-          
-          if (srcId != destId && objetId != destId && this.isConteneurSuffisant(objetId, destId)) {
-            if (srcId) {
-              await this.enleverDeConteneur(objetId, srcId);
+        const srcId = objetVersConteneur[itemId];
+        if (srcId != destId && itemId != destId) { // déplacement de l'objet
+          const item = this.getObjet(itemId);
+          const dest = this.getObjet(destId);
+          const src = this.getObjet(srcId);
+          if (!dest) {
+            // on peut toujours vider son sac
+            await this.enleverDeConteneur(itemId, src);
           }
-            if (destId) { // Dropped over an item !!!
-              await this.ajouterAConteneur(objetId, destId);
+          else if (!dest.isConteneur()) {
+            // regrouper?
+            await this.regrouperEquipementsSimilaires(item, dest);
           }
+          else if (this.conteneurPeutContenir(dest, item)) {
+            // déplacer?
+            await this.enleverDeConteneur(itemId, src);
+            await this.ajouterDansConteneur(itemId, dest);
           }
         }
       }
       this.computeEncombrementTotalEtMalusArmure();
     }
     if (dragData.type == "Actor") {
-      this.addSubacteur(objetId);
+      this.addSubacteur(itemId);
     }
     return true;
   }
 
+  conteneurPeutContenir(dest, item) {
+    const destData = Misc.data(dest);
+    if (destData?.type != 'conteneur') {
+      ui.notifications.warn(`Impossible de déplacer un objet (${item.name}) vers un ${destData.type} qui n'est pas un conteneur (${dest.name}) !`);
+      return false;
+    }
+    if (this._isConteneurContenu(item, conteneurId)) {
+      ui.notifications.warn(`Impossible de déplacer un conteneur parent (${item.name}) dans un de ses contenus ${destData.name} !`);
+      return false; // Loop detected !
+    }
+
+    // Calculer le total actuel des contenus
+    let encContenu = this.getRecursiveEnc(dest) - Number(destData.data.encombrement);
+    let newEnc = this.getRecursiveEnc(item); // Calculer le total actuel du nouvel objet
+
+    // Teste si le conteneur de destination a suffisament de capacité pour recevoir le nouvel objet
+    if (Number(destData.data.capacite) < encContenu + newEnc) {
+      ui.notifications.warn(
+        `Le conteneur ${dest.name} a une capacité de ${destData.data.capacite}, et contient déjà ${encContenu}.
+        Impossible d'y ranger: ${item.name} d'encombrement ${newEnc}!`);
+      return false;
+    }
+    return true;
+
+  }
+
   /* -------------------------------------------- */
   async moveItemsBetweenActors(itemId, sourceActorId) {
     let itemsList = []
@@ -1042,6 +1022,13 @@ export class RdDActor extends Actor {
     }
   }
 
+  async regrouperEquipementsSimilaires(item, dest) {
+    if (item.isEquipementSimilaire(dest)) {
+      await dest.quantiteIncDec(Misc.templateData(item).quantite);
+      await this.deleteEmbeddedDocuments('Item', [item.id]);
+    }
+  }
+
   /* -------------------------------------------- */
   detectSurEncombrement() {
     return Math.max(0, Math.ceil(Number(this.encTotal) - this.getEncombrementMax()));
@@ -3433,7 +3420,7 @@ export class RdDActor extends Actor {
 
   async diminuerQuantiteObjet(id, nb, options = { supprimerSiZero: false }) {
     const item = this.getObjet(id);
-    if (item){
+    if (item) {
       await item.diminuerQuantite(nb, options);
     }
   }

module/item.js

@@ -4,7 +4,8 @@ import { RdDUtility } from "./rdd-utility.js";
 
 const typesObjetsEquipement = ["objet", "arme", "armure", "conteneur", "herbe", "ingredient", "livre", "potion", "munition", "nourritureboisson", "monnaie"];
 const typesObjetsOeuvres = ["oeuvre", "recettecuisine", "musique", "chant", "danse", "jeu"];
-const encBrin = 0.00005;
+const encBrin = 0.00005;// un brin = 1 décigramme = 1/10g = 1/10000kg = 1/20000 enc
+
 /* -------------------------------------------- */
 export class RdDItem extends Item {
 
@@ -16,17 +17,42 @@ export class RdDItem extends Item {
     return typesObjetsOeuvres;
   }
 
+  isConteneur() {
+    return Misc.data(this).type == 'conteneur';
+  }
+
+  isAlcool() {
+    const itemData = Misc.data(this);
+    return itemData.type == 'nourritureboisson' && itemData.data.boisson && itemData.data.alcoolise;
+  }
+
+  isPotion() {
+    return Misc.data(this).type == 'potion';
+  }
+
+  isEquipement() {
+    return RdDItem.getTypeObjetsEquipement().includes(Misc.data(this).type);
+  }
+
+  getEnc() {
+    const itemData = Misc.data(this);
+    switch (itemData.type) {
+      case 'herbe':
+        return encBrin;
+    }
+    return itemData.data.encombrement
+  }
+
   prepareDerivedData() {
     super.prepareDerivedData();
-    const itemData = Misc.data(this);
-    if (RdDItem.getTypeObjetsEquipement().includes(itemData.type)) {
+    if (this.isEquipement(this)) {
       this._calculsEquipement();
     }
-    if (itemData.type == 'potion') {
+    if (this.isPotion()) {
       this.prepareDataPotion()
     }
+    const itemData = Misc.data(this);
     itemData.data.actionPrincipale = this.getActionPrincipale({ warnIfNot: false });
-
   }
 
   prepareDataPotion() {
@@ -42,7 +68,7 @@ export class RdDItem extends Item {
 
   _calculsEquipement() {
     const tplData = Misc.templateData(this);
-    const quantite = Misc.data(this).type == 'conteneur' ? 1 : (tplData.quantite ?? 0);
+    const quantite = this.isConteneur() ? 1 : (tplData.quantite ?? 0);
     const enc = this.getEnc();
     if (enc != undefined) {
       tplData.encTotal = Math.max(enc, 0) * quantite;
@@ -52,15 +78,6 @@ export class RdDItem extends Item {
     }
   }
 
-  getEnc() {
-    const itemData = Misc.data(this);
-    switch (itemData.type) {
-      case 'herbe':
-        return encBrin;
-    }
-    return itemData.data.encombrement
-  }
-
   getActionPrincipale(options = { warnIfNot: true }) {
     const itemData = Misc.data(this);
     if ((itemData.data.quantite ?? 0) <= 0) {
@@ -80,31 +97,53 @@ export class RdDItem extends Item {
     return undefined;
   }
 
-  isAlcool() {
-    const itemData = Misc.data(this);
-    return itemData.type == 'nourritureboisson' && itemData.data.boisson && itemData.data.alcoolise;
-  }
-
   async diminuerQuantite(nombre, options = { diminuerQuantite: true, supprimerSiZero: false }) {
     if (options.diminuerQuantite == false) return;
+    await this.quantiteIncDec(-nombre, options);
+  }
+
+  async quantiteIncDec(nombre, options = { diminuerQuantite: true, supprimerSiZero: false }) {
     const itemData = Misc.data(this);
     const quantite = itemData.data.quantite;
     if (quantite != undefined) {
-      const reste = Math.max(quantite - nombre, 0);
+      const reste = Math.max(quantite + nombre, 0);
 
-      if (options.supprimerSiZero && reste == 0) {
+      if (reste == 0) {
+        if (options.supprimerSiZero){
           ui.notifications.notify(`${itemData.name} supprimé de votre équipement`);
           await this.delete();
         }
         else {
-        ui.notifications.notify(`Quantité de ${itemData.name} réduite de ${nombre}.${reste == 0
-          ? "Il ne vous en reste plus, vous pouvez le supprimer de votre équipement, ou trouver un moyen de vous en procurer."
-          : ""}`);
+          ui.notifications.notify(`Il ne vous reste plus de ${itemData.name}, vous pouvez le supprimer de votre équipement, ou trouver un moyen de vous en procurer.`);
+          await this.update({ "data.quantite": 0 });
+        }
+      }
+      else {
         await this.update({ "data.quantite": reste });
       }
     }
   }
 
+  /* -------------------------------------------- */
+  // détermine si deux équipements sont similaires: de même type, et avec les même champs hormis la quantité
+  isEquipementSimilaire(other) {
+    const itemData = Misc.data(this);
+    const otherData = Misc.data(other);
+    const tplData = Misc.templateData(this);
+    const otherTplData = Misc.templateData(other);
+    if (!this.isEquipement()) return false;
+    if (itemData.type != otherData.type) return false;
+    if (itemData.name != otherData.name) return false;
+    if (tplData.quantite == undefined) return false;
+
+    for (const [key, value] of Object.entries(tplData)) {
+      if (['quantite', 'encTotal', 'prixTotal'].includes(key)) continue;
+      if (value != otherTplData[key]) return false;
+    }
+    return true;
+  }
+
+
   /* -------------------------------------------- */
   async postItem() {
     console.log(this);