Centralize position getting and sorting

This commit is contained in:
ShadowNinja 2014-05-30 17:41:41 -04:00
parent 02905caaeb
commit abd6a4c709
2 changed files with 12 additions and 18 deletions

View File

@ -7,10 +7,8 @@ minetest.register_chatcommand("protect", {
if param == "" then if param == "" then
return false, "Invalid usage, see /help protect." return false, "Invalid usage, see /help protect."
end end
local pos1, pos2 = areas:getPos1(name), areas:getPos2(name) local pos1, pos2 = areas:getPos(name)
if pos1 and pos2 then if not (pos1 and pos2) then
pos1, pos2 = areas:sortPos(pos1, pos2)
else
return false, "You need to select an area first." return false, "You need to select an area first."
end end
@ -45,10 +43,8 @@ minetest.register_chatcommand("set_owner", {
return false, "Incorrect usage, see /help set_owner." return false, "Incorrect usage, see /help set_owner."
end end
local pos1, pos2 = areas:getPos1(name), areas:getPos2(name) local pos1, pos2 = areas:getPos(name)
if pos1 and pos2 then if not (pos1 and pos2) then
pos1, pos2 = areas:sortPos(pos1, pos2)
else
return false, "You need to select an area first." return false, "You need to select an area first."
end end
@ -87,10 +83,8 @@ minetest.register_chatcommand("add_owner", {
return return
end end
local pos1, pos2 = areas:getPos1(name), areas:getPos2(name) local pos1, pos2 = areas:getPos(name)
if pos1 and pos2 then if not (pos1 and pos2) then
pos1, pos2 = areas:sortPos(pos1, pos2)
else
return false, "You need to select an area first." return false, "You need to select an area first."
end end

10
pos.lua
View File

@ -118,12 +118,12 @@ minetest.register_chatcommand("area_pos", {
end, end,
}) })
function areas:getPos1(playerName) function areas:getPos(playerName)
return areas.pos1[playerName] local pos1, pos2 = areas.pos1[playerName], areas.pos2[playerName]
if not (pos1 and pos2) then
return nil
end end
return areas:sortPos(pos1, pos2)
function areas:getPos2(playerName)
return areas.pos2[playerName]
end end
function areas:setPos1(playerName, pos) function areas:setPos1(playerName, pos)