token-mold icon indicating copy to clipboard operation
token-mold copied to clipboard

[FEATURE] Update for V12 Stable Update

Open wonsnot opened this issue 1 year ago • 31 comments

Is your feature request related to a problem? Please describe. Foundry has updated to V12 stable and this module is listed as incompatible.

Describe the solution you'd like Update and test on Foundry V12

wonsnot avatar Jun 14 '24 01:06 wonsnot

I updated to version 12 too and need and update on this module

GomatiGit avatar Jun 14 '24 15:06 GomatiGit

+1

Deladriendil avatar Jun 15 '24 00:06 Deladriendil

everytime you forget how useful a module is, until you update and find that it is incompatible and you suddenly miss the features you took for granted. +1 for v12 update please! (automatic HP rolling and adjactive on mobs is/was a stable for our games, it is a tough life without this!)

Someone2345 avatar Jun 17 '24 14:06 Someone2345

That's why you ideally don't upgrade you (production) foundry until your favorite modules are updated - or look into contributing to the module to get it ready faster if you have the skills to do so ;)

ThiefMaster avatar Jun 17 '24 14:06 ThiefMaster

+1

kajishun avatar Jun 18 '24 03:06 kajishun

I'll try to address v12 in my fork once I have a v12 server running.

BeardedGnome avatar Jun 21 '24 01:06 BeardedGnome

V12 update - Fixed many of the warnings, but Foundry changed how dice are rolled and broke rolled health. Still investigating.

BeardedGnome avatar Jun 23 '24 05:06 BeardedGnome

Love token mold, Hoping to get it working in v12 as well. Another vote!

BenNelsonNeb avatar Jun 30 '24 15:06 BenNelsonNeb

I've rewritten the HP generation and will attempt to have something published (unofficial fork) by the end of the month.

BeardedGnome avatar Jul 03 '24 21:07 BeardedGnome

I've published an unofficial fork that should fix the issue, please test the version that matches your Foundry install.

Foundry v11: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.21.0/module.json Foundry v12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.1/module.json

BeardedGnome avatar Jul 08 '24 05:07 BeardedGnome

I've published an unofficial fork that should fix the issue, please test the version that matches your Foundry install.

Foundry v11: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.21.0/module.json Foundry v12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.1/module.json

how does one use this exactly, I'm new to all this.

tobjv avatar Jul 23 '24 19:07 tobjv

I've published an unofficial fork that should fix the issue, please test the version that matches your Foundry install. Foundry v11: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.21.0/module.json Foundry v12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.1/module.json

how does one use this exactly, I'm new to all this.

This article explains how to install a new module by manifest URL. https://foundryvtt.com/article/modules/

wonsnot avatar Jul 23 '24 23:07 wonsnot

I've published an unofficial fork that should fix the issue, please test the version that matches your Foundry install.

Foundry v11: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.21.0/module.json Foundry v12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.1/module.json

Thanks! It seems to be working for me. I'll raise an issue over there if I run into any.

wonsnot avatar Jul 23 '24 23:07 wonsnot

works for me so far as well

tobjv avatar Jul 24 '24 13:07 tobjv

Foundry v12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.1/module.json

This loaded and worked for me. Thanks.

kiloforce avatar Aug 05 '24 06:08 kiloforce

With the v12 fork I get an error in a pf2e world when trying to open the Token Mold config dialog.

It's coming from this code:

const types = Actor.implementation.TYPES;
// FIXME: DAE throws on 'base' type
const shellMap = new Map( types.map((t) => [t, new Actor.implementation({ name: t, type: t })]), );
Error: An error occurred while rendering TokenMoldForm 27. PF2e System | Actor type base does not exist and actor module sub-types are not supported
    onError .../scripts/foundry.js:654
    render .../scripts/foundry.js:5794
    promise callback*render .../scripts/foundry.js:5792
    _renderActorDirectoryMenu .../modules/token-mold/scripts/token-mold.js:297
    _renderActorDirectoryMenu .../modules/token-mold/scripts/token-mold.js:291
    _hookActorDirectory .../modules/token-mold/scripts/token-mold.js:243
    initHooks .../modules/token-mold/scripts/token-mold.js:57
...
Caused by: Error: PF2e System | Actor type base does not exist and actor module sub-types are not supported
    ErrorPF2e .../systems/pf2e/pf2e.mjs:1
    construct .../systems/pf2e/pf2e.mjs:1608
    get _actorAttributes/shellMap< .../modules/token-mold/scripts/token-mold.js:1388
    get _actorAttributes .../modules/token-mold/scripts/token-mold.js:1388
    getData .../modules/token-mold/scripts/token-mold.js:1179
    _render .../scripts/foundry.js:5837
    _render .../scripts/foundry.js:6571
    render .../scripts/foundry.js:5792
    _renderActorDirectoryMenu .../modules/token-mold/scripts/token-mold.js:297
    _renderActorDirectoryMenu .../modules/token-mold/scripts/token-mold.js:291
    _hookActorDirectory .../modules/token-mold/scripts/token-mold.js:243
    initHooks .../modules/token-mold/scripts/token-mold.js:57
...

ThiefMaster avatar Aug 05 '24 19:08 ThiefMaster

const types = Actor.implementation.TYPES.filter(x => x !== 'base');

Seems to be a good workaround for this error. Not sure if this breaks any functionality for the stat overlay, but when not using it it's certainly better than not being able to configure the module at all and at least during a quick test it seemed to offer all relevant attributes.

ThiefMaster avatar Aug 05 '24 20:08 ThiefMaster

Foundry v12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.1/module.json

This 90% works for me. When I bring in new tokens that don't have vision already on them, its not overriding their vision to be on, although I have it selected. I'm having to go back and indivitually check every token to make sure vision is on. Beyond that, its working well.

DanBonser avatar Aug 05 '24 23:08 DanBonser

IGNORE Added it and it works great. however, now i have no roll table of adjectives. is there a way to just load those?

Its there now, weird

mattimeo84 avatar Aug 06 '24 00:08 mattimeo84

const types = Actor.implementation.TYPES.filter(x => x !== 'base');

Seems to be a good workaround for this error. Not sure if this breaks any functionality for the stat overlay, but when not using it it's certainly better than not being able to configure the module at all and at least during a quick test it seemed to offer all relevant attributes.

You are correct. I need to filter out the 'base' type in a couple places in the code.

BeardedGnome avatar Aug 06 '24 05:08 BeardedGnome

Foundry v12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.1/module.json

This 90% works for me. When I bring in new tokens that don't have vision already on them, its not overriding their vision to be on, although I have it selected. I'm having to go back and indivitually check every token to make sure vision is on. Beyond that, its working well.

I have no idea how vision works. I'll see if I can reproduce the issue. No ETA at this time.

BeardedGnome avatar Aug 06 '24 05:08 BeardedGnome

I've published an unofficial fork that should fix the issue, please test the version that matches your Foundry install.

Foundry v11: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.21.0/module.json Foundry v12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.1/module.json

I really appreciate you doing this and it's made the module function for me. The one hangup that I've run into is that I can not get the settings to show up when I hit the gear icon in the actors tab. Could be a me problem, though.

Mrfikser avatar Aug 08 '24 19:08 Mrfikser

No, it's a bug that's not been fixed yet. See my previous comment in here for a workaround (you can edit the installed module for this until there's a fixed release).

ThiefMaster avatar Aug 08 '24 19:08 ThiefMaster

I'm the type that will wait for a fix. Thanks.

Mrfikser avatar Aug 08 '24 19:08 Mrfikser

@ThiefMaster - Try this release.

V12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.2/module.json

BeardedGnome avatar Aug 09 '24 08:08 BeardedGnome

Thanks, works fine AFAICT.

ThiefMaster avatar Aug 09 '24 13:08 ThiefMaster

@ThiefMaster - Try this release.

V12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.2/module.json

I don't suppose there's a way to make this the official version in Foundry is there? I'm not asking you to take anything from the original author. It looks like it has been over a year since they've pushed an update out and it would be nice for those of us less inclined technical if something like that happened.

Mrfikser avatar Aug 26 '24 20:08 Mrfikser

@ThiefMaster - Try this release. V12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.2/module.json

I don't suppose there's a way to make this the official version in Foundry is there? I'm not asking you to take anything from the original author. It looks like it has been over a year since they've pushed an update out and it would be nice for those of us less inclined technical if something like that happened.

I don't have any way of doing that. If one of the maintainers wants to review my pull requests, we can get master updated.

BeardedGnome avatar Aug 26 '24 21:08 BeardedGnome

@ThiefMaster - Try this release. V12: https://raw.githubusercontent.com/BeardedGnome/token-mold/v2.22.2/module.json

I don't suppose there's a way to make this the official version in Foundry is there? I'm not asking you to take anything from the original author. It looks like it has been over a year since they've pushed an update out and it would be nice for those of us less inclined technical if something like that happened.

I don't have any way of doing that. If one of the maintainers wants to review my pull requests, we can get master updated.

I hope there's a way to make this happen. I'm using what you've given us, but it bugs me that it can't be updated elsewhere. Thanks for making this amazing mod continue working as Foundry updates.

Mrfikser avatar Aug 26 '24 22:08 Mrfikser

I don't suppose there's a way to make this the official version in Foundry is there? I'm not asking you to take anything from the original author. It looks like it has been over a year since they've pushed an update out and it would be nice for those of us less inclined technical if something like that happened.

Fortunately, it was listed under the MIT license (https://github.com/Moerill/token-mold/blob/master/LICENSE).

An abandoned project is left with really two options:

  1. One could fork the project, create a new divergent project, and submit the divergent project to be listed on Foundry. This would be a good option if significant improvements were planned.

  2. Alternatively, if only minor changes are needed, one could fork the project and make a case to the Foundry Project to update the listing and point to a new source link.

Foundry contact information can be found on the Module Packaging Guide: https://foundryvtt.com/article/packaging-guide/

kiloforce avatar Aug 27 '24 03:08 kiloforce