Hopefully deal with item stacking issues (again)

Octo-Studios / relics

Other

42 stars 32 forks source link

Hopefully deal with item stacking issues (again) #43

Closed wchen1990 closed 2 years ago

wchen1990 commented 3 years ago

Change NBTUtils get methods to use getTag since it shouldn't be creating any tags if it's only getting and there's a check to see if the tag exists in the first place

SSKirillSS commented 2 years ago

Seems like I can't reproduce this issue anymore (tested for a long time on my server). Does this change rly required?

wchen1990 commented 2 years ago

Would probably be for the best since we already checked for the existence of the tag anyways so there's no reason to create the tag. It just makes more sense semantically.