fvtt-module-lmrtfy icon indicating copy to clipboard operation
fvtt-module-lmrtfy copied to clipboard

Unable to roll initiative in PF2E

Open nguyenjessev opened this issue 1 year ago • 4 comments

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 https://github.com/League-of-Foundry-Developers/fvtt-module-lmrtfy/blob/024c88b65f1c92a3be691bac3fc94c84a5bd8a4b/src/roller.js#L352

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 https://github.com/League-of-Foundry-Developers/fvtt-module-lmrtfy/blob/024c88b65f1c92a3be691bac3fc94c84a5bd8a4b/src/roller.js#L351 https://github.com/League-of-Foundry-Developers/fvtt-module-lmrtfy/blob/024c88b65f1c92a3be691bac3fc94c84a5bd8a4b/src/roller.js#L356

nguyenjessev avatar May 14 '23 19:05 nguyenjessev

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 avatar Jul 30 '23 19:07 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

nethershaw avatar Jul 30 '23 19:07 nethershaw

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

Imper1um avatar Aug 18 '23 03:08 Imper1um

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

nethershaw avatar Dec 05 '23 19:12 nethershaw