jopeek / fvtt-loot-sheet-npc-5e

FVTT - Loot Sheet NPC 5E
MIT License
44 stars 86 forks source link

Clicking "Make Lootable" while having multiple monster and hero tokens corrupts player character sheets #381

Open sbutwina opened 1 year ago

sbutwina commented 1 year ago

Describe the bug When selecting multiple tokens that contain both monster actors and player characters, the "Make Lootable" button when pressed on the monster also converts the player character to a non-functioning, lootable token. This then corrupts the players character sheet.

To Reproduce Steps to reproduce the behavior:

  1. Create a scene with one or more monsters and at least one PC
  2. Select all tokens
  3. Right-click on one of the monster tokens
  4. Click "Make Lootable" Notice that all tokens are shown as lootable, including the player token
  5. Try double-clicking on the user token Notice that the character sheet does not load
  6. Delete the character token and re-add from the Actors tab
  7. Double-click the token to open the character sheet Notice that class, levels, features, etc. are empty

At this point, the player character needs to be recreated. There is no way I have been able to undo the change and recover the character

Expected behavior Character tokens should not be made lootable.

Desktop (please complete the following information):

Additional context Note that, when this happened to me, it was solely accidental I still had PCs selected. But this led to a destructive mistake that required me to pause the session while I did another D&D Beyond pull and update items on the character sheets