Use correct constructor in MappedLegacyBlockItem (#721)

FlorianMichael · web-flow · commit 1ac504afb940 · 2024-04-17T08:46:48.000+02:00
diff --git a/common/src/main/java/com/viaversion/viabackwards/api/data/MappedLegacyBlockItem.java b/common/src/main/java/com/viaversion/viabackwards/api/data/MappedLegacyBlockItem.java
@@ -33,7 +33,7 @@ public MappedLegacyBlockItem(int id, short data, @Nullable String name, boolean
         this.id = id;
         this.data = data;
         this.name = name != null ? "§f" + name : null;
-        this.block = block ? new IdAndData(id, data) : null;
+        this.block = block ? data != -1 ? new IdAndData(id, data) : new IdAndData(id) : null;
     }
 
     public int getId() {

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public MappedLegacyBlockItem(int id, short data, @Nullable String name, boolean`
`33`	`33`	`this.id = id;`
`34`	`34`	`this.data = data;`
`35`	`35`	`this.name = name != null ? "§f" + name : null;`
`36`		`- this.block = block ? new IdAndData(id, data) : null;`
	`36`	`+ this.block = block ? data != -1 ? new IdAndData(id, data) : new IdAndData(id) : null;`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`public int getId() {`