ShyNieke / StatuesMod

A little decorative and functional mod
https://www.curseforge.com/minecraft/mc-mods/statues
Other
8 stars 5 forks source link

[Bug]: Unable to register name on small statue character #64

Closed lSherozl closed 2 years ago

lSherozl commented 2 years ago

General Info

Forge version

36.2.28

Mod version

1.7.6 Latest 1.16.5 Version

Java version

Java 8 321

Issue Description

Does not show my name on the smaller player statues stature, it does show it on the bigger one, here is an image: 277517970_300850588790221_2199820208817243622_n

Additional Information

No response

Mrbysco commented 2 years ago

What is your in-game name so I can see if I can reproduce it?

lSherozl commented 2 years ago

What is your in-game name so I can see if I can reproduce it?

Sorry just saw this my in game name is....... ISherozI ......that is with 2 capital I's not lowercase L's

lSherozl commented 2 years ago

I am also playing on a modpack, better Minecraft plus, do not know if its a conflict with some mod, but the bigger statue of me works fine

Mrbysco commented 2 years ago

I think this is an issue in 1.16 Jade. In the 1.18 version of Jade it renders the name correctly

lSherozl commented 2 years ago

I think this is an issue in 1.16 Jade. In the 1.18 version of Jade it renders the name correctly

So I should talk to the Jade owners?

Snownee commented 2 years ago

so what the error is? it should be logged in WailaErrorOutput.txt

lSherozl commented 2 years ago

so what the error is? it should be logged in WailaErrorOutput.txt

Sorry I don't know what that is, the error is that in the picture instead of displaying my name with jade on the 1.16.5 version, it displays "Error"

Snownee commented 2 years ago

it's under .minecraft folder

Mrbysco commented 2 years ago

WailaErrorOutput.txt I happen to have the output from when I was trying to debug the issue

Mrbysco commented 2 years ago

The code that gives the Player Statue's block it's name hasn't really changed between the 1.16 and 1.18 version

Snownee commented 2 years ago

I think this is the cause: https://github.com/ShyNieke/StatuesMod/blob/1.16.5/src/main/java/com/shynieke/statues/tiles/PlayerTile.java#L218-L221

I think the return value should be nonnull if you return true in hasCustomName

Mrbysco commented 2 years ago

I wonder why that works in 1.18 🤔 Did Jade's internal change

Snownee commented 2 years ago

yes, I added a null check.

Mrbysco commented 2 years ago

A new version has been uploaded to CurseForge to fix the issue 👍

lSherozl commented 2 years ago

A new version has been uploaded to CurseForge to fix the issue 👍

yes, I added a null check.

Thank you both