ortegamarcel / TheWitcherTRPG

FoundryVTT The witcher trpg system
GNU General Public License v3.0
2 stars 1 forks source link

Fix Valuables and Diagrams on Loot Sheet #13

Open ortegamarcel opened 1 year ago

ortegamarcel commented 1 year ago

It seems that the Actor -> Item sheet no longer allows you to open Valuables or Diagrams, or add them to it?

Originally reported by Davrill here.

ortegamarcel commented 1 year ago

@Dravill I cannot find any problem. Everything works fine for me. If you still encounter that problem, please write a step-by-step guide on how to reproduce it and post the according error message from the console. (Press F12 and go to "Console".)

Davrill commented 1 year ago

Thank you for checking - I'll do some checking on it, and come back to you. Currently rolling back to version 10.303 and checking my backed-up files

Davrill commented 1 year ago

Okie, so I rolled back to the version we're using on the Forge (0.96 of AM's system), using the stable 10.303 version of VTT, and I can see the loot actor I've made: 096 Build_chest1 When I click the + button it adds a new one, as expected: 096 Build_chest2 I've then updated the Witcher System to the Alpha4 one, remaining on 10.303 build to minimise the issues, and when I open the loot chest Actor, I see the following: 097_Alpha4_Chest1 I can't see any of the previous items that I'd installed. If I export the Data of the Actor file - the file shows the items as being present, just not being rendered in the game. I then tried generating a new item for each of the tabs - with the result below: 097_Alpha4_Chest3 As you can see, it creates new files for the armours and components, but nothing for the Valuables or Diagrams - BUT is generating a 'Created item ID' in the Console. So it's creating them, but just not showing them.

When I then roll back to 0.96, the items all appear visible once more, along with the newly 'created item ID' on all the tabs, including the Valuables and Diagrams.

Davrill commented 1 year ago

Sorry for so many images. I also tried one more thing in the 0.96 build - making a new container actor item and dropping an item from the Valuable Compendium into it with the Console up - a number of error messages were shown? Not sure if they are relevant either: 096_error_msg

ortegamarcel commented 1 year ago

Oh, I see. You meant the loot actor... yes, I can reproduce/see the problem. I will fix it. (I am using Foundry v11.305)

It's good that you post so many images, makes it easier to understand.

Davrill commented 1 year ago

No worries, thank you for coming back to me, and I'm glad they are useful :)

ortegamarcel commented 1 year ago

I fixed it, and created a Pull Request. You can find screenshots and more details there: https://github.com/ortegamarcel/TheWitcherTRPG/pull/14

I will merge it, after some more testing and it will be available with the next release v0.97-alpha.6.

Melkkart commented 1 year ago

Hi, after seeing this bug, I started to look at the actor sheets and the glyph, rune, etc that are dragged to a player sheet do not appear, they are there, because the weight increases, but they are not visible. image