Get rid of minetest.serialize in cooking code + cleaning of ench. table

[xdecor.git] / mailbox.lua

diff --git a/mailbox.lua b/mailbox.lua
index dd356ff7cdd5941281d9224a4653206c089a7528..b8b18bb7dc929157cac30fac887607f730c1e9c4 100644 (file)
--- a/mailbox.lua
+++ b/mailbox.lua
@@ -58,75 +58,67 @@ xdecor.register("mailbox", {
                        meta:set_string("stack1", stack_name)
                end
        end,
-       allow_metadata_inventory_put = function(pos, listname, _, stack, _)
+       allow_metadata_inventory_put = function(pos, listname, _, stack, player)
+               local player_name = player:get_player_name()
                if listname == "drop" then
                        local meta = minetest.get_meta(pos)
                        local inv = meta:get_inventory()
-                       if inv:room_for_item("mailbox", stack) then return -1 end
+                       if inv:room_for_item("mailbox", stack) then
+                               return -1
+                       else
+                               minetest.chat_send_player(player_name, "The mailbox is full")
+                       end
                end
+
                return 0
        end
 })
 
 local function img_col(stack)
        if not stack then return "" end
-       if stack.type == "node" then
-               if stack.inventory_image ~= "" then
-                       return stack.inventory_image:match("([%w_]+)%.png")..".png"
-               else
-                       return stack.tiles[1]:match("([%w_]+)%.png")..".png"
-               end
-       elseif stack.type == "tool" or stack.type == "craft" then
+       if stack.inventory_image ~= "" then
                return stack.inventory_image:match("([%w_]+)%.png")..".png"
-       else return "" end
+       else
+               return stack.tiles[1]:match("([%w_]+)%.png")..".png"
+       end
 end
 
 function mailbox.formspec(pos, owner, num)
-       local xbg = default.gui_bg..default.gui_bg_img..default.gui_slots
        local spos = pos.x..","..pos.y..","..pos.z
        local meta = minetest.get_meta(pos)
-       local giver = ""
-       local def_stack1 = minetest.registered_items[meta:get_string("stack1"):match("([%w_]+:[%w_]+)")]
-       local def_stack2 = minetest.registered_items[meta:get_string("stack2"):match("([%w_]+:[%w_]+)")]
-       local def_stack3 = minetest.registered_items[meta:get_string("stack3"):match("([%w_]+:[%w_]+)")]
-       local def_stack4 = minetest.registered_items[meta:get_string("stack4"):match("([%w_]+:[%w_]+)")]
-       local def_stack5 = minetest.registered_items[meta:get_string("stack5"):match("([%w_]+:[%w_]+)")]
-       local def_stack6 = minetest.registered_items[meta:get_string("stack6"):match("([%w_]+:[%w_]+)")]
-       local def_stack7 = minetest.registered_items[meta:get_string("stack7"):match("([%w_]+:[%w_]+)")]
+       local giver, img = "", ""
 
        if num == 1 then
                for i = 1, 7 do
                        if meta:get_string("giver"..i) ~= "" then
                                giver = giver.."#FFFF00,"..meta:get_string("giver"..i):sub(1, 12)..
                                        ","..i..",#FFFFFF,x "..meta:get_string("stack"..i):match("%s(%d+)")..","
+
+                               img = img..i.."="..img_col(minetest.registered_items[
+                                       meta:get_string("stack"..i):match("([%w_:]+)")])..","
                        end
                end
 
-               return "size[9.5,9]"..xbg..
-                       default.get_hotbar_bg(0.75,5.25)..
-                       "label[0,0;Mailbox :]"..
-                       "label[6,0;Last donators :]"..
-                       "box[6,0.72;3.3,3.5;#555555]"..
-                       "tablecolumns[color;text;image,"..
-                               "1="..img_col(def_stack1)..","..
-                               "2="..img_col(def_stack2)..","..
-                               "3="..img_col(def_stack3)..","..
-                               "4="..img_col(def_stack4)..","..
-                               "5="..img_col(def_stack5)..","..
-                               "6="..img_col(def_stack6)..","..
-                               "7="..img_col(def_stack7)..";color;text]"..
-                       "tableoptions[background=#00000000;highlight=#00000000;border=false]"..
+               return [[ size[9.5,9]
+                       label[0,0;Mailbox]
+                       label[6,0;Last donators]
+                       box[6,0.72;3.3,3.5;#555555]
+                       listring[current_player;main]
+                       list[current_player;main;0.75,5.25;8,4;]
+                       tableoptions[background=#00000000;highlight=#00000000;border=false] ]]
+                       .."tablecolumns[color;text;image,"..img.."0;color;text]"..
                        "table[6,0.75;3.3,4;givers;"..giver.."]"..
                        "list[nodemeta:"..spos..";mailbox;0,0.75;6,4;]"..
-                       "list[current_player;main;0.75,5.25;8,4;]"..
                        "listring[nodemeta:"..spos..";mailbox]"..
-                       "listring[current_player;main]"
+                       xbg..default.get_hotbar_bg(0.75,5.25)
        else
-               return "size[8,5]"..xbg..
-                       default.get_hotbar_bg(0,1.25)..
-                       "label[0.5,0;Send your goods\nto "..owner.." :]"..
+               return [[ size[8,5]
+                       list[current_player;main;0,1.25;8,4;]
+                       tablecolumns[color;text;color;text]
+                       tableoptions[background=#00000000;highlight=#00000000;border=false] ]]
+                       .."table[0,0;3,1;sendform;#FFFFFF,Send your goods to,,,#FFFF00,"..owner.."]"..
                        "list[nodemeta:"..spos..";drop;3.5,0;1,1;]"..
-                       "list[current_player;main;0,1.25;8,4;]"
+                       xbg..default.get_hotbar_bg(0,1.25)
        end
 end