Avancement OwnedItems et Items

- remplacer xxxOwnedItems par xxxEmbeddedDocuments
- corrections des accès aux items vs itemData
- corrections cases spéciales TMR et effets draconiques
- correction monnaies/payer

module/tmr/conquete.js

@@ -29,14 +29,14 @@ export class Conquete extends Draconique {
   }
 
   async _creerConquete(actor, queue) {
-    let existants = actor.data.items.filter(it => this.isCase(it)).map(it => it.data.coord);
+    let existants = actor.data.items.filter(it => this.isCase(it)).map(it => Misc.data(it).data.coord);
     let possibles = TMRUtility.filterTMR(tmr => !TMRUtility.isCaseHumide(tmr) && !existants.includes(tmr.coord));
     let conquete = Misc.rollOneOf(possibles);
-    await this.createCaseTmr(actor, 'Conquête: ' + conquete.label, conquete, queue._id);
+    await this.createCaseTmr(actor, 'Conquête: ' + conquete.label, conquete, queue.id);
   }
 
   async onActorDeleteCaseTmr(actor, casetmr) {
-    await actor.deleteOwnedItem(casetmr.data.sourceid);
+    await actor.deleteEmbeddedDocuments('Item', [casetmr.data.sourceid]);
   }
 
 }

module/tmr/debordement.js

@@ -14,7 +14,7 @@ export class Debordement extends Draconique {
   async onActorCreateOwned(actor, souffle) {
     const existants = actor.data.items.filter(it => this.isCase(it)).map(it => it.data.coord);
     const tmr = TMRUtility.getTMRAleatoire(it => !(TMRUtility.isCaseHumide(it) || existants.includes(it.coord)));
-    await this.createCaseTmr(actor, 'Debordement: ' + tmr.label, tmr, souffle._id);
+    await this.createCaseTmr(actor, 'Debordement: ' + tmr.label, tmr, souffle.id);
   }
 
   code() { return 'debordement' }

module/tmr/desorientation.js

@@ -43,7 +43,7 @@ export class Desorientation extends Draconique {
     const existants = actor.data.items.filter(it => this.isCase(it)).map(it => it.data.coord);
     let tmrs = TMRUtility.filterTMR(it => it.type == type && !existants.includes(it.coord));
     for (let tmr of tmrs) {
-      await this.createCaseTmr(actor, 'Désorientation: ' + tmr.label, tmr, souffle._id);
+      await this.createCaseTmr(actor, 'Désorientation: ' + tmr.label, tmr, souffle.id);
     }
   }
 

module/tmr/draconique.js

@@ -1,3 +1,4 @@
+import { Misc } from "../misc.js";
 import { TMRUtility } from "../tmr-utility.js";
 import { PixiTMR } from "./pixi-tmr.js";
 
@@ -8,11 +9,11 @@ const registeredEffects = [
  * Définition des informations d'une "draconique" (queue, ombre, tête, souffle) qui influence les TMR
  */
 export class Draconique {
-  static isCaseTMR(element) { return element.type == 'casetmr'; }
-  static isQueueDragon(element) { return element.type == 'queue' || element.type == 'ombre'; }
-  static isSouffleDragon(element) { return element.type == 'souffle'; }
-  static isTeteDragon(element) { return element.type == 'tete'; }
-  static isQueueSouffle(it) { return Draconique.isQueueDragon(it) || Draconique.isSouffleDragon(it); }
+  static isCaseTMR(itemData) { return itemData.type == 'casetmr'; }
+  static isQueueDragon(itemData) { return itemData.type == 'queue' || itemData.type == 'ombre'; }
+  static isSouffleDragon(itemData) { return itemData.type == 'souffle'; }
+  static isTeteDragon(itemData) { return itemData.type == 'tete'; }
+  static isQueueSouffle(itemData) { return Draconique.isQueueDragon(itemData) || Draconique.isSouffleDragon(itemData); }
 
   tmrLabel(linkData) { return TMRUtility.getTMRLabel(linkData.data.coord); }
 
@@ -37,7 +38,8 @@ export class Draconique {
    * @returns true si l'item correspond
    */
   match(item) {
-    return Draconique.isQueueDragon(item) || Draconique.isSouffleDragon(item) || Draconique.isTeteDragon(item);
+    const itemData = Misc.data(item);
+    return Draconique.isQueueDragon(itemData) || Draconique.isSouffleDragon(itemData) || Draconique.isTeteDragon(itemData);
   }
 
   /**
@@ -110,35 +112,40 @@ export class Draconique {
 
   /**
    * 
-   * @param {*} it un item à tester
+   * @param {*} item un item à tester
    * @param {*} coord les coordonnées d'une case. Si undefined toute case du type correspondra, 
    */
-  isCase(it, coord = undefined) {
-    return Draconique.isCaseTMR(it) && it.data.specific == this.code() && (coord ? it.data.coord == coord : true);
+  isCase(item, coord = undefined) {
+    const itemData = Misc.data(item);
+    return Draconique.isCaseTMR(itemData) && itemData.data.specific == this.code() && (coord ? itemData.data.coord == coord : true);
   }
-
+  
   find(list, coord = undefined) {
-    return list.find(c => this.isCase(c, coord));
+    return list.find(c => this.isCase(Misc.data(c), coord));
   }
-
+  
   async createCaseTmr(actor, label, tmr, sourceId = undefined) {
-    await actor.createEmbeddedDocuments('Item',[{
-      name: label, type: 'casetmr', img: this.img(), _id: randomID(16),
+    const casetmrData = {
+      name: label, type: 'casetmr', img: this.img(),
       data: { coord: tmr.coord, specific: this.code(), sourceid: sourceId }
-    }]);
+    };
+    await actor.createEmbeddedDocuments('Item', [casetmrData]);
   }
-
+  
   async deleteCasesTmr(actor, draconique) {
-    let caseTmrs = actor.data.items.filter(it => this.isCase(it) && it.data.sourceid == draconique._id);
-    for (let casetmr of caseTmrs) {
-      await actor.deleteOwnedItem(casetmr._id);
-    }
+    let caseTmrs = actor.data.items.filter(it => this.isCaseForSource(it, draconique));
+    await actor.deleteEmbeddedDocuments('Item', caseTmrs.map(it => it.id));
+  }
+  
+  isCaseForSource(item, draconique) {
+    const itemData = Misc.data(item);
+    return Draconique.isCaseTMR(itemData) && itemData.data.specific == this.code() && itemData.data.sourceid == draconique.id;
   }
 
   async onVisiteSupprimer(actor, tmr, onRemoveToken) {
     let existants = actor.data.items.filter(it => this.isCase(it, tmr.coord));
+    await actor.deleteEmbeddedDocuments('Item', existants.map(it => it.id));
     for (let casetmr of existants) {
-      await actor.deleteOwnedItem(casetmr._id);
       onRemoveToken(tmr, casetmr);
     }
   }

module/tmr/fermeture-cites.js

@@ -32,7 +32,7 @@ export class FermetureCites extends Draconique {
     let existants = actor.data.items.filter(it => this.isCase(it)).map(it => it.data.coord);
     let ouvertes = TMRUtility.filterTMR(it => it.type == 'cite' && !existants.includes(it.coord));
     for (let tmr of ouvertes) {
-      await this.createCaseTmr(actor, 'Fermeture: ' + tmr.label, tmr, souffle._id);
+      await this.createCaseTmr(actor, 'Fermeture: ' + tmr.label, tmr, souffle.id);
     }
   }
 }

module/tmr/pelerinage.js

@@ -14,7 +14,7 @@ export class Pelerinage extends Draconique {
 
   async onActorCreateOwned(actor, queue) { 
     let tmr = TMRUtility.getTMRAleatoire();
-    await this.createCaseTmr(actor, 'Pèlerinage: ' + tmr.label, tmr, queue._id);
+    await this.createCaseTmr(actor, 'Pèlerinage: ' + tmr.label, tmr, queue.id);
   }
   
   
@@ -32,7 +32,7 @@ export class Pelerinage extends Draconique {
   }
   
   async onActorDeleteCaseTmr(actor, casetmr) {
-    await actor.deleteOwnedItem(casetmr.data.sourceid);
+    await actor.deleteEmbeddedDocuments('Item', [casetmr.data.sourceid]);
   }
 
 }

module/tmr/periple.js

@@ -16,7 +16,7 @@ export class Periple extends Draconique {
     let terrain = new Roll("1d2").evaluate().total == 1 ? 'sanctuaire' : 'necropole';
     let tmrs = TMRUtility.getListTMR(terrain);
     for (let tmr of tmrs) {
-      await this.createCaseTmr(actor, 'Périple: ' + tmr.label, tmr, souffle._id);
+      await this.createCaseTmr(actor, 'Périple: ' + tmr.label, tmr, souffle.id);
     }
   }
 

module/tmr/pont-impraticable.js

@@ -14,7 +14,7 @@ export class PontImpraticable extends Draconique {
   async onActorCreateOwned(actor, souffle) {
     const ponts = TMRUtility.getListTMR('pont');
     for (let tmr of ponts) {
-      await this.createCaseTmr(actor, 'Pont impraticable: ' + tmr.label, tmr, souffle._id);
+      await this.createCaseTmr(actor, 'Pont impraticable: ' + tmr.label, tmr, souffle.id);
     }
   }
 

module/tmr/present-cites.js

@@ -40,7 +40,7 @@ export class PresentCites extends Draconique {
     else {
       let cites = TMRUtility.filterTMR(it => it.type == 'cite');
       for (let tmr of cites) {
-        await this.createCaseTmr(actor, 'Présent: ' + tmr.label, tmr, tete._id);
+        await this.createCaseTmr(actor, 'Présent: ' + tmr.label, tmr, tete.id);
       }
     }
   }
@@ -60,6 +60,6 @@ export class PresentCites extends Draconique {
   }
 
   async ouvrirLePresent(actor, casetmr) {
-    await actor.deleteOwnedItem(casetmr._id);
+    await actor.deleteEmbeddedDocuments('Item', [casetmr.id]);
   }
 }

module/tmr/quete-eaux.js

@@ -11,7 +11,7 @@ export class QueteEaux extends Draconique {
   match(item) { return Draconique.isTeteDragon(item) && Grammar.toLowerCaseNoAccent(item.name).includes("quete des eaux"); }
   manualMessage() { return "Vous devrez re-configurer votre Quête des Eaux une fois un lac ou marais vaincu" }
   async onActorCreateOwned(actor, tete) {
-    await this.createCaseTmr(actor, "Quête des eaux à déterminer", { coord: 'A0' }, tete._id);
+    await this.createCaseTmr(actor, "Quête des eaux à déterminer", { coord: 'A0' }, tete.id);
   }
 
   code() { return 'maitrisee' }

module/tmr/reserve-extensible.js

@@ -13,7 +13,7 @@ export class ReserveExtensible extends Draconique {
   async onActorCreateOwned(actor, tete) {
     const existants = actor.data.items.filter(it => this.isCase(it)).map(it => it.data.coord);
     const tmr = TMRUtility.getTMRAleatoire(it => !(it.type == 'fleuve' || existants.includes(it.coord)));
-    await this.createCaseTmr(actor, "Nouvelle Réserve extensible", tmr, tete._id);
+    await this.createCaseTmr(actor, "Nouvelle Réserve extensible", tmr, tete.id);
   }
 
   code() { return 'reserve_extensible' }

module/tmr/terre-attache.js

@@ -12,7 +12,7 @@ export class TerreAttache extends Draconique {
   manualMessage() { return "Vous pouvez re-configurer votre Terre d'Attache" }
 
   async onActorCreateOwned(actor, tete) {
-    await this.createCaseTmr(actor, "Terre d'attache à déterminer", { coord: 'A0' }, tete._id);
+    await this.createCaseTmr(actor, "Terre d'attache à déterminer", { coord: 'A0' }, tete.id);
   }
 
   code() { return 'attache' }

module/tmr/trou-noir.js

@@ -14,7 +14,7 @@ export class TrouNoir extends Draconique {
   async onActorCreateOwned(actor, souffle) {
     const existants = actor.data.items.filter(it => this.isCase(it)).map(it => it.data.coord);
     const tmr = TMRUtility.getTMRAleatoire(it => !(TMRUtility.isCaseHumide(it) || existants.includes(it.coord)));
-    await this.createCaseTmr(actor, 'Trou noir: ' + tmr.label, tmr, souffle._id);
+    await this.createCaseTmr(actor, 'Trou noir: ' + tmr.label, tmr, souffle.id);
   }
 
   code() { return 'trounoir' }

module/tmr/urgence-draconique.js

@@ -23,20 +23,20 @@ export class UrgenceDraconique extends Draconique {
         whisper: ChatUtility.getWhisperRecipientsAndGMs(game.user.name),
         content: `En l'absence de sorts en réserve, l'urgence draconique de ${actor.name} se transforme en ${ideeFixe.name}`
       });
-      await actor.createEmbeddedDocuments('Item',[ideeFixe]);
-      await actor.deleteOwnedItem(queue._id);
+      await actor.createEmbeddedDocuments('Item', [ideeFixe]);
+      await actor.deleteEmbeddedDocuments('Item', [queue.id]);
       return;
     }
     else {
       const demiReve = actor.getDemiReve();
       coordSortsReserve.sort(Misc.ascending(t => TMRUtility.distanceTMR(t, demiReve)));
       const tmr = TMRUtility.getTMR(coordSortsReserve[0]);
-      await this.createCaseTmr(actor, 'Urgence draconique: ' + tmr.label, tmr, queue._id);
+      await this.createCaseTmr(actor, 'Urgence draconique: ' + tmr.label, tmr, queue.id);
     }
   }
 
   async onActorDeleteCaseTmr(actor, casetmr) {
-    await actor.deleteOwnedItem(casetmr.data.sourceid);
+    await actor.deleteEmbeddedDocuments('Item', [casetmr.data.sourceid]);
   }
 
   code() { return 'urgence' }