Messages de confirmations

* Confirmer pour monter dans les TMR
* les confirmations peuvent être désactivées (par utilisateur)
* plusieurs groupes différents sont gérés

module/actor.js

@@ -34,6 +34,7 @@ import { DialogItemAchat } from "./dialog-item-achat.js";
 import { RdDItem } from "./item.js";
 import { RdDPossession } from "./rdd-possession.js";
 import { ENTITE_BLURETTE, ENTITE_INCARNE, ENTITE_NONINCARNE, SYSTEM_RDD, SYSTEM_SOCKET_ID } from "./constants.js";
+import { RdDConfirm } from "./rdd-confirm.js";
 
 const POSSESSION_SANS_DRACONIC = {
   img: 'systems/foundryvtt-reve-de-dragon/icons/entites/possession.webp',
@@ -1157,7 +1158,8 @@ export class RdDActor extends Actor {
   /* -------------------------------------------- */
   /** Fonction de remise à plat de l'équipement (ie vide les champs 'contenu') */
   async nettoyerConteneurs() {
-    RdDUtility.confirmer({
+    RdDConfirm.confirmer({
+      settingConfirmer: "confirmation-vider",
       content: `<p>Etes vous certain de vouloir vider tous les conteneurs ?</p>`,
       title: 'Vider les conteneurs',
       buttonLabel: 'Vider',
@@ -3178,7 +3180,17 @@ export class RdDActor extends Actor {
       ui.notifications.warn("Le joueur ou le MJ est déja dans les Terres Médianes avec ce personnage ! Visualisation uniquement");
       mode = "visu"; // bascule le mode en visu automatiquement
     }
+    RdDConfirm.confirmer({
+      bypass: mode == 'visu',
+      settingConfirmer: "confirmation-tmr",
+      content: `<p>Voulez vous monter dans les TMR en mode ${mode}?</p>`,
+      title: 'Confirmer la montée dans les TMR',
+      buttonLabel: 'Monter dans les TMR',
+      onAction: async () => await this._doDisplayTMR(mode)
+    });
+  }
 
+  async _doDisplayTMR(mode) {
     let isRapide = mode == "rapide";
     if (mode != "visu") {
       let minReveValue = (isRapide && !EffetsDraconiques.isDeplacementAccelere(this) ? 3 : 2) + this.countMonteeLaborieuse();

module/item.js

@@ -18,6 +18,10 @@ const typesObjetsEquipement = [
   "potion",
 ]
 const typesObjetsOeuvres = ["oeuvre", "recettecuisine", "musique", "chant", "danse", "jeu"]
+const typesObjetsDraconiques = ["queue", "ombre", "souffle", "tete", "signedraconique", "sortreserve"]
+const typesObjetsConnaissance = ["meditation", "recettealchimique", "sort"]
+const typesObjetsEffet = ["possession", "poison", "maladie"]
+const typesObjetsCompetence = ["competence", "compcreature"]
 const encBrin = 0.00005; // un brin = 1 décigramme = 1/10g = 1/10000kg = 1/20000 enc
 const encPepin = 0.0007; /* un pépin de gemme = 1/10 cm3 = 1/1000 l = 3.5/1000 kg = 7/2000 kg = 7/1000 enc
 densité 3.5 (~2.3 à 4, parfois plus) -- https://www.juwelo.fr/guide-des-pierres/faits-et-chiffres/ 
@@ -74,16 +78,37 @@ export class RdDItem extends Item {
   }
 
   isCompetence() {
-    return this.type == 'competence';
+    return typesObjetsCompetence.includes(this.type)
   }
-
   isEquipement() {
     return typesObjetsEquipement.includes(this.type)
   }
+  isOeuvre() {
+    return typesObjetsOeuvres.includes(this.type)
+  }
+  isDraconique() {
+    return typesObjetsDraconiques.includes(this.type)
+  }
+  isEffet() {
+    return typesObjetsEffet.includes(this.type)
+  }
+  isConnaissance() {
+    return typesObjetsConnaissance.includes(this.type)
+  }
   isConteneur() {
     return this.type == 'conteneur';
   }
 
+  getItemGroup() {
+    if (this.isEquipement()) return "equipement";
+    if (this.isOeuvre()) return "oeuvre";
+    if (this.isDraconique()) return "draconique";
+    if (this.isConnaissance()) return "connaissance";
+    if (this.isEffet()) return "effet";
+    if (this.isCompetence()) return "competence";
+    return "autres";
+  }
+
   isConteneurNonVide() {
     return this.isConteneur() && (this.system.contenu?.length ?? 0) > 0;
   }

module/rdd-confirm.js

@@ -0,0 +1,55 @@
+import { ReglesOptionelles } from "./regles-optionelles.js";
+
+export class RdDConfirm {
+  /* -------------------------------------------- */
+  static confirmer(options, autresActions) {
+    options.bypass = options.bypass || !(options.settingConfirmer == undefined || ReglesOptionelles.isUsing(options.settingConfirmer));
+    if (options.bypass) {
+      options.onAction();
+    }
+    else {
+      let buttons = {
+        "action": RdDConfirm._createButtonAction(options),
+        "cancel": RdDConfirm._createButtonCancel()
+      };
+      if (options.settingConfirmer) {
+        buttons = mergeObject(RdDConfirm._createButtonActionSave(options), buttons);
+      }
+      if (autresActions) {
+        buttons = mergeObject(autresActions, buttons);
+      }
+      const dialogDetails = {
+        title: options.title,
+        content: options.content,
+        default: "cancel",
+        buttons: buttons
+      };
+      new Dialog(dialogDetails, { width: 200 * Object.keys(buttons) }).render(true);
+    }
+  }
+
+  static _createButtonCancel() {
+    return { icon: '<i class="fas fa-times"></i>', label: "Annuler" };
+  }
+
+  static _createButtonAction(options) {
+    return {
+      icon: '<i class="fas fa-check"></i>',
+      label: options.buttonLabel,
+      callback: () => options.onAction()
+    };
+  }
+
+  static _createButtonActionSave(options) {
+    return {
+      "actionSave": {
+        icon: '<i class="far fa-save"></i>',
+        label: options.buttonLabel + "<br>(Ne plus demander)",
+        callback: () => {
+          ReglesOptionelles.set(options.settingConfirmer, false);
+          options.onAction();
+        }
+      }
+    };
+  }
+}

module/rdd-utility.js

@@ -12,6 +12,7 @@ import { RdDItem } from "./item.js";
 import { Monnaie } from "./item-monnaie.js";
 import { RdDPossession } from "./rdd-possession.js";
 import { RdDNameGen } from "./rdd-namegen.js";
+import { RdDConfirm } from "./rdd-confirm.js";
 
 /* -------------------------------------------- */
 // This table starts at 0 -> niveau -10
@@ -262,7 +263,7 @@ export class RdDUtility {
     Handlebars.registerHelper('caseTmr-type', coord => TMRUtility.getTMRType(coord));
     Handlebars.registerHelper('typeTmr-name', coord => TMRUtility.typeTmrName(coord));
     Handlebars.registerHelper('min', (...args) => Math.min(...args.slice(0, -1)));
-    Handlebars.registerHelper('regle-optionnelle', (option) => ReglesOptionelles.isUsing(option) );
+    Handlebars.registerHelper('regle-optionnelle', (option) => ReglesOptionelles.isUsing(option));
 
     Handlebars.registerHelper('filtreTriCompetences', competences => competences.filter(it => it.system.isVisible)
       .sort((a, b) => {
@@ -855,36 +856,14 @@ export class RdDUtility {
     return chatData;
   }
 
-  /* -------------------------------------------- */
-  static confirmer(options) {
-    const d = new Dialog({
-      title: options.title,
-      content: options.content,
-      buttons: {
-        action: {
-          icon: '<i class="fas fa-check"></i>',
-          label: options.buttonLabel,
-          callback: () => {
-            options.onAction();
-          }
-        },
-        cancel: {
-          icon: '<i class="fas fa-times"></i>',
-          label: "Annuler"
-        }
-      },
-      default: "cancel"
-    });
-    d.render(true);
-  }
-
   /* -------------------------------------------- */
   static confirmerSuppressionSubacteur(sheet, subActor, htmlToDelete) {
-    RdDUtility.confirmer({
+    RdDConfirm.confirmer({
-      content: `<p>Etes vous certain de vouloir supprimer le lien vers le véhicule/monture/suivant : ${subActor.name} ?</p>`,
+      settingConfirmer: "confirmation-supprimer-lien-acteur",
+      content: `<p>Etes vous certain de vouloir supprimer le lien vers ${subActor.name} ?</p>`,
       title: 'Confirmer la suppression',
       buttonLabel: 'Supprimer le lien',
-      onAction:  () => {
+      onAction: () => {
         console.log('Delete : ', subActor.id);
         sheet.actor.removeSubacteur(subActor.id);
         RdDUtility.slideOnDelete(sheet, htmlToDelete);
@@ -900,42 +879,36 @@ export class RdDUtility {
       return;
     }
     
-    let msgTxt = '<p>Etes vous certain de vouloir supprimer cet objet ?';
+    const confirmationSuppression = {
-    let buttons = {
+      settingConfirmer: "confirmation-supprimer-" + item.getItemGroup(),
-      delete: {
+      content: `<p>Etes vous certain de vouloir supprimer: ${item.name}?</p>`,
-        icon: '<i class="fas fa-check"></i>',
+      title: `Supprimer ${item.name}`,
-        label: "Supprimer l'objet",
+      buttonLabel: "Supprimer",
-        callback: () => {
+      onAction: () => {
-          console.log('Delete : ', itemId);
+        console.log('Delete : ', itemId);
-          sheet.actor.deleteEmbeddedDocuments('Item', [itemId], { renderSheet: false });
+        sheet.actor.deleteEmbeddedDocuments('Item', [itemId], { renderSheet: false });
-          RdDUtility.slideOnDelete(sheet, htmlToDelete);
+        RdDUtility.slideOnDelete(sheet, htmlToDelete);
-        }
-      },
-      cancel: {
-        icon: '<i class="fas fa-times"></i>',
-        label: "Annuler"
       }
+    };
+    if (item.isConteneurNonVide()) {
+      confirmationSuppression.content += `<p>Ce conteneur n'est pas vide. Que voulez vous supprimer?</p>`;
+      confirmationSuppression.settingConfirmer = undefined;
+      RdDConfirm.confirmer(confirmationSuppression,
+        {
+          'deleteall': {
+            icon: '<i class="fas fa-check"></i>',
+            label: "Supprimer conteneur et contenu",
+            callback: () => {
+              console.log("Delete : ", itemId);
+              sheet.actor.deleteAllConteneur(itemId, { renderSheet: false });
+              RdDUtility.slideOnDelete(sheet, htmlToDelete);
+            }
+          }
+        });
     }
-    if (item.type == 'conteneur' && item.system.contenu.length > 0) {
+    else {
-      msgTxt += "<br>Ce conteneur n'est pas vide. Choisissez l'option de suppression";
+      RdDConfirm.confirmer(confirmationSuppression)
-      buttons['deleteall'] = {
-        icon: '<i class="fas fa-check"></i>',
-        label: "Supprimer le conteneur et tout son contenu",
-        callback: () => {
-          console.log("Delete : ", itemId);
-          sheet.actor.deleteAllConteneur(itemId, { renderSheet: false });
-          RdDUtility.slideOnDelete(sheet, htmlToDelete);
-        }
-      }
     }
-    msgTxt += "</p>";
-    let d = new Dialog({
-      title: "Confirmer la suppression",
-      content: msgTxt,
-      buttons: buttons,
-      default: "cancel"
-    });
-    d.render(true);
   }
 
   static slideOnDelete(sheet, htmlToDelete) {

module/regles-optionelles.js

@@ -11,11 +11,22 @@ const listeReglesOptionelles = [
   { name: 'degat-minimum-malus-libre-simple', group: 'Règles de combat', descr: "Le malus libre d'attaque remplace une des valeurs de dés d'encaissement si elle est plus petite. Exemple : la difficulté libre de l'attaquant est de -4. Sur le jet d'encaissement, si 1 résultat est inférieur à 4, alors il devient 4.", default: false },
   { name: 'degat-minimum-malus-libre', group: 'Règles de combat', descr: "Le malus libre d'attaque remplace une valeur de dés d'encaissement si elle est plus petite. Exemple : la difficulté libre de l'attaquant est de -4. Sur le jet d'encaissement, tout résultat inférieur à 4 devient 4.", default: false },
   { name: 'degat-ajout-malus-libre', group: 'Règles de combat', descr: "Le malus libre d'attaque s'ajoute au jet d'encaissement et aux autres bonus. Exemple : la difficulté libre de l'attaquant est de -4. Le jet d'encaissement est effectué à 2d10+4, plus les bonus de situation et d'armes.", default: false },
-  { name: 'astrologie', group: 'Règles générales', descr: "Appliquer les ajustements astrologiques aux jets de chance et aux rituels", default: true },
+  { name: 'astrologie', group: 'Règles générales', descr: "Appliquer les ajustements astrologiques aux jets de chance et aux rituels"},
-  { name: 'afficher-prix-joueurs', group: 'Règles générales', descr: "Afficher le prix de l'équipement des joueurs", default: true, uniquementJoueur: true},
+  { name: 'afficher-prix-joueurs', group: 'Règles générales', descr: "Afficher le prix de l'équipement des joueurs", uniquementJoueur: true},
-  { name: 'appliquer-fatigue', group: 'Règles générales', descr: "Appliquer les règles de fatigue", default: true },
+  { name: 'appliquer-fatigue', group: 'Règles générales', descr: "Appliquer les règles de fatigue"},
   { name: 'afficher-colonnes-reussite', group: 'Règles générales', descr: "Afficher le nombre de colonnes de réussite ou d'échec", default: false },
+  { name: 'confirmation-tmr', group: 'Confirmations', descr: "Confirmer pour monter dans les TMR", scope: "client"},
+  { name: 'confirmation-vider', group: 'Confirmations', descr: "Confirmer pour vider l'équipement", scope: "client"},
+  { name: 'confirmation-supprimer-lien-acteur',  group: 'Confirmations', descr: "Confirmer pour détacher un animal/suivant/véhicule", scope: "client"},
+  { name: 'confirmation-supprimer-equipement',   group: 'Confirmations', descr: "Confirmer la suppression des équipements", scope: "client"},
+  { name: 'confirmation-supprimer-oeuvre',       group: 'Confirmations', descr: "Confirmer la suppression des oeuvres", scope: "client"},
+  { name: 'confirmation-supprimer-connaissance', group: 'Confirmations', descr: "Confirmer la suppression des connaissances", scope: "client"},
+  { name: 'confirmation-supprimer-draconique',   group: 'Confirmations', descr: "Confirmer la suppression des queues, souffles, têtes", scope: "client"},
+  { name: 'confirmation-supprimer-effet',        group: 'Confirmations', descr: "Confirmer la suppression des effets (maladie, poisons, possessions)", scope: "client"},
+  { name: 'confirmation-supprimer-competence',   group: 'Confirmations', descr: "Confirmer la suppression des compétences", scope: "client"},
+  { name: 'confirmation-supprimer-autres',       group: 'Confirmations', descr: "Confirmer la suppression des autres types d'Objets", scope: "client"},
 ];
+
 const uniquementJoueur = listeReglesOptionelles.filter(it => it.uniquementJoueur).map(it=>it.name);
 
 export class ReglesOptionelles extends FormApplication {
@@ -23,7 +34,7 @@ export class ReglesOptionelles extends FormApplication {
     for (const regle of listeReglesOptionelles) {
       const name = regle.name;
       const id = ReglesOptionelles._getIdRegle(name);
-      game.settings.register(SYSTEM_RDD, id, { name: id, scope: "world", config: false, default: regle.default == undefined ? true : regle.default, type: Boolean });
+      game.settings.register(SYSTEM_RDD, id, { name: id, scope: regle.scope ?? "world", config: false, default: regle.default == undefined ? true : regle.default, type: Boolean });
     }
 
     game.settings.registerMenu(SYSTEM_RDD, "rdd-options-regles", {
@@ -31,8 +42,7 @@ export class ReglesOptionelles extends FormApplication {
       label: "Choix des règles optionelles",
       hint: "Ouvre la fenêtre de sélection des règles optionelles",
       icon: "fas fa-bars",
-      type: ReglesOptionelles,
+      type: ReglesOptionelles
-      restricted: true
     });
   }
 
@@ -50,7 +60,7 @@ export class ReglesOptionelles extends FormApplication {
       id: "optional-settings",
       template: "systems/foundryvtt-reve-de-dragon/templates/regles-optionelles.html",
       height: 600,
-      width: 350,
+      width: 450,
       minimizable: false,
       closeOnSubmit: true,
       title: "Règles optionnelles"
@@ -60,7 +70,7 @@ export class ReglesOptionelles extends FormApplication {
 
   getData() {
     let formData = super.getData();
-    const regles = listeReglesOptionelles.map(it => {
+    const regles = listeReglesOptionelles.filter(it => game.user.isGM || it.scope == "client").map(it => {
       it = duplicate(it);
       it.id = ReglesOptionelles._getIdRegle(it.name);
       it.active = ReglesOptionelles.isUsing(it.name);
@@ -78,6 +88,14 @@ export class ReglesOptionelles extends FormApplication {
     return game.settings.get(SYSTEM_RDD, ReglesOptionelles._getIdRegle(name));
   }
 
+  static isSet(name) {
+    return ReglesOptionelles.isUsing(name);
+  }
+
+  static set(name, value) {
+    return game.settings.set(SYSTEM_RDD, ReglesOptionelles._getIdRegle(name), value ? true: false);
+  }
+
   activateListeners(html) {
     html.find(".select-option").click((event) => {
       if (event.currentTarget.attributes.name) {