FoundryVTT-BetterRolls5e

FoundryVTT-BetterRolls5e copied to clipboard

Better rolls doesn't currently work with v9. When I try to roll an attack from a sheet I get the following error...

Dice roll evaluation failed: roll.dice is undefined.

I mean, V9 isn't really actually out yet (I haven't even installed it yet, I'd need to maintain a separate installation). That said something to keep in mind to see if there's a way to get dice info in a backwards compatible way.

All of the .roll() and .reroll() calls must be changed to .roll({async:false}) and .reroll({async:false}) respectively Tested on v9 and appears to be working. You may be looking to take advantage of async rolls, in which case this isn't the way to go of course, but it's a quick fix to get this particular part of v9 working.

This should be backwards compatible, since any older versions should ignore the argument.

As BlitzKraig mentioned, it has to do with the async default of foundry as of V9. You can indeed pass async:false to all .roll, .reroll and .evaluate calls in order to "fix" it. It is somewhat of a quick fix to get the module working at least. I've attached a fixed zip as a workaround. This works for me.

betterrolls5e.zip - async:false workaround

says core version is set to V9, is this issue fixed?

Should be unless there's a bug, aka a use case I didn't cover.