From 733e263f86bd4f73ef2b8aa0e24a802931c1909e Mon Sep 17 00:00:00 2001
From: 1F616EMO <root@1f616emo.xyz>
Date: Fri, 25 Oct 2024 18:56:21 +0800
Subject: [PATCH] Reduce code duplication in register_on_punchnode

Merges "pos1" and "pos1only" into the same code.
---
 pos.lua | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)

diff --git a/pos.lua b/pos.lua
index 284984e..c24c3d6 100644
--- a/pos.lua
+++ b/pos.lua
@@ -229,24 +229,18 @@ minetest.register_on_punchnode(function(pos, node, puncher)
 	local name = puncher:get_player_name()
 	-- Currently setting position
 	if name ~= "" and areas.set_pos[name] then
-		if areas.set_pos[name] == "pos1" then
-			areas:setPos1(name, pos)
-			areas.set_pos[name] = "pos2"
+		if areas.set_pos[name] == "pos2" then
+			areas:setPos2(name, pos)
+			areas.set_pos[name] = nil
 			minetest.chat_send_player(name,
-					S("Position @1 set to @2", "1",
+					S("Position @1 set to @2", "2",
 					minetest.pos_to_string(pos)))
-		elseif areas.set_pos[name] == "pos1only" then
+		else
 			areas:setPos1(name, pos)
-			areas.set_pos[name] = nil
+			areas.set_pos[name] = areas.set_pos[name] == "pos1" and "pos2" or nil
 			minetest.chat_send_player(name,
 					S("Position @1 set to @2", "1",
 					minetest.pos_to_string(pos)))
-		elseif areas.set_pos[name] == "pos2" then
-			areas:setPos2(name, pos)
-			areas.set_pos[name] = nil
-			minetest.chat_send_player(name,
-					S("Position @1 set to @2", "2",
-					minetest.pos_to_string(pos)))
 		end
 	end
 end)