Upstream/Minestom
1
0
Fork 0
mirror of https://github.com/Minestom/Minestom.git synced 2025-01-06 16:37:38 +01:00
Code Issues Projects Releases Wiki Activity

Fix item attributes not being read correctly (#1806)

Browse Source
This commit is contained in:
TogAr2 2023-03-29 15:09:24 +02:00 committed by GitHub
parent ec8034750a
commit 9abb3079f6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 18 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/net/minestom/server/item/ItemSerializers.java
View File

@ -57,7 +57,7 @@ public final class ItemSerializers {
final int operation = reader.getTag(OPERATION);
final String name = reader.getTag(NAME);
final Attribute attribute = Attribute.fromKey(attributeName.toUpperCase(Locale.ROOT));
final Attribute attribute = Attribute.fromKey(attributeName.toLowerCase(Locale.ROOT));
// Wrong attribute name, stop here
if (attribute == null) return null;
final AttributeOperation attributeOperation = AttributeOperation.fromId(operation);

17
src/test/java/net/minestom/server/item/ItemAttributeTest.java
View File

@ -4,6 +4,10 @@ import net.minestom.server.attribute.Attribute;
import net.minestom.server.attribute.AttributeOperation;
import net.minestom.server.item.attribute.AttributeSlot;
import net.minestom.server.item.attribute.ItemAttribute;
import net.minestom.server.tag.TagHandler;
import net.minestom.server.tag.TagWritable;
import org.jglrxavpok.hephaistos.nbt.NBT;
import org.jglrxavpok.hephaistos.nbt.NBTCompound;
import org.junit.jupiter.api.Test;
import java.util.List;
@ -25,6 +29,19 @@ public class ItemAttributeTest {
assertEquals(attributes, item.meta().getAttributes());
}
@Test
public void attributeReader() {
var attributes = List.of(new ItemAttribute(
new UUID(0, 0), "generic.attack_damage", Attribute.ATTACK_DAMAGE,
AttributeOperation.ADDITION, 2, AttributeSlot.MAINHAND));
TagHandler handler = TagHandler.newHandler();
handler.setTag(ItemTags.ATTRIBUTES, attributes);
var item = ItemStack.fromNBT(Material.STICK, handler.asCompound());
assertEquals(attributes, item.meta().getAttributes());
}
@Test
public void attributeNbt() {
var item = ItemStack.builder(Material.STICK)