cs96and / FoundryVTT-damage-log

FoundryVTT module to report all damage and healing to a separate chat tab
MIT License
9 stars 9 forks source link

BUG: Error Report gets thrown out in console when this module is combined with "Wild Magic Surge 5e" #16

Closed GambetTV closed 2 years ago

GambetTV commented 2 years ago

Describe the bug An error report is being thrown out when the two modules are used together. I'll be reporting on this to the Wild Magic Surge 5e module too.

What version of FoundryVTT are you using? v9 - Most recent

To Reproduce Steps to reproduce the behavior:

  1. Activate this module and Wild Magic Surge 5e.
  2. Hit F12 to open console.
  3. Deal damage to a token, watch the console report pop up.

Uncaught (in promise) Error: Syntax error, unrecognized expression: HP: 58 -> 57 [Detected 1 package: wild-magic-surge-5e] at Function.se.error (jquery.min.js:2) at se.tokenize (jquery.min.js:2) at se.select (jquery.min.js:2) at Function.se [as find] (jquery.min.js:2) at S.fn.init.find (jquery.min.js:2) at new S.fn.init (jquery.min.js:2) at S (jquery.min.js:2) at SpellParser.RollContent (SpellParser.js:19) at SpellParser.SpellDetails (SpellParser.js:34) at SpellParser.IsSpell (SpellParser.js:78) at MagicSurgeCheck.isValid (MagicSurgeCheck.js:72) at MagicSurgeCheck.Check (MagicSurgeCheck.js:38) at module.js:346 at Function._call (foundry.js:294) at Function.callAll (foundry.js:253) at ClientDatabaseBackend.callback (foundry.js:10067) at foundry.js:10006 at Array.map () at ClientDatabaseBackend._handleCreateDocuments (foundry.js:10006) at ClientDatabaseBackend._createDocuments (foundry.js:9903) at async Function.createDocuments (document.mjs:338) at async Function.create (document.mjs:439)

cs96and commented 2 years ago

Link to Wild Magic issue https://github.com/johnnolan/wild-magic-surge-5e/issues/58

cs96and commented 2 years ago

Looking at the call stack, this appears to be an issue with the Wild Magic module.

cs96and commented 2 years ago

Fixed in Wild Magic Surge module.