From 64e70c3b6447e1c81a100d334ae5617d9ad5a5d4 Mon Sep 17 00:00:00 2001
From: TheMode <themode@outlook.fr>
Date: Sat, 10 Apr 2021 00:01:06 +0200
Subject: [PATCH] Use the builder methods instead of fields

---
 .../java/net/minestom/server/item/meta/CompassMeta.java    | 7 +++----
 src/main/java/net/minestom/server/item/meta/MapMeta.java   | 7 +++++--
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/src/main/java/net/minestom/server/item/meta/CompassMeta.java b/src/main/java/net/minestom/server/item/meta/CompassMeta.java
index 8a153fa40..edf642398 100644
--- a/src/main/java/net/minestom/server/item/meta/CompassMeta.java
+++ b/src/main/java/net/minestom/server/item/meta/CompassMeta.java
@@ -85,18 +85,17 @@ public class CompassMeta extends ItemMeta implements ItemMetaBuilder.Provider<Co
         @Override
         public void read(@NotNull NBTCompound nbtCompound) {
             if (nbtCompound.containsKey("LodestoneTracked")) {
-                this.lodestoneTracked = nbtCompound.getByte("LodestoneTracked") == 1;
+                lodestoneTracked(nbtCompound.getByte("LodestoneTracked") == 1);
             }
             if (nbtCompound.containsKey("LodestoneDimension")) {
-                this.lodestoneDimension = nbtCompound.getString("LodestoneDimension");
+                lodestoneDimension(nbtCompound.getString("LodestoneDimension"));
             }
             if (nbtCompound.containsKey("LodestonePos")) {
                 final NBTCompound posCompound = nbtCompound.getCompound("LodestonePos");
                 final int x = posCompound.getInt("X");
                 final int y = posCompound.getInt("Y");
                 final int z = posCompound.getInt("Z");
-
-                this.lodestonePosition = new Position(x, y, z);
+                lodestonePosition(new Position(x, y, z));
             }
         }
 
diff --git a/src/main/java/net/minestom/server/item/meta/MapMeta.java b/src/main/java/net/minestom/server/item/meta/MapMeta.java
index 243880a7d..f6b6c2398 100644
--- a/src/main/java/net/minestom/server/item/meta/MapMeta.java
+++ b/src/main/java/net/minestom/server/item/meta/MapMeta.java
@@ -11,6 +11,7 @@ import org.jglrxavpok.hephaistos.nbt.NBTCompound;
 import org.jglrxavpok.hephaistos.nbt.NBTList;
 import org.jglrxavpok.hephaistos.nbt.NBTTypes;
 
+import java.util.ArrayList;
 import java.util.List;
 import java.util.concurrent.CopyOnWriteArrayList;
 import java.util.function.Supplier;
@@ -151,6 +152,7 @@ public class MapMeta extends ItemMeta {
 
             if (compound.containsKey("Decorations")) {
                 final NBTList<NBTCompound> decorationsList = compound.getList("Decorations");
+                List<MapDecoration> mapDecorations = new ArrayList<>();
                 for (NBTCompound decorationCompound : decorationsList) {
                     final String id = decorationCompound.getString("id");
                     final byte type = decorationCompound.getAsByte("type");
@@ -170,14 +172,15 @@ public class MapMeta extends ItemMeta {
                         rotation = decorationCompound.getAsDouble("rot");
                     }
 
-                    this.decorations.add(new MapDecoration(id, type, x, z, rotation));
+                    mapDecorations.add(new MapDecoration(id, type, x, z, rotation));
                 }
+                decorations(mapDecorations);
             }
 
             if (compound.containsKey("display")) {
                 final NBTCompound displayCompound = compound.getCompound("display");
                 if (displayCompound.containsKey("MapColor")) {
-                    this.mapColor = new Color(displayCompound.getAsInt("MapColor"));
+                    mapColor(new Color(displayCompound.getAsInt("MapColor")));
                 }
             }
         }