HimbeersaftLP / LibSkin

Virion for working with player skins & PocketMine plugin for stealing other player's skins
GNU Lesser General Public License v3.0
27 stars 8 forks source link

Bug with save skins with skins from Markedplace #6

Closed galaxytwenty closed 1 year ago

galaxytwenty commented 1 year ago

For example the wandering trader!

This is the skin it looks when im joining in to the server:

skinOnJoin

This is the skin it looks when im runnin /skin save and then /skin load: SkinOnLoadSkin

And this is the saved skin texture from plugin_data: Razor8I9I

This is only with some skins from markedplace, and not with downloaded skins.

galaxytwenty commented 1 year ago

Screenshot from this content from markedplace: marketPlace

HimbeersaftLP commented 1 year ago

Hmmm, I think this might be due to the geometry data?

galaxytwenty commented 1 year ago

I dont think so, if i save the geo and set the geo when load skin ... its exactly the same..

aaand he save the skin wrong in .png

HimbeersaftLP commented 1 year ago

can you send me the player.dat file so i can look at the raw skin data?

HimbeersaftLP commented 1 year ago

(the player.dat file before overwriting with the wrong skin using my library ofc)

galaxytwenty commented 1 year ago

Sure i can send when im home,You mean the Player.dat after joining with the skin, before i run save and load , right?

galaxytwenty commented 1 year ago

can i send via DC?

HimbeersaftLP commented 1 year ago

yes exactly and I think you can upload it right here on github

galaxytwenty commented 1 year ago

sure, i can try uploading here.

galaxytwenty commented 1 year ago

must ziping the .dat... GH say we dont support that file type

galaxytwenty commented 1 year ago

glx20.zip

galaxytwenty commented 1 year ago

there is the player.dat directly after joinen with the wandering trader skin

HimbeersaftLP commented 1 year ago

Thanks, I'll take a look at it

galaxytwenty commented 1 year ago

np, hopefully it can be fixed :)

galaxytwenty commented 1 year ago

have you found any points in these file ?

HimbeersaftLP commented 1 year ago

Sorry I didn't have time, yet

HimbeersaftLP commented 1 year ago

Hello, I found the problem.

You will need to save and restore both the geometry and the geometry name.

As an example, I updated the example plugin like this to include that change: https://github.com/HimbeersaftLP/LibSkin/commit/4f1a9369fbd737b09e2a9e88d981cad03916836e#diff-fdfadedddd0afa2f4f19afd620913f573521b6d7ed08a168b8bcd376b2b82594

HimbeersaftLP commented 1 year ago

@galaxytwenty Please let my know if this fix works for you as well

galaxytwenty commented 1 year ago

Of course, i testing this when im home.I let you know. Thanks

galaxytwenty commented 1 year ago

Thats work´s fine now. Best Thanks :)

HimbeersaftLP commented 1 year ago

You're welcome!

HimbeersaftLP commented 1 year ago

Thanks for testing it

galaxytwenty commented 1 year ago

Thanks for i can use it 👍