Text edit for Rally implementation in Bard Class (#1760)

module/data/chat-message/actorRoll.mjs

@@ -40,7 +40,6 @@ export default class DHActorRoll extends foundry.abstract.TypeDataModel {
             hasSave: new fields.BooleanField({ initial: false }),
             hasTarget: new fields.BooleanField({ initial: false }),
             isDirect: new fields.BooleanField({ initial: false }),
-            isCritical: new fields.BooleanField({ initial: false }),
             onSave: new fields.StringField(),
             source: new fields.SchemaField({
                 actor: new fields.StringField(),

module/data/fields/action/damageField.mjs

@@ -49,11 +49,14 @@ export default class DamageField extends fields.SchemaField {
 
         formulas = DamageField.formatFormulas.call(this, formulas, config);
 
+        messageId = config.message?._id ?? messageId;
+        const message = game.messages.get(messageId);
         const damageConfig = {
             dialog: {},
             ...config,
             roll: formulas,
-            data: this.getRollData()
+            data: this.getRollData(),
+            isCritical: message?.system.roll.isCritical
         };
         delete damageConfig.evaluate;
 
@@ -61,7 +64,7 @@ export default class DamageField extends fields.SchemaField {
             damageConfig.dialog.configure = false;
         if (config.hasSave) config.onSave = damageConfig.onSave = this.save.damageMod;
 
-        damageConfig.source.message = config.message?._id ?? messageId;
+        damageConfig.source.message = messageId;
         damageConfig.directDamage = !!damageConfig.source?.message;
 
         // if(damageConfig.source?.message && game.modules.get('dice-so-nice')?.active)

module/dice/d20Roll.mjs

@@ -207,7 +207,7 @@ export default class D20Roll extends DHRoll {
                 rerolls: dice.results.filter(x => x.rerolled)
             }
         }));
-        data.isCritical = config.isCritical = roll.isCritical;
+        data.isCritical = roll.isCritical;
         data.extra = roll.dice
             .filter(d => !roll.baseTerms.includes(d))
             .map(d => {

module/dice/damageRoll.mjs

@@ -138,16 +138,9 @@ export default class DamageRoll extends DHRoll {
     }
 
     constructFormula(config) {
-        this.options.roll.forEach((part, index) => {
+        for (const [index, part] of this.options.roll.entries()) {
             part.roll = new Roll(Roll.replaceFormulaData(part.formula, config.data));
-            this.constructFormulaPart(config, part, index);
-        });
-        return this.options.roll;
-    }
-
-    constructFormulaPart(config, part, index) {
             part.roll.terms = Roll.parse(part.roll.formula, config.data);
-
             if (part.applyTo === CONFIG.DH.GENERAL.healingTypes.hitPoints.id) {
                 part.modifiers = this.applyBaseBonus(part);
                 this.addModifiers(part);
@@ -186,7 +179,9 @@ export default class DamageRoll extends DHRoll {
                 }
             }
 
-        return (part.roll._formula = this.constructor.getFormula(part.roll.terms));
+            part.roll._formula = this.constructor.getFormula(part.roll.terms);
+        }
+        return this.options.roll;
     }
 
     /* To Remove When Reaction System */

src/packs/classes/class_Bard_vegl3bFOq3pcFTWT.json

@@ -4,7 +4,7 @@
   "type": "class",
   "img": "icons/tools/instruments/harp-red.webp",
   "system": {
-    "description": "<p><strong>Note: At level 5 use Rally (Level 5) instead of Rally under class feature. (Automation will be implemented in a later release)</strong></p><p><br />Bards are the most charismatic people in all the realms. Members of this class are masters of captivation and specialize in a variety of performance types, including singing, playing musical instruments, weaving tales, or telling jokes. Whether performing for an audience or speaking to an individual, bards thrive in social situations. Members of this profession bond and train at schools or guilds, but a current of egotism runs through those of the bardic persuasion. While they may be the most likely class to bring people together, a bard of ill temper can just as easily tear a party apart.</p>",
+    "description": "<p>Bards are the most charismatic people in all the realms. Members of this class are masters of captivation and specialize in a variety of performance types, including singing, playing musical instruments, weaving tales, or telling jokes. Whether performing for an audience or speaking to an individual, bards thrive in social situations. Members of this profession bond and train at schools or guilds, but a current of egotism runs through those of the bardic persuasion. While they may be the most likely class to bring people together, a bard of ill temper can just as easily tear a party apart.</p>",
     "domains": [
       "grace",
       "codex"