sixam/mail
1
0
Fork 0
mirror of https://github.com/mt-mods/mail.git synced 2025-04-30 08:21:44 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix crash if non-existent mail was selected

Browse source
This commit is contained in:
savilli 2023-08-05 15:58:14 +02:00
parent 2a18322cdb
commit c2da8afe8c
1 changed files with 16 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
ui/events.lua
View file

@ -18,8 +18,10 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
if formname ~= "mail:inbox" and formname ~= "mail:outbox"
and formname ~= "mail:drafts" and formname ~= "mail:trash" then
return
elseif fields.quit then
return
end
if fields.quit then
return true
end
-- Get player name and handle / convert common input fields
@ -64,7 +66,12 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
end
mail.selected_idxs.sortfield[name] = evt.column-1 -- update column
mail.show_mail_menu(name)
return
return true
end
local inbox = getInbox()[evt.row-1]
if not inbox then
mail.show_mail_menu(name)
return true
end
if mail.selected_idxs.multipleselection[name] then
if not mail.selected_idxs.inbox[name] then
@ -73,7 +80,7 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
local selected_id = 0
if mail.selected_idxs.inbox[name] and #mail.selected_idxs.inbox[name] > 0 then
for i, selected_msg in ipairs(mail.selected_idxs.inbox[name]) do
if getInbox()[evt.row-1].id == selected_msg then
if inbox.id == selected_msg then
selected_id = i
table.remove(mail.selected_idxs.inbox[name], i)
break
@ -81,13 +88,13 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
end
end
if selected_id == 0 then
table.insert(mail.selected_idxs.inbox[name], getInbox()[evt.row-1].id)
table.insert(mail.selected_idxs.inbox[name], inbox.id)
end
else
mail.selected_idxs.inbox[name] = { (getInbox()[evt.row-1] or {}).id }
mail.selected_idxs.inbox[name] = { inbox.id }
end
if evt.type == "DCL" and getInbox()[evt.row-1] then
mail.show_message(name, getInbox()[evt.row-1].id)
if evt.type == "DCL" then
mail.show_message(name, inbox.id)
else
mail.show_mail_menu(name)
end
@ -102,7 +109,7 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
end
mail.selected_idxs.sortfield[name] = evt.column-1 -- update column
mail.show_mail_menu(name)
return
return true
end
if mail.selected_idxs.multipleselection[name] then
if not mail.selected_idxs.outbox[name] then