daft-develop / mob-attack-tool

A module for Foundry VTT that offers a tool for handling mob attacks in the dnd5e system.
GNU General Public License v3.0
6 stars 6 forks source link

[BUG] 5e 3.x chat card enrichment breaks attack chat cards #6

Closed Shambles1337 closed 4 months ago

Shambles1337 commented 4 months ago

Hi there, posting my screenshots from the issue I raised over at https://github.com/Stendarpaval/mob-attack-tool/issues/61#issuecomment-2064661559.

Here are my settings (Same as yours far as I can tell) Screenshot 2024-04-18 212101

The tool appears and I can increase/decrease the number in the mob and verify the target, as well as make the attack with advantage but when I click the "Mob Attack" button, the attack never resolves. Screenshot 2024-04-18 212651

Pasting a screenshot of my chat too, but it just shows the initiative no attacks. Screenshot 2024-04-18 212512

daft-develop commented 4 months ago

Thanks for adding the screenshots. I agree it looks like all your settings are the same, but slightly different buggy behaviour. I see from the screen shot you're using Command vs Control for disadvantage, assuming a Mac? I'm testing on Edge, Chrome, and Firefox for Windows and none are exactly like you're seeing. Despite that, I suspect you're seeing whatever bug I see as well, just showing up a little different.

For completeness, can you confirm you're running D&D 5e 3.1.2 and Mob Attack 0.5.0?

If you've got some time, can you try opening a developer console on your browser (it may take a few steps in Safari) and see if it reports any errors when you try to make a Mob attack? You might get some warnings about data.data being depreciated, but I'm expecting some sort of error that mentions '_formula'. image

If you're getting a different kind of error, it may be something unrelated. Either way, I'll let you know once I've posted a new release to fix the issue I see, and we can test if that resolves your issue at the same time.

Shambles1337 commented 4 months ago

So to confirm I am running D&D 5e 3.1.2 and Mob Attack 0.5.0. Also, I'm not on a Mac, it's a windows PC, but I think the reason it looks different is because I'm not actually using the browser, I am running the game directly using the Foundry VTT client. I did test in Chrome, Edge, and Firefox and it was not working in any browser though.

Also, I have an update, I was able to get mob attack tool to work (I think mostly). I noticed when I went back to my original world, that I'm running my campaign in, another mod wasn't working despite it being the most up to date version. What I did was Uninstall all my mods, then uninstalled the Foundry Client, and then installed an earlier version (Version 9, which my buddy has installed and is working) as well as an older version of D&D 5e (I didn't think to write down the version) reinstalled mob attack tool (version 0.3.13). I confirmed it was working. I installed the other mod that wasn't working as well (Dice so Nice) and then I uninstalled V9 of Foundry VTT and installed the newest version of 5e, updated both mods, and installed the newest manifest for Mob attack tool (0.5.0) and Dice So Nice and both are now working (mostly, see below).

I am working on slowly re-introducing my other previously installed mods one by one and testing everything to ensure I don't break anything in the process.

I am not able to see the attack rolls now for some reason, but it resolves the attacks now, so the mod is at least usable. Screenshot 2024-04-19 130203

Screenshot of the Developer Mode image

Second Screenshot from Developer Mode (Both from Firefox) image

Hope this is helpful. I realize I have another mod added in when I took these screenshots, if that's an issue I can take other screenshot without the mod in and resend.

daft-develop commented 4 months ago

Thank you for the update. Not sure why the whole re-install was necessary either, but now it sounds like your error is identical to the one I see. Some of the changes in the 5e system from 3.x onward try to "enrich" the chat cards for dice rolls, but the way MAT currently generates the attack card conflicts with some of the system assumptions. I'll have to rework how the attack cards are generated to resolve it, but it is reproducible so it should be something I can resolve relatively quickly.

daft-develop commented 4 months ago

@Shambles1337 I've just pushed a new release where I think the issue has been fixed. I did a quick test on Foundry v10, v11 with 5e 2.x, and v11 with 5e 3.1.2 and all are showing the attack cards correctly. Also tested with Dice So Nice and MidiQoL very briefly and it appears to be working with both as well. Please give it a test and let me know if you find any other issues related to this (add as a note) or any new bug I've introduced (please feel free to create a new ticket.)

Shambles1337 commented 4 months ago

The new version works beautifully! Thank you so much!