From c05e57efb1f3d55c89354c28a84e76abe63aadd5 Mon Sep 17 00:00:00 2001
From: NO11 <no11@noreply.git.minetest.land>
Date: Sat, 24 Jul 2021 14:09:47 +0000
Subject: [PATCH] Fix some crashes with set_mod_namespace and bugs

---
 mods/HELP/mcl_item_id/init.lua | 46 +++++++++++++++++++++-------------
 1 file changed, 28 insertions(+), 18 deletions(-)

diff --git a/mods/HELP/mcl_item_id/init.lua b/mods/HELP/mcl_item_id/init.lua
index 911d8225b..9a2f926e8 100644
--- a/mods/HELP/mcl_item_id/init.lua
+++ b/mods/HELP/mcl_item_id/init.lua
@@ -1,22 +1,38 @@
-mcl_item_id = {}
+mcl_item_id = {
+    mod_namespaces = {},
+}
 
 local game = "mineclone"
 
 function mcl_item_id.set_mod_namespace(modname, namespace)
     local namespace = namespace or modname
-    mcl_item_id[modname .. "_namespace"] = namespace
+    mcl_item_id.mod_namespaces[modname] = namespace
+    minetest.register_on_mods_loaded(function()
+        for item, def in pairs(minetest.registered_items) do
+            local item_split = item:find(":")
+            if item_split then
+                local id_modname = item:sub(1, item_split - 1)
+                local id_string = item:sub(item_split)
+                if id_modname == modname then
+                    minetest.register_alias_force(namespace .. id_string, item)
+                end
+            end
+        end
+    end)
 end
 
 function mcl_item_id.get_mod_namespace(modname)
-    local namespace = mcl_item_id[modname .. "_namespace"]
+    local namespace = mcl_item_id.mod_namespaces[modname]
     if namespace then
         return namespace
     else
-        return
+        return game
     end
 end
 
 local same_id = {
+    enchanting = { "table" },
+    experience = { "bottle" },
     heads = { "skeleton", "zombie", "creeper", "wither_skeleton" },
     mobitems = { "rabbit", "chicken" },
     walls = {
@@ -34,13 +50,11 @@ local same_id = {
 
 tt.register_snippet(function(itemstring)
     local def = minetest.registered_items[itemstring]
-    local desc = def.description
     local item_split = itemstring:find(":")
-    local id_part1 = itemstring:sub(1, item_split)
-    local id_part2 = itemstring:sub(item_split)
-    local modname = id_part1:gsub("%:", "")
-    local new_id = game .. id_part2
-    local mod_namespace = mcl_item_id.get_mod_namespace(modname)
+    local id_string = itemstring:sub(item_split)
+    local id_modname = itemstring:sub(1, item_split - 1)
+    local new_id = game .. id_string
+    local mod_namespace = mcl_item_id.get_mod_namespace(id_modname)
     for mod, ids in pairs(same_id) do
         for _, id in pairs(ids) do
             if itemstring == "mcl_" .. mod .. ":" .. id  then
@@ -48,16 +62,12 @@ tt.register_snippet(function(itemstring)
             end
         end
     end
-    
-    if mod_namespace then
-        new_id = mod_namespace .. id_part2
-    end
-    if new_id ~= game .. ":book_enchanted" then
+    if mod_namespace ~= game then
+        new_id = mod_namespace .. id_string
+    else
         minetest.register_alias_force(new_id, itemstring)
     end
     if minetest.settings:get_bool("mcl_item_id_debug", false) then
         return new_id, "#555555"
     end
-end)
-
-minetest.register_alias_force(game .. ":book_enchanted", "mcl_enchanting:book_enchanted")
+end)
\ No newline at end of file