Convert add coins dialog to appv2

Author: CarlosFdez

src/module/actor/sheet/base.ts

@@ -69,7 +69,7 @@ import type {
 } from "./data-types.ts";
 import { createBulkPerLabel, onClickCreateSpell } from "./helpers.ts";
 import { ItemSummaryRenderer } from "./item-summary-renderer.ts";
-import { AddCoinsPopup } from "./popups/add-coins-popup.ts";
+import { AddCurrencyDialog } from "./popups/add-currency-dialog.ts";
 import { IdentifyItemPopup } from "./popups/identify-popup.ts";
 import { ItemTransferDialog } from "./popups/item-transfer-dialog.ts";
 import { IWREditor } from "./popups/iwr-editor.ts";
@@ -707,7 +707,7 @@ abstract class ActorSheetPF2e<TActor extends ActorPF2e> extends fav1.sheets.Acto
             },
             // INVENTORY
             "add-coins": () => {
-                return new AddCoinsPopup(this.actor).render(true);
+                return AddCurrencyDialog.wait({ actor: this.actor });
             },
             "decrease-quantity": async (event) => {
                 const item = await inventoryItemFromDOM(event);

src/module/actor/sheet/popups/add-coins-popup.ts

@@ -0,0 +1,61 @@
+import { ActorPF2e } from "@actor/base.ts";
+import { DENOMINATIONS } from "@item/physical/values.ts";
+import { ErrorPF2e } from "@util";
+import fields = foundry.data.fields;
+
+/** Simple dialog to add coins of various denominations to an actor */
+export class AddCurrencyDialog extends fa.api.DialogV2<AddCurrencyConfiguration> {
+    static override DEFAULT_OPTIONS: DeepPartial<AddCurrencyConfiguration> = {
+        id: "add-coins",
+        window: {
+            icon: "fa-solid fa-coins",
+            title: "PF2E.AddCoinsTitle",
+        },
+    };
+
+    static override async wait(options: WaitParams): Promise<unknown> {
+        const actor = options.actor;
+        const content = document.createElement("div");
+        const templatePath = "systems/pf2e/templates/actors/add-currency-dialog.hbs";
+
+        const denominations = DENOMINATIONS.map((d) => ({
+            name: d,
+            field: new fields.NumberField({
+                min: 0,
+                integer: true,
+                label: game.i18n.localize(CONFIG.PF2E.currencies[d]),
+            }),
+        }));
+
+        content.innerHTML = await fa.handlebars.renderTemplate(templatePath, { actor, denominations });
+        const buttons: fa.api.DialogV2Button[] = [
+            {
+                type: "button",
+                icon: "fa-regular fa-floppy-disk",
+                label: "PF2E.AddCoinsTitle",
+                action: "save",
+                callback: (_, button) => {
+                    const form = button.closest("form");
+                    if (!form) throw ErrorPF2e("Unexpected missing form");
+                    const formData = new fa.ux.FormDataExtended(form).object;
+                    const combineStacks = !!formData.combineStacks;
+                    const coins = {
+                        pp: Number(formData.pp) || 0,
+                        gp: Number(formData.gp) || 0,
+                        sp: Number(formData.sp) || 0,
+                        cp: Number(formData.cp) || 0,
+                    };
+                    return actor.inventory.addCoins(coins, { combineStacks });
+                },
+            },
+        ];
+
+        return super.wait(Object.assign(options, { content, buttons }));
+    }
+}
+
+interface AddCurrencyConfiguration extends fa.api.DialogV2Configuration {
+    actor: ActorPF2e;
+}
+
+type WaitParams = DeepPartial<fa.api.DialogV2Configuration> & { actor: ActorPF2e };

src/module/actor/sheet/popups/add-currency-dialog.ts

@@ -0,0 +1,7 @@
+{{#each denominations as |denomination|}}
+    {{formGroup denomination.field name=denomination.name}}
+{{/each}}
+<div class="form-group">
+    <label for="{{actor.uuid}}-add-coins-combine-stacks">{{localize "PF2E.AddCoinsCombineStacksLabel"}}</label>
+    <input id="{{actor.uuid}}-add-coins-combine-stacks" type="checkbox" name="combineStacks" checked />
+</div>