Simplifier la sélection

Lorsque plusieurs tokens sont ciblés, laisser le joueur choisir parmi
ceux-là

module/actor.js

@@ -35,6 +35,7 @@ import { ENTITE_BLURETTE, ENTITE_INCARNE, ENTITE_NONINCARNE, HIDE_DICE, SHOW_DIC
 import { RdDConfirm } from "./rdd-confirm.js";
 import { DialogValidationEncaissement } from "./dialog-validation-encaissement.js";
 import { RdDRencontre } from "./item-rencontre.js";
+import { DialogSelectTarget } from "./dialog-select-target.js";
 
 const POSSESSION_SANS_DRACONIC = {
   img: 'systems/foundryvtt-reve-de-dragon/icons/entites/possession.webp',
@@ -2551,18 +2552,17 @@ export class RdDActor extends Actor {
       competence: this.getCompetence(idOrName)
     }
     if (rollData.competence.type == 'competencecreature') {
-      if (rollData.competence.system.iscombat && options.tryTarget) {
-        const target = RdDCombat.getTarget();
-        if (target) {
-          if (rollData.competence.system.ispossession) {
-            RdDPossession.onAttaquePossession(target, this, rollData.competence)
-          }
-          else {
-            const arme = RdDItemCompetenceCreature.armeNaturelle(rollData.competence)
-            RdDCombat.rddCombatTarget(this, target).attaque(competence, arme)
-          }
-          return;
-        }
+      if (rollData.competence.system.iscombat && options.tryTarget && DialogSelectTarget.hasTargets()) {
+        DialogSelectTarget.selectOneToken(target => {
+            if (rollData.competence.system.ispossession) {
+              RdDPossession.onAttaquePossession(target, this, rollData.competence)
+            }
+            else {
+              const arme = RdDItemCompetenceCreature.armeNaturelle(rollData.competence)
+              RdDCombat.rddCombatTarget(target, this).attaque(competence, arme)
+            }
+          });
+        return;
       }
       // Transformer la competence de créature
       RdDItemCompetenceCreature.setRollDataCreature(rollData)
@@ -3222,8 +3222,7 @@ export class RdDActor extends Actor {
 
   /* -------------------------------------------- */
   rollArme(arme) {
-    const target = RdDCombat.getTarget();
-    if (!target) {
+    if (!DialogSelectTarget.hasTargets()) {
       RdDConfirm.confirmer({
         settingConfirmer: "confirmer-combat-sans-cible",
         content: `<p>Voulez vous faire un jet de compétence ${arme.system.competence} sans choisir de cible valide?
@@ -3237,20 +3236,20 @@ export class RdDActor extends Actor {
       });
       return;
     }
-    if (RdDCombat.isTargetEntite(target)){
-      ui.notifications.warn(`Vous ne pouvez pas attaquer une entité non incarnée avec votre ${arme.name}!!!!`);
-      return;
-    }
-    // if (RdDCombat.isTargetEntite(targets[0])) {
-    //   ui.notifications.warn("Vous ne pouvez pas cibler une entité non incarnée!");
-    //   return;
-    // }
+    DialogSelectTarget.selectOneToken(target => {
+      if (RdDCombat.isTargetEntite(target)){
+        ui.notifications.warn(`Vous ne pouvez pas attaquer une entité non incarnée avec votre ${arme.name}!!!!`);
+        return;
+      }
+  
+      
+      const competence = this.getCompetence(arme.system.competence)
+      if (competence.system.ispossession) {
+        return RdDPossession.onAttaquePossession(target, this, competence);
+      }
+      RdDCombat.rddCombatTarget(target, this).attaque(competence, arme);
+    })
 
-    const competence = this.getCompetence(arme.system.competence)
-    if (competence.system.ispossession) {
-      return RdDPossession.onAttaquePossession(target, this, competence);
-    }
-    RdDCombat.rddCombatTarget(this, target).attaque(competence, arme);
   }
 
   /* -------------------------------------------- */

module/dialog-select-target.js

@@ -0,0 +1,60 @@
+
+export class DialogSelectTarget extends Dialog {
+  static hasTargets() {
+    return (game.user.targets?.size ?? 0) > 0;
+  }
+
+  static async selectOneToken(onSelectTarget = target => { }) {
+    if (DialogSelectTarget.hasTargets()) {
+      const targets = game.user.targets.map(it => it);
+      switch (targets.size) {
+        case 0: return;
+        case 1:
+          onSelectTarget(targets[0]);
+          return;
+        default:
+          {
+            const tokens = targets.map(it => { return { id: it.id, name: it.document.name, img: it.document.texture.src ?? it.actor.img ?? 'icons/svg/mystery-man.svg' } })
+            const html = await renderTemplate("systems/foundryvtt-reve-de-dragon/templates/dialog-select-target.html", {
+              tokens: tokens
+            });
+            new DialogSelectTarget(html, onSelectTarget, targets).render(true);
+          }
+      }
+    }
+  }
+
+  constructor(html, onSelectTarget, targets) {
+    const options = {
+      classes: ["rdd-dialog-select-target"],
+      width: 'fit-content',
+      height: 'fit-content',
+      'max-height': 600,
+      'z-index': 99999
+    };
+    const conf = {
+      title: "Choisir une cible",
+      content: html,
+      buttons: {}
+    };
+    super(conf, options);
+    this.onSelectTarget = onSelectTarget;
+    this.targets = targets;
+  }
+
+  activateListeners(html) {
+    super.activateListeners(html);
+    html.find("li.select-target").click((event) => {
+      this.targetSelected($(event.currentTarget)?.data("token-id"));
+    });
+  }
+
+
+  targetSelected(tokenId) {
+    const target = this.targets.find(it => it.id == tokenId);
+    this.close();
+    if (target) {
+      this.onSelectTarget(target);
+    }
+  }
+}

module/rdd-combat.js

@@ -1,5 +1,6 @@
 import { ChatUtility } from "./chat-utility.js";
 import { ENTITE_BLURETTE, ENTITE_INCARNE, ENTITE_NONINCARNE, HIDE_DICE, SYSTEM_RDD, SYSTEM_SOCKET_ID } from "./constants.js";
+import { DialogSelectTarget } from "./dialog-select-target.js";
 import { Grammar } from "./grammar.js";
 import { RdDItemArme } from "./item-arme.js";
 import { RdDItemCompetence } from "./item-competence.js";
@@ -469,17 +470,6 @@ export class RdDCombat {
     return true;
   }
 
-  /* -------------------------------------------- */
-  static rddCombatTarget(attacker, target) {
-    const defender = target?.actor;
-    const defenderTokenId = target?.id;
-    return new RdDCombat(attacker, defender, defenderTokenId, target)
-  }
-
-  static isTargetEntite(target) {
-    return target?.actor.type == 'entite' && target?.actor.system.definition.typeentite == ENTITE_NONINCARNE;
-  }
-
   /* -------------------------------------------- */
   static getTarget() {
     const targets = game.user.targets;
@@ -488,10 +478,25 @@ export class RdDCombat {
         for (let t of targets) {
           return t;
         }
+      case 0:
+        ui.notifications.warn("Vous devez choisir une cible à attaquer!");
+        break;
       default:
+        DialogSelectTarget.selectOneToken(t => console.info(`selecte target ${t}`));
         ui.notifications.warn("Vous devez choisir une cible (et <strong>une seule</strong>) à attaquer!");
         return;
+      }    
+    }  
+
+    static isTargetEntite(target) {
+      return target?.actor.type == 'entite' && target?.actor.system.definition.typeentite == ENTITE_NONINCARNE;
     }
+
+  /* -------------------------------------------- */
+  static rddCombatTarget(target, attacker) {
+    const defender = target?.actor;
+    const defenderTokenId = target?.id;
+    return new RdDCombat(attacker, defender, defenderTokenId, target)
   }
 
   /* -------------------------------------------- */
@@ -500,6 +505,7 @@ export class RdDCombat {
     let defender = defenderTokenId ? canvas.tokens.get(defenderTokenId)?.actor : undefined;
     let target = undefined
     if (!defenderTokenId || !defender) {
+      console.warn(`RdDCombat.rddCombatForAttackerAndDefender: appel avec defenderTokenId ${defenderTokenId} incorrect, ou pas de defender correspondant`);
       target = RdDCombat.getTarget()
       if (!target) {
         return;