From 3f4d52487df0c3d245ece94f054847b5f2e556bc Mon Sep 17 00:00:00 2001 From: Vincent Vandemeulebrouck Date: Sun, 28 Feb 2021 01:50:15 +0100 Subject: [PATCH] =?UTF-8?q?Refactor:=20extrait=20m=C3=A9thodes=20communes?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - suppression de casetmr lors de visite/conquete - suppression de toutes les cases TMRs liées - notification lors de suppression d'une casetmr pour enlever la queue # Conflicts: # module/actor.js --- module/actor.js | 10 ++++++++++ module/rdd-tmr-dialog.js | 20 ++++++++++--------- module/tmr/conquete.js | 7 ------- module/tmr/desorientation.js | 10 ---------- module/tmr/draconique.js | 36 +++++++++++++++++++++++++---------- module/tmr/fermeture-cites.js | 7 ------- module/tmr/pelerinage.js | 9 --------- 7 files changed, 47 insertions(+), 52 deletions(-) diff --git a/module/actor.js b/module/actor.js index 659a971d..0f497714 100644 --- a/module/actor.js +++ b/module/actor.js @@ -3205,6 +3205,9 @@ export class RdDActor extends Actor { case 'souffle': await this.onDeleteOwnedDraconique(item, options, id); break; + case 'casetmr': + await this.onDeleteOwnedCaseTmr(item, options, id); + break; } } @@ -3226,6 +3229,13 @@ export class RdDActor extends Actor { } } + async onDeleteOwnedCaseTmr(item, options, id) { + let draconique = Draconique.all().find(it => it.isCase(item)); + if (draconique) { + draconique.onActorDeleteCaseTmr(this, item) + } + } + notifyGestionTeteSouffleQueue(item, manualMessage = true) { ChatMessage.create({ whisper: ChatUtility.getWhisperRecipientsAndGMs(game.user.name), diff --git a/module/rdd-tmr-dialog.js b/module/rdd-tmr-dialog.js index a3bcff7e..ecbc3caf 100644 --- a/module/rdd-tmr-dialog.js +++ b/module/rdd-tmr-dialog.js @@ -558,8 +558,10 @@ export class RdDTMRDialog extends Dialog { await this._conquerir(tmr, { difficulte: -9, action: 'Conquérir la cité', - onConqueteReussie: r => EffetsDraconiques.fermetureCites.onConquete(r.actor, tmr, (casetmr) => this.removeToken(tmr, casetmr)), - onConqueteEchec: r => this.close(), + onConqueteReussie: r => EffetsDraconiques.fermetureCites.onVisiteSupprimer(r.actor, tmr, (casetmr) => this.removeToken(tmr, casetmr)), + onConqueteEchec: r => { + this.close() + }, canClose: false }); } @@ -576,8 +578,8 @@ export class RdDTMRDialog extends Dialog { await this._conquerir(tmr, { difficulte: -7, action: 'Conquérir', - onConqueteReussie: r => EffetsDraconiques.conquete.onConquete(r.actor, tmr, (casetmr) => this.removeToken(tmr, casetmr)), - onConqueteEchec: r => { }, + onConqueteReussie: r => EffetsDraconiques.conquete.onVisiteSupprimer(r.actor, tmr, (casetmr) => this.removeToken(tmr, casetmr)), + onConqueteEchec: r => this.close(), canClose: false }); } @@ -639,8 +641,8 @@ export class RdDTMRDialog extends Dialog { dialog.render(true); } - async validerPelerinage(tmr) { - await EffetsDraconiques.pelerinage.onFinPelerinage(this.actor, tmr, (casetmr) => this.removeToken(tmr, casetmr)); + async validerVisite(tmr) { + await EffetsDraconiques.pelerinage.onVisiteSupprimer(this.actor, tmr, (casetmr) => this.removeToken(tmr, casetmr)); } @@ -836,17 +838,17 @@ export class RdDTMRDialog extends Dialog { await this.manageCaseHumide(tmr); await this.conquerirCiteFermee(tmr); await this.conquerirTMR(tmr); - await this.validerPelerinage(tmr); + await this.validerVisite(tmr); await this.declencheSortEnReserve(tmr.coord); await this.actor.checkSoufflePeage(tmr); } } - + /* -------------------------------------------- */ async forceDemiRevePositionView() { this._updateDemiReve(); } - + /* -------------------------------------------- */ async forceDemiRevePosition(coord) { await this.actor.updateCoordTMR(coord); diff --git a/module/tmr/conquete.js b/module/tmr/conquete.js index 92d665c4..550c6427 100644 --- a/module/tmr/conquete.js +++ b/module/tmr/conquete.js @@ -35,11 +35,4 @@ export class Conquete extends Draconique { await this.createCaseTmr(actor, 'Conquête: ' + conquete.label, conquete, queue._id); } - async onConquete(actor, tmr, onRemoveToken) { - let existants = actor.data.items.filter(it => this.isCase(it, tmr.coord)); - for (let casetmr of existants) { - await actor.deleteOwnedItem(casetmr.data.sourceid); - onRemoveToken(tmr, casetmr); - } - } } diff --git a/module/tmr/desorientation.js b/module/tmr/desorientation.js index f2dad665..1dcec96a 100644 --- a/module/tmr/desorientation.js +++ b/module/tmr/desorientation.js @@ -24,10 +24,6 @@ export class Desorientation extends Draconique { return Object.keys(TMRType).filter(it => !dejaDesorientes.includes(it)); } - async onActorDeleteOwned(actor, souffle) { - await this._supprimerCasesTmr(actor, souffle); - } - code() { return 'desorientation' } tooltip(linkData) { return `Désorientation, cette case n'existe plus !` } img() { return 'icons/svg/explosion.svg' } @@ -51,11 +47,5 @@ export class Desorientation extends Draconique { } } - async _supprimerCasesTmr(actor, souffle) { - let caseTmrs = actor.data.items.filter(it => it.data.sourceId == souffle._id); - for (let casetmr of caseTmrs) { - await actor.deleteOwnedItem(casetmr._id); - } - } } diff --git a/module/tmr/draconique.js b/module/tmr/draconique.js index 9af391ec..bbbac5fe 100644 --- a/module/tmr/draconique.js +++ b/module/tmr/draconique.js @@ -7,8 +7,7 @@ const registeredEffects = [ /** * Définition des informations d'une "draconique" (queue, ombre, tête, souffle) qui influence les TMR */ -export class Draconique -{ +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'; } @@ -28,6 +27,7 @@ export class Draconique static all() { return Object.values(registeredEffects); } + static get(code) { return registeredEffects[code]; } @@ -56,10 +56,11 @@ export class Draconique } async onActorDeleteOwned(actor, item) { - let caseTmrs = actor.data.items.filter(it => this.isCase(it) && it.data.sourceid == item._id); - for (let casetmr of caseTmrs) { - await actor.deleteOwnedItem(casetmr._id); - } + this.deleteCasesTmr(actor, item); + return false; + } + + async onActorDeleteCaseTmr(actor, casetmr) { return false; } /** @@ -72,7 +73,7 @@ export class Draconique * @returns un tooltip à afficher au dessus du token */ tooltip(linkData) { return undefined } - + /** * @param {*} img l'url du fichier image à utiliser pour le token. Si indéfini (et si createSprite n'est pas surchargé), * un disque est utilisé. @@ -102,7 +103,7 @@ export class Draconique if (this.img()) { return pixiTMR.sprite(this.code()); } - else{ + else { return pixiTMR.circle() } } @@ -120,11 +121,26 @@ export class Draconique return list.find(c => this.isCase(c, coord)); } - async createCaseTmr(actor, label, tmr, sourceId=undefined) { + async createCaseTmr(actor, label, tmr, sourceId = undefined) { await actor.createOwnedItem({ name: label, type: 'casetmr', img: this.img(), _id: randomID(16), - data: { coord: tmr.coord, specific: this.code(), sourceid:sourceId } + data: { coord: tmr.coord, specific: this.code(), sourceid: sourceId } }); } + 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); + } + } + + async onVisiteSupprimer(actor, tmr, onRemoveToken) { + let existants = actor.data.items.filter(it => this.isCase(it, tmr.coord)); + for (let casetmr of existants) { + await actor.deleteOwnedItem(casetmr._id); + onRemoveToken(tmr, casetmr); + } + } + } \ No newline at end of file diff --git a/module/tmr/fermeture-cites.js b/module/tmr/fermeture-cites.js index 6a8602d3..8d695ae6 100644 --- a/module/tmr/fermeture-cites.js +++ b/module/tmr/fermeture-cites.js @@ -35,11 +35,4 @@ export class FermetureCites extends Draconique { await this.createCaseTmr(actor, 'Fermeture: ' + tmr.label, tmr, souffle._id); } } - - async onConquete(actor, tmr, onRemoveToken) { - const citeFermee = actor.data.items.find(it => this.isCase(it, tmr.coord)); - await actor.deleteOwnedItem(citeFermee._id); - onRemoveToken(tmr, citeFermee); - } - } diff --git a/module/tmr/pelerinage.js b/module/tmr/pelerinage.js index 76cc2fe9..d9ba19c5 100644 --- a/module/tmr/pelerinage.js +++ b/module/tmr/pelerinage.js @@ -30,13 +30,4 @@ export class Pelerinage extends Draconique { decallage: tmrConstants.right }); } - - async onFinPelerinage(actor, tmr, onRemoveToken) { - const pelerinages = actor.data.items.filter(it => this.isCase(it, tmr.coord)); - for (let p of pelerinages){ - await actor.deleteOwnedItem(p.data.sourceid); - onRemoveToken(tmr, p); - } - } - }