Add spam flag

.github/workflows/test.yml

@@ -9,9 +9,9 @@ jobs:
     timeout-minutes: 10
     strategy:
       matrix:
-        ENGINE_VERSION: [5.0.1, 5.1.1, 5.2.0, 5.3.0, 5.4.1, 5.5.1, 5.6.1, 5.7.0, 5.8.0, 5.9.0, latest]
+        ENGINE_VERSION: [5.0.1, 5.1.1, 5.2.0, 5.3.0, 5.4.1, 5.5.1, 5.6.1, 5.7.0, 5.8.0, latest]
 
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v3
     - name: test
-      run: docker compose up --exit-code-from sut
+      run: docker-compose up --exit-code-from sut

README.md

@@ -11,8 +11,7 @@ It adds a mail-system that allows players to send each other messages in-game an
 
 # Screenshot
 
-![Main view](screenshot_1.4.0_1.png)
-![Message view](screenshot_1.4.0_2.png)
+![](screenshot_1.2.0.png)
 
 # Installation
 
@@ -44,8 +43,7 @@ Mails can be deleted, marked as read or unread, replied to and forwarded to anot
 * Multiple selection (new in 1.1.0)
 * Settings
 * Chat, on join, HUD and sound notifications
-* Anti-spam detection
-* Translated in : English, French, German, Chinese (both traditional and simplified), Spanish, Brazilian Portuguese, Hungarian, Indonesian.
+* Translated in : English, French, German, Chinese (both traditional and simplified), Spanish, Brazilian Portuguese.
 
 # Compatibility / Migration
 
@@ -53,10 +51,8 @@ Overview:
 * `v1` all the data is in the `<worldfolder>/mails.db` file
 * `v2` every player has its own (in-) mailbox in the `<worldfolder>/mails/<playername>.json` file
 * `v3` every player has an entry in the `<playername>` `mod_storage/` (inbox, outbox, drafts, contacts, mailing lists, settings)
-* `v3.1` database fix after the message id mess
 
 # Dependencies
-
 * None
 
 # License
@@ -76,7 +72,7 @@ See the "LICENSE" file
 * fluxionary (Minor fixups)
 * Toby1710 (UX fixes)
 * Peter Nerlich (CC, BCC)
-* Emojigit (Performance, Traditional Chinese translation)
+* Emojigit (Traditional Chinese translation)
 * Niklp09 (German translation)
 * Dennis Jenkins (UX fixes)
 * Thomas Rudin (Maintenance)
@@ -88,12 +84,7 @@ See the "LICENSE" file
 * nyomi (Hungarian translation)
 * whosit (UI fixes)
 * Wuzzy (German translation)
-* savilli (UX fixes)
-* Panquesito7 (Maintenance)
-* Eredin (Spanish translation)
 * Muhammad Rifqi Priyo Susanto (Indonesian translation)
-* aBlueShadow (sfinv compatibility)
-* Singularis (UX and storage fixes)
 
 # Contribute
 

api.lua

@@ -1,7 +1,7 @@
 -- see: mail.md
 
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local f = string.format
 

docker-compose.yml

@@ -1,4 +1,4 @@
-version: "4.1"
+version: "3.6"
 
 services:
  sut:
@@ -16,4 +16,4 @@ services:
     - "30000:30000/udp"
 
 volumes:
-  world_data: {}
+  world_data: {}

init.lua

@@ -5,9 +5,6 @@ mail = {
 	-- mod storage
 	storage = minetest.get_mod_storage(),
 
-	-- translation
-	S = minetest.get_translator(minetest.get_current_modname()),
-
 	-- ui theme prepend
 	theme = "",
 

migrate.lua

@@ -91,29 +91,17 @@ local function search_box(playername, box, uuid)
 	return false
 end
 
-local function search_boxes(playername, boxes, uuid)
-	local result
-	for _, b in ipairs(boxes) do
-		result = search_box(playername, b, uuid)
-		if result then return result end
-	end
-end
-
 local function is_uuid_existing(uuid)
-	local boxes = {"inbox", "outbox", "drafts", "trash"}
-	if mail.storage.get_keys then
-		for _, k in ipairs(mail.storage:get_keys()) do
-			if string.sub(k,1,5) == "mail/" then
-				local p = string.sub(k, 6)
-				local result = search_boxes(p, boxes, uuid)
+    for _, k in ipairs(mail.storage:get_keys()) do
+        if string.sub(k,1,5) == "mail/" then
+			local p = string.sub(k, 6)
+			local result
+			local boxes = {"inbox", "outbox", "drafts", "trash"}
+			for _, b in ipairs(boxes) do
+				result = search_box(p, b, uuid)
 				if result then return result end
 			end
 		end
-	else
-		for p, _ in minetest.get_auth_handler().iterate() do
-			local result = search_boxes(p, boxes, uuid)
-			if result then return result end
-		end
     end
     return false
 end
@@ -128,78 +116,55 @@ local function are_message_sames(a, b)
 	   and a.body == b.body
 end
 
-local function replace_other_player_message_uuid(p, m, uuid, new_uuid)
-	local er = mail.get_storage_entry(p)
-	for _, r in ipairs(er.inbox) do
-		if r.id == uuid and not are_message_sames(m, r) then
-			r.id = new_uuid
-		end
-	end
-	for _, r in ipairs(er.outbox) do
-		if r.id == uuid and not are_message_sames(m, r) then
-			r.id = new_uuid
-		end
-	end
-	for _, r in ipairs(er.drafts) do
-		if r.id == uuid and not are_message_sames(m, r) then
-			r.id = new_uuid
-		end
-	end
-	for _, r in ipairs(er.trash) do
-		if r.id == uuid and not are_message_sames(m, r) then
-			r.id = new_uuid
-		end
-	end
-	mail.set_storage_entry(p, er)
-end
-
-local function fix_box_duplicate_uuids(playername, box)
+local function fix_duplicate_uuids(playername, box)
 	local e = mail.get_storage_entry(playername)
 	for _, m in ipairs(e[box]) do
 		local uuid = m.id
 		local exists = is_uuid_existing(uuid)
 		if exists and not are_message_sames(exists, m) then
 			local new_uuid = mail.new_uuid() -- generates a new uuid to replace doublons
-			if mail.storage.get_keys then
-				for _, k in ipairs(mail.storage:get_keys()) do
-					if string.sub(k,1,5) == "mail/" then
-						local p = string.sub(k, 6)
-						replace_other_player_message_uuid(p, m, uuid, new_uuid)
+			for _, k in ipairs(mail.storage:get_keys()) do
+				if string.sub(k,1,5) == "mail/" then
+					local p = string.sub(k, 6)
+					local er = mail.get_storage_entry(p)
+					for _, r in ipairs(er.inbox) do
+						if r.id == uuid and not are_message_sames(m, r) then
+							r.id = new_uuid
+						end
 					end
-				end
-			else
-				for p, _ in minetest.get_auth_handler().iterate() do
-					replace_other_player_message_uuid(p, m, uuid, new_uuid)
+					for _, r in ipairs(er.outbox) do
+						if r.id == uuid and not are_message_sames(m, r) then
+							r.id = new_uuid
+						end
+					end
+					for _, r in ipairs(er.drafts) do
+						if r.id == uuid and not are_message_sames(m, r) then
+							r.id = new_uuid
+						end
+					end
+					for _, r in ipairs(er.trash) do
+						if r.id == uuid and not are_message_sames(m, r) then
+							r.id = new_uuid
+						end
+					end
+					mail.set_storage_entry(p, er)
 				end
 			end
 		end
 	end
 end
 
-local function fix_player_duplicate_uuids(playername)
-	fix_box_duplicate_uuids(playername, "inbox")
-	fix_box_duplicate_uuids(playername, "outbox")
-	fix_box_duplicate_uuids(playername, "drafts")
-	fix_box_duplicate_uuids(playername, "trash")
-end
-
 -- repair database for uuid doublons
 local function repair_storage()
 	-- iterate through players
-	-- get_keys() was introduced in 5.7
-	if mail.storage.get_keys then
-		for _, k in ipairs(mail.storage:get_keys()) do
-			if string.sub(k,1,5) == "mail/" then
-				local p = string.sub(k, 6)
-				fix_player_duplicate_uuids(p)
-			end
+	for _, k in ipairs(mail.storage:get_keys()) do
+		if string.sub(k,1,5) == "mail/" then
+			local p = string.sub(k, 6)
+			fix_duplicate_uuids(p, "inbox")
+			fix_duplicate_uuids(p, "outbox")
+			fix_duplicate_uuids(p, "drafts")
+			fix_duplicate_uuids(p, "trash")
 		end
-	else
-		minetest.after(0, function()
-			for p, _ in minetest.get_auth_handler().iterate() do
-				fix_player_duplicate_uuids(p)
-			end
-		end)
 	end
 end
 

onjoin.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 minetest.register_on_joinplayer(function(player)
 	minetest.after(2, function(name)

player_recipients.lua

@@ -1,6 +1,4 @@
--- translation
-local S = mail.S
-
+local S = minetest.get_translator("mail")
 local has_canonical_name = minetest.get_modpath("canonical_name")
 
 mail.register_on_player_receive(function(name, msg)

ui/about.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:about"
 
@@ -19,7 +19,7 @@ local contributors = {
 	{ name = "BuckarooBanzay", groups = {"c"} },
 	{ name = "Chache", groups = {"i"} },
 	{ name = "Dennis Jenkins", groups = {"c"} },
-	{ name = "Emojigit", groups = {"c", "i"} },
+	{ name = "Emojigit", groups = {"i"} },
 	{ name = "Eredin", groups = {"i"} },
 	{ name = "fluxionary", groups = {"c"} },
 	{ name = "imre84", groups = {"c"} },
@@ -55,7 +55,7 @@ function mail.show_about(name)
 			label[0.2,0;Mail]
 
 			label[0.2,0.5;]] .. S("Provided by mt-mods") .. [[]
-			label[0.2,0.9;]] .. S("Version: @1", "1.5.0-dev") .. [[]
+			label[0.2,0.9;]] .. S("Version: @1", "1.4.0-dev") .. [[]
 
 			box[0,1.5;3,0.45;]] .. mail.get_color("highlighted") .. [[]
 			label[0.2,1.5;]] .. S("Licenses") .. [[]

ui/compose.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:compose"
 

ui/contacts.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:contacts"
 

ui/drafts.lua

@@ -1,5 +1,6 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
+
 
 function mail.show_drafts(name)
 	local trash_tab = ""

ui/edit_contact.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:editcontact"
 

ui/edit_maillists.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:editmaillist"
 

ui/inbox.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 function mail.show_inbox(name, sortfieldindex, sortdirection, filter)
     sortfieldindex = tonumber(sortfieldindex or mail.selected_idxs.sortfield[name])
@@ -77,7 +77,6 @@ function mail.show_inbox(name, sortfieldindex, sortdirection, filter)
 
     local unread_color_enable = mail.get_setting(name, "unreadcolorenable")
     local cc_color_enable = mail.get_setting(name, "cccolorenable")
-    local mute_list = mail.get_setting(name, "mute_list")
 
     if #messages > 0 then
         for _, message in ipairs(messages) do
@@ -104,7 +103,7 @@ function mail.show_inbox(name, sortfieldindex, sortdirection, filter)
             if message.spam then
                 table.insert(displayed_color, "warning")
             end
-            if table.indexof(mute_list, message.from) >= 1 then
+            if table.indexof(mail.get_setting(name, "mute_list"), message.from) >= 1 then
                 table.insert(displayed_color, "muted")
             end
             formspec[#formspec + 1] = "," .. mail.get_color(displayed_color)

ui/maillists.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:maillists"
 

ui/message.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:message"
 

ui/outbox.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 function mail.show_outbox(name, sortfieldindex, sortdirection, filter)
     sortfieldindex = tonumber(sortfieldindex or mail.selected_idxs.sortfield[name])

ui/select_contact.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:selectcontact"
 

ui/settings.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local FORMNAME = "mail:settings"
 

ui/trash.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local trash_formspec = "size[8.5,11;]" .. mail.theme .. [[
 	tabheader[0.3,1;boxtab;]] ..

util/contact.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 function mail.compile_contact_list(name, selected, playernames)
 	-- TODO: refactor this - not just compiles *a* list, but *the* list for the contacts screen (too inflexible)

util/normalize.lua

@@ -1,5 +1,4 @@
--- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 local function recursive_expand_recipient_names(sender, list, is_toplevel, recipients, undeliverable)
     for _, name in ipairs(list) do

util/settings.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 mail.settings = {
     chat_notifications = {

util/time_ago.lua

@@ -1,5 +1,5 @@
 -- translation
-local S = mail.S
+local S = minetest.get_translator("mail")
 
 function mail.time_ago(t)
     local elapsed = os.time() - t