#13

- Add setting to show equation for numerical changes.
Larkinabout · May 8, 2024 · bb72916 · bb72916
1 parent b54b22f
commit bb72916
Show file tree

Hide file tree

Showing 5 changed files with 54 additions and 4 deletions.
diff --git a/languages/en.json b/languages/en.json
@@ -17,6 +17,8 @@
     "changeLog.settings.playerPropertiesForm.name": "Changes for Player",
     "changeLog.settings.playerPropertiesForm.label": "Changes for Player",
     "changeLog.settings.playerPropertiesForm.hint": "Changes to log for the player",
+    "changeLog.settings.showEquation.name": "Show Equation",
+    "changeLog.settings.showEquation.hint": "For numerical changes, choose whether to show the equation on the chat card",
     "changeLog.settings.showRecipients.name": "Show Recipients",
     "changeLog.settings.showRecipients.hint": "Choose whether to show recipients on the chat card",
     "changeLog.settings.showSender.name": "Show Sender",

diff --git a/src/change-log.js b/src/change-log.js
@@ -99,6 +99,7 @@ export class ChangeLog {
             this.getGmActorTypes(),
             this.getGmProperties(),
             this.getPlayerProperties(),
+            this.getShowEquation(),
             this.getShowRecipients(),
             this.getShowSender()
         ])
@@ -124,6 +125,10 @@ export class ChangeLog {
         this.playerProperties = await Utils.getSetting('playerProperties')?.split(DELIMITER) ?? []
     }
 
+    async getShowEquation () {
+        this.showEquation = await Utils.getSetting('showEquation')
+    }
+
     async getShowRecipients () {
         this.showRecipients = await Utils.getSetting('showRecipients')
     }
@@ -199,6 +204,10 @@ export class ChangeLog {
 
             if (!isEveryone && !isGm && !isPlayer) continue
 
+            const oldValue = Utils.getValueByDotNotation(doc, key)
+            const newValue = Utils.getValueByDotNotation(data, key)
+            const { sign, adjustmentValue } = this.#getAdjustment(oldValue, newValue)
+
             const templateData = {
                 tokenId: token?.id || null,
                 actorId: actor?.id || null,
@@ -208,8 +217,10 @@ export class ChangeLog {
                 documentType,
                 key,
                 modifiedByName,
-                oldValue: Utils.getValueByDotNotation(doc, key),
-                newValue: Utils.getValueByDotNotation(data, key)
+                oldValue,
+                newValue,
+                sign,
+                adjustmentValue
             }
 
             const whisperData = {
@@ -249,6 +260,8 @@ export class ChangeLog {
 
             if (!isEveryone && !isGm && !isPlayer) continue
 
+            const { sign, adjustmentValue } = this.#getAdjustment(oldValue, newValue)
+
             const templateData = {
                 tokenId: token?.id || null,
                 actorId: actor?.id || null,
@@ -259,7 +272,9 @@ export class ChangeLog {
                 key,
                 modifiedByName,
                 oldValue,
-                newValue
+                newValue,
+                sign,
+                adjustmentValue
             }
 
             const whisperData = {
@@ -309,6 +324,14 @@ export class ChangeLog {
         this.#createChatMessage('delete', templateData, whisperData)
     }
 
+    #getAdjustment (oldValue, newValue) {
+        if (!this.showEquation) return null
+        const difference = (Number.isInteger(oldValue) && Number.isInteger(newValue)) ? oldValue - newValue : null
+        const sign = (Number.isInteger(difference)) ? (difference < 0) ? 'fa-plus' : 'fa-minus' : null
+        const adjustmentValue = (Number.isInteger(difference)) ? Math.abs(difference) : null
+        return { sign, adjustmentValue }
+    }
+
     #getAudience (documentType, actorType, key) {
         return {
             isEveryone: (this.everyoneActorTypes.includes(actorType) && this.everyoneProperties.includes(`${documentType}.${key}`)),
@@ -334,7 +357,10 @@ export class ChangeLog {
     }
 
     async #createChatMessage (changeType, templateData, whisperData) {
-        const { tokenId, actorId, documentId, document1Name, document2Name, documentType, key, oldValue, newValue, modifiedByName } = templateData
+        const {
+            tokenId, actorId, documentId, document1Name, document2Name, documentType,
+            key, oldValue, newValue, sign, adjustmentValue, modifiedByName
+        } = templateData
         const { actor, isEveryone, isGm, isPlayer } = whisperData
 
         if (!this.#isValidChange({ oldValue, newValue })) return
@@ -347,6 +373,8 @@ export class ChangeLog {
                 propertyName: Utils.getPropertyName(`${documentType}.${key}`),
                 oldValue: Utils.getPropertyValue(oldValue),
                 newValue: Utils.getPropertyValue(newValue),
+                sign: sign || 'fa-arrow-right',
+                adjustmentValue,
                 tooltip: `<div>${game.i18n.localize('changeLog.modifiedBy')}: ${modifiedByName}</div>`
             }
         )

diff --git a/src/settings.js b/src/settings.js
@@ -52,6 +52,16 @@ export const registerSettings = function () {
         scope: 'world'
     })
 
+    game.settings.register(MODULE.ID, 'showEquation', {
+        name: game.i18n.localize('changeLog.settings.showEquation.name'),
+        hint: game.i18n.localize('changeLog.settings.showEquation.hint'),
+        scope: 'world',
+        config: true,
+        type: Boolean,
+        default: false,
+        onChange: () => { game.changeLog.getShowEquation() }
+    })
+
     game.settings.register(MODULE.ID, 'showSender', {
         name: game.i18n.localize('changeLog.settings.showSender.name'),
         hint: game.i18n.localize('changeLog.settings.showSender.hint'),

diff --git a/styles/change-log.css b/styles/change-log.css
@@ -46,6 +46,12 @@
     justify-content: center;
 }
 
+.change-log-card-adjustment-value {
+    display: flex;
+    flex: 1;
+    justify-content: center;
+}
+
 .change-log-card-new-value {
     display: flex;
     flex: 1;

diff --git a/templates/chat-card.hbs b/templates/chat-card.hbs
@@ -5,6 +5,10 @@
     <div class="change-log-card-values">
         {{#if oldValue includeZero=true}}
         <div class="change-log-card-old-value">{{{oldValue}}}</div>
+        <i class="fa {{sign}}"></i>
+        {{/if}}
+        {{#if adjustmentValue includeZero=true}}
+        <div class="change-log-card-adjustment-value">{{{adjustmentValue}}}</div>
         <i class="fa fa-arrow-right"></i>
         {{/if}}
         {{#if newValue includeZero=true}}<div class="change-log-card-new-value">{{{newValue}}}</div>{{/if}}