From 57c18fce9569f68822fb2814044d71ec99bf49e7 Mon Sep 17 00:00:00 2001 From: ShadowNinja Date: Sat, 12 Jul 2014 19:10:26 -0400 Subject: [PATCH] Make all admin checks use one table --- chatcommands.lua | 4 ++-- internal.lua | 4 ++-- legacy.lua | 4 ++-- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/chatcommands.lua b/chatcommands.lua index ecf521a..b467331 100644 --- a/chatcommands.lua +++ b/chatcommands.lua @@ -35,7 +35,7 @@ minetest.register_chatcommand("set_owner", { description = "Protect an area beetween two positions and give" .." a player access to it without setting the parent of the" .." area to any existing area", - privs = {areas=true}, + privs = areas.adminPrivs, func = function(name, param) local ownerName, areaName = param:match('^(%S+)%s(.+)$') @@ -175,7 +175,7 @@ minetest.register_chatcommand("find_areas", { minetest.register_chatcommand("list_areas", { description = "List your areas, or all areas if you are an admin.", func = function(name, param) - local admin = minetest.check_player_privs(name, {areas=true}) + local admin = minetest.check_player_privs(name, areas.adminPrivs) local areaStrings = {} for id, area in pairs(areas.areas) do if admin or areas:isAreaOwner(id, name) then diff --git a/internal.lua b/internal.lua index f032b1b..e06b99d 100644 --- a/internal.lua +++ b/internal.lua @@ -109,7 +109,7 @@ end -- Also checks the size of the area and if the user already -- has more than max_areas. function areas:canPlayerAddArea(pos1, pos2, name) - if minetest.check_player_privs(name, {areas=true}) then + if minetest.check_player_privs(name, self.adminPrivs) then return true end @@ -188,7 +188,7 @@ end -- Checks if a player owns an area or a parent of it function areas:isAreaOwner(id, name) local cur = self.areas[id] - if cur and minetest.check_player_privs(name, {areas=true}) then + if cur and minetest.check_player_privs(name, self.adminPrivs) then return true end while cur do diff --git a/legacy.lua b/legacy.lua index b689cc0..01db69a 100644 --- a/legacy.lua +++ b/legacy.lua @@ -5,7 +5,7 @@ minetest.register_chatcommand("legacy_load_areas", { params = "", description = "Loads, converts, and saves the areas from" .." a legacy save file.", - privs = {areas=true, server=true, privs=true}, + privs = {areas=true, server=true}, func = function(name, param) minetest.chat_send_player(name, "Converting areas...") local version = tonumber(param) @@ -86,7 +86,7 @@ end -- Checks if a node is owned by you function areas.isNodeOwner(pos, name) - if minetest.check_player_privs(name, {areas=true}) then + if minetest.check_player_privs(name, areas.adminPrivs) then return true end for id, area in pairs(areas:getAreasAtPos(pos)) do