From 7424ff3ddf96c4aaf69158628aaae84f6cae0ca1 Mon Sep 17 00:00:00 2001
From: MihailRis <mexaruc@gmail.com>
Date: Tue, 16 Dec 2025 00:45:15 +0300
Subject: [PATCH] cleanup

---
 src/items/Inventories.cpp   | 10 ++++++----
 src/items/Inventories.hpp   |  2 +-
 src/objects/Players.cpp     |  5 +----
 src/voxels/GlobalChunks.cpp |  1 -
 4 files changed, 8 insertions(+), 10 deletions(-)

diff --git a/src/items/Inventories.cpp b/src/items/Inventories.cpp
index 4d2baa36d..249f5d66d 100644
--- a/src/items/Inventories.cpp
+++ b/src/items/Inventories.cpp
@@ -1,9 +1,10 @@
 #include "Inventories.hpp"
 
-#include <algorithm>
-
 #include "world/Level.hpp"
 #include "world/World.hpp"
+#include "content/Content.hpp"
+
+#include <algorithm>
 
 Inventories::Inventories(Level& level) : level(level) {
 }
@@ -32,8 +33,9 @@ std::shared_ptr<Inventory> Inventories::createVirtual(size_t size) {
     });
 }
 
-void Inventories::store(const std::shared_ptr<Inventory>& inv) {
-    map[inv->getId()] = inv;
+void Inventories::store(const std::shared_ptr<Inventory>& inventory) {
+    inventory->check(*level.content.getIndices());
+    map[inventory->getId()] = inventory;
 }
 
 void Inventories::remove(int64_t id) {
diff --git a/src/items/Inventories.hpp b/src/items/Inventories.hpp
index 98e956839..6ce500256 100644
--- a/src/items/Inventories.hpp
+++ b/src/items/Inventories.hpp
@@ -26,7 +26,7 @@ public:
     /// @brief Create runtime-only inventory (has negative id)
     std::shared_ptr<Inventory> createVirtual(size_t size);
 
-    /// @brief Store inventory
+    /// @brief Check and add inventory to the storage
     void store(const std::shared_ptr<Inventory>& inv);
 
     /// @brief Remove inventory from map
diff --git a/src/objects/Players.cpp b/src/objects/Players.cpp
index 6bce34abd..a90d6d07d 100644
--- a/src/objects/Players.cpp
+++ b/src/objects/Players.cpp
@@ -88,9 +88,7 @@ Player* Players::create(int64_t id) {
     auto player = playerPtr.get();
     add(std::move(playerPtr));
 
-    auto inventory = player->getInventory();
-    inventory->check(*level.content.getIndices());
-    level.inventories->store(std::move(inventory));
+    level.inventories->store(player->getInventory());
     return player;
 }
 
@@ -150,7 +148,6 @@ void Players::deserialize(const dv::value& src) {
         if (inventory->getId() == 0) {
             inventory->setId(level.getWorld()->getNextInventoryId());
         }
-        inventory->check(*level.content.getIndices());
         level.inventories->store(inventory);
     }
 }
diff --git a/src/voxels/GlobalChunks.cpp b/src/voxels/GlobalChunks.cpp
index d6cc34d20..6d19dda79 100644
--- a/src/voxels/GlobalChunks.cpp
+++ b/src/voxels/GlobalChunks.cpp
@@ -124,7 +124,6 @@ std::shared_ptr<Chunk> GlobalChunks::create(int x, int z, bool lighting) {
 
         chunk->flags.loaded = true;
         for (auto& entry : chunk->inventories) {
-            entry.second->check(indices);
             level.inventories->store(entry.second);
         }
     }