From b1ccd494f7176d1982898521f1a18c7f7ee3e8d6 Mon Sep 17 00:00:00 2001
From: BuckarooBanzay <BuckarooBanzay@users.noreply.github.com>
Date: Tue, 28 Mar 2023 19:28:49 +0200
Subject: [PATCH] fix some issues

---
 storage.lua      | 2 +-
 storage.spec.lua | 6 ++++++
 ui/compose.lua   | 4 ++--
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/storage.lua b/storage.lua
index 470b9a9..c9ed52c 100644
--- a/storage.lua
+++ b/storage.lua
@@ -119,7 +119,7 @@ end
 -- get all contacts
 function mail.get_contacts(playername)
 	local entry = mail.get_storage_entry(playername)
-	return entry.contact
+	return entry.contacts
 end
 
 -- returns the maillists of a player
diff --git a/storage.spec.lua b/storage.spec.lua
index e44dd15..1be9ea4 100644
--- a/storage.spec.lua
+++ b/storage.spec.lua
@@ -18,6 +18,12 @@ mtt.register("storage", function(callback)
     assert(contacts[1].note == contact.note)
     assert(contacts[1].name == contact.name)
 
+    -- read through api
+    local contacts2 = mail.get_contacts(playername)
+    assert(#contacts2 == 1)
+    assert(contacts2[1].note == contact.note)
+    assert(contacts2[1].name == contact.name)
+
     -- update
     mail.update_contact(playername, {
         name = contact.name,
diff --git a/ui/compose.lua b/ui/compose.lua
index 172ca40..ad7f653 100644
--- a/ui/compose.lua
+++ b/ui/compose.lua
@@ -32,7 +32,7 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
 
 	local name = player:get_player_name()
     if fields.send then
-        local err = mail.send({
+        local success, err = mail.send({
             from = name,
             to = fields.to,
             cc = fields.cc,
@@ -40,7 +40,7 @@ minetest.register_on_player_receive_fields(function(player, formname, fields)
             subject = fields.subject,
             body = fields.body,
         })
-        if err then
+        if not success then
             minetest.chat_send_player(name, err)
             return
         end