Unable to roll initiative in PF2E

[nguyenjessev]

Description

When I request an initiative roll from a player, nothing happens on their end when they click "Initiative" in their popup window. Tried this on a fresh world where LMRTFY was the only active module

Steps to reproduce

Start a new world, set up a dummy user and actor, activate LMRTFY. Request an initiative roll from the dummy user/actor, then in another window try to click the "Initiative" button in the popup.

Expected behavior

Initiative is rolled

Actual behavior

In the console, there is an error on line 352 saying that skill is undefined

fvtt-module-lmrtfy/src/roller.js

Line 352 in 024c88b

rollNames.push(`${skill.ability}-based`);

Possibly related, I noticed that the initiative object does not have an ability property, just a statistic property. Not sure if it's related though because I tried changing line 351 but then got a new error on line 356 saying that initiative.roll is not a function

fvtt-module-lmrtfy/src/roller.js

Line 351 in 024c88b

const skill = actor.data.data.skills[initiative.ability];

fvtt-module-lmrtfy/src/roller.js

Line 356 in 024c88b

initiative.roll({ event, options });

[nethershaw]

Same -- but for all skills and abilities with the latest pf2e system on FVTT 11 Stable:

roller.js:287 Uncaught (in promise) TypeError: skill.roll is not a function
    at LMRTFYRoller._makeRoll (roller.js:287:35)
    at LMRTFYRoller._onSkillCheck (roller.js:541:18)
    at HTMLButtonElement.dispatch (jquery.min.js:2:43184)
    at y.handle (jquery.min.js:2:41168)

If you don't intend to support Pathfinder systems, say so. Broken support is worse than no support. I wasted too much time setting this up for my GM only to find that it has no chance of ever working in its current state.

[nethershaw]

This fork contains fixes -- only for pf2e -- that the author hasn't bothered to upstream. You might consider working together. Or communicating.

https://github.com/iBoMbY/fvtt-module-lmrtfy

[Imper1um]

I wasn't entirely sure how pf2e rolls initiative, but I did repair the functionality. The lmrtfy-pf2e module that was made is so far separated from this version now that merging the two would require a major amount of changes. I updated the Initiative check on PR #189

[nethershaw]

@Varriount @vtt-lair Maintain this repository or find someone else to do it.