Skip to content

Conversation

@TheFaser
Copy link
Contributor

Fix #1377

@booky10
Copy link
Collaborator

booky10 commented Oct 27, 2025

Did you test this actually changes something? If working correctly, return this.propertyList.decode(nbt, wrapper); (see immediately below your changes) should also handle nbt lists

@TheFaser
Copy link
Contributor Author

Did you test this actually changes something? If working correctly, return this.propertyList.decode(nbt, wrapper); (see immediately below your changes) should also handle nbt lists

Yes, I've tested this on Fabric. I also thought that the lists should be processed already, but it doesn't happen on Fabric

@TheFaser TheFaser changed the title add support for decoding item profile properties from NBT lists simplify NBT list conversion logic Oct 30, 2025
@TheFaser
Copy link
Contributor Author

I tested it on Fabric 1.21.10 and on Paper 1.21.9, it works without errors

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] [FABRIC 1.21.9+] Incorrect reading of player_head

2 participants