stefouch / foundry-year-zero-roller Goto Github PK

The way the YZEDiceroller stores results in the roll.terms DsN will pick up the already rolled dice AND the dice that are rerolled for a grand total of more dice than you started with.

Courtesy of @aMediocreDad, here is a temporary fix:

Hooks.on("diceSoNiceRollStart", async (_id, context) => {
    if (context.roll.pushed !== 0 && !context.roll.pushable)
        context.roll.terms.forEach((term) => {
            if (term && typeof term === "object") {
                term.results = term.results?.filter((r) => r.active === true && r.indexPush > 0);
            }
        });
});

Hiya... Just a courtesy call to mention that @typhonjs-fvtt/eslint-foundry.js has been deprecated and the new NPM module is @typhonjs-fvtt/eslint-config-foundry.js. The reason for the deprecation is that to enable the module as a shareable ESLint config required a name change for ESLint naming conventions.

The new module has updated support for 0.7.8 to 0.8.0 Foundry VTT releases and also now features a difference set / exclusion set config of global variables dropped / now undeclared between the 0.7.9 to 0.8.0 versions to use for migration.

Happy coding and good luck with your module / system & Foundry VTT migration to 0.8.x and beyond!