]> git.lizzy.rs Git - worldedit.git/blobdiff - worldedit/manipulations.lua
Correct spelling of Minetest
[worldedit.git] / worldedit / manipulations.lua
index 2e3e3696a3fbf5bbbc6e9486df2a7e16bd354f1e..ee5156137ab780fbe131de0cad6044ecd2fcbb32 100644 (file)
-worldedit = worldedit or {}\r
-local minetest = minetest --local copy of global\r
-\r
---modifies positions `pos1` and `pos2` so that each component of `pos1` is less than or equal to its corresponding conent of `pos2`, returning two new positions\r
-worldedit.sort_pos = function(pos1, pos2)\r
-       pos1 = {x=pos1.x, y=pos1.y, z=pos1.z}\r
-       pos2 = {x=pos2.x, y=pos2.y, z=pos2.z}\r
-       if pos1.x > pos2.x then\r
-               pos2.x, pos1.x = pos1.x, pos2.x\r
-       end\r
-       if pos1.y > pos2.y then\r
-               pos2.y, pos1.y = pos1.y, pos2.y\r
-       end\r
-       if pos1.z > pos2.z then\r
-               pos2.z, pos1.z = pos1.z, pos2.z\r
-       end\r
-       return pos1, pos2\r
-end\r
+--- Generic node manipulations.\r
+-- @module worldedit.manipulations\r
 \r
---determines the volume of the region defined by positions `pos1` and `pos2`, returning the volume\r
-worldedit.volume = function(pos1, pos2)\r
-       local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
-       return (pos2.x - pos1.x + 1) * (pos2.y - pos1.y + 1) * (pos2.z - pos1.z + 1)\r
-end\r
+local mh = worldedit.manip_helpers\r
 \r
---sets a region defined by positions `pos1` and `pos2` to `nodename`, returning the number of nodes filled\r
-worldedit.set = function(pos1, pos2, nodenames)\r
-       if type(nodenames) == "string" then\r
-               nodenames = {nodenames}\r
-       end\r
 \r
-       local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
-\r
-       --set up voxel manipulator\r
-       local manip = minetest.get_voxel_manip()\r
-       local emerged_pos1, emerged_pos2 = manip:read_from_map(pos1, pos2)\r
-       local area = VoxelArea:new({MinEdge=emerged_pos1, MaxEdge=emerged_pos2})\r
+--- Sets a region to `node_names`.\r
+-- @param pos1\r
+-- @param pos2\r
+-- @param node_names Node name or list of node names.\r
+-- @return The number of nodes set.\r
+function worldedit.set(pos1, pos2, node_names)\r
+       pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       --fill emerged area with ignore\r
-       local nodes = {}\r
-       local ignore = minetest.get_content_id("ignore")\r
-       for i = 1, worldedit.volume(emerged_pos1, emerged_pos2) do\r
-               nodes[i] = ignore\r
-       end\r
+       local manip, area = mh.init(pos1, pos2)\r
+       local data = mh.get_empty_data(area)\r
 \r
-       --fill selected area with node\r
-       local node_ids = {}\r
-       for i,v in ipairs(nodenames) do\r
-               node_ids[i] = minetest.get_content_id(nodenames[i])\r
-       end\r
-       if #node_ids == 1 then --only one type of node\r
-               local id = node_ids[1]\r
-               for i in area:iterp(pos1, pos2) do nodes[i] = id end --fill area with node\r
-       else --several types of nodes specified\r
+       if type(node_names) == "string" then -- Only one type of node\r
+               local id = minetest.get_content_id(node_names)\r
+               -- Fill area with node\r
+               for i in area:iterp(pos1, pos2) do\r
+                       data[i] = id\r
+               end\r
+       else -- Several types of nodes specified\r
+               local node_ids = {}\r
+               for i, v in ipairs(node_names) do\r
+                       node_ids[i] = minetest.get_content_id(v)\r
+               end\r
+               -- Fill area randomly with nodes\r
                local id_count, rand = #node_ids, math.random\r
-               for i in area:iterp(pos1, pos2) do nodes[i] = node_ids[rand(id_count)] end --fill randomly with all types of specified nodes\r
-       end\r
-\r
-       --update map nodes\r
-       manip:set_data(nodes)\r
-       manip:write_to_map()\r
-       manip:update_map()\r
-\r
-       return worldedit.volume(pos1, pos2)\r
-end\r
-\r
---replaces all instances of `searchnode` with `replacenode` in a region defined by positions `pos1` and `pos2`, returning the number of nodes replaced\r
-worldedit.replace = function(pos1, pos2, searchnode, replacenode)\r
-       local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
-\r
-       --set up voxel manipulator\r
-       local manip = minetest.get_voxel_manip()\r
-       local emerged_pos1, emerged_pos2 = manip:read_from_map(pos1, pos2)\r
-       local area = VoxelArea:new({MinEdge=emerged_pos1, MaxEdge=emerged_pos2})\r
-\r
-       local nodes = manip:get_data()\r
-       local searchnode_id = minetest.get_content_id(searchnode)\r
-       local replacenode_id = minetest.get_content_id(replacenode)\r
-       local count = 0\r
-       for i in area:iterp(pos1, pos2) do --replace searchnode with replacenode\r
-               if nodes[i] == searchnode_id then\r
-                       nodes[i] = replacenode_id\r
-                       count = count + 1\r
+               for i in area:iterp(pos1, pos2) do\r
+                       data[i] = node_ids[rand(id_count)]\r
                end\r
        end\r
 \r
-       --update map nodes\r
-       manip:set_data(nodes)\r
-       manip:write_to_map()\r
-       manip:update_map()\r
+       mh.finish(manip, data)\r
 \r
-       return count\r
+       return worldedit.volume(pos1, pos2)\r
 end\r
 \r
---replaces all nodes other than `searchnode` with `replacenode` in a region defined by positions `pos1` and `pos2`, returning the number of nodes replaced\r
-worldedit.replaceinverse = function(pos1, pos2, searchnode, replacenode)\r
-       local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
+--- Sets param2 of a region.\r
+-- @param pos1\r
+-- @param pos2\r
+-- @param param2 Value of param2 to set\r
+-- @return The number of nodes set.\r
+function worldedit.set_param2(pos1, pos2, param2)\r
+       pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       --set up voxel manipulator\r
-       local manip = minetest.get_voxel_manip()\r
-       local emerged_pos1, emerged_pos2 = manip:read_from_map(pos1, pos2)\r
-       local area = VoxelArea:new({MinEdge=emerged_pos1, MaxEdge=emerged_pos2})\r
+       local manip, area = mh.init(pos1, pos2)\r
+       local param2_data = manip:get_param2_data()\r
 \r
-       local nodes = manip:get_data()\r
-       local searchnode_id = minetest.get_content_id(searchnode)\r
-       local replacenode_id = minetest.get_content_id(replacenode)\r
-       local count = 0\r
-       for i in area:iterp(pos1, pos2) do --replace anything that is not searchnode with replacenode\r
-               if nodes[i] ~= searchnode_id then\r
-                       nodes[i] = replacenode_id\r
-                       count = count + 1\r
-               end\r
+       -- Set param2 for every node\r
+       for i in area:iterp(pos1, pos2) do\r
+               param2_data[i] = param2\r
        end\r
 \r
-       --update map nodes\r
-       manip:set_data(nodes)\r
+       -- Update map\r
+       manip:set_param2_data(param2_data)\r
        manip:write_to_map()\r
        manip:update_map()\r
 \r
-       return count\r
+       return worldedit.volume(pos1, pos2)\r
 end\r
 \r
---copies the region defined by positions `pos1` and `pos2` along the `axis` axis ("x" or "y" or "z") by `amount` nodes, returning the number of nodes copied\r
-worldedit.copy = function(pos1, pos2, axis, amount) --wip: replace the old version below\r
+--- Replaces all instances of `search_node` with `replace_node` in a region.\r
+-- When `inverse` is `true`, replaces all instances that are NOT `search_node`.\r
+-- @return The number of nodes replaced.\r
+function worldedit.replace(pos1, pos2, search_node, replace_node, inverse)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       if amount == 0 then\r
-               return\r
-       end\r
+       local manip, area = mh.init(pos1, pos2)\r
+       local data = manip:get_data()\r
 \r
-       local other1, other2\r
-       if axis == "x" then\r
-               other1, other2 = "y", "z"\r
-       elseif axis == "y" then\r
-               other1, other2 = "x", "z"\r
-       else --axis == "z"\r
-               other1, other2 = "x", "y"\r
-       end\r
+       local search_id = minetest.get_content_id(search_node)\r
+       local replace_id = minetest.get_content_id(replace_node)\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
-       manip:read_from_map(pos1, pos2)\r
+       local count = 0\r
 \r
-       --prepare slice along axis\r
-       local extent = {\r
-               [axis] = 1,\r
-               [other1]=pos2[other1] - pos1[other1] + 1,\r
-               [other2]=pos2[other2] - pos1[other2] + 1,\r
-       }\r
-       local nodes = {}\r
-       local schematic = {size=extent, data=nodes}\r
-\r
-       local currentpos = {x=pos1.x, y=pos1.y, z=pos1.z}\r
-       local stride = {x=1, y=extent.x, z=extent.x * extent.y}\r
-       local get_node = minetest.get_node\r
-       for index1 = 1, extent[axis] do --go through each slice\r
-               --copy slice into schematic\r
-               local newindex1 = (index1 + offset[axis]) * stride[axis] + 1 --offset contributed by axis plus 1 to make it 1-indexed\r
-               for index2 = 1, extent[other1] do\r
-                       local newindex2 = newindex1 + (index2 + offset[other1]) * stride[other1]\r
-                       for index3 = 1, extent[other2] do\r
-                               local i = newindex2 + (index3 + offset[other2]) * stride[other2]\r
-                               local node = get_node(pos)\r
-                               node.param1 = 255 --node will always appear\r
-                               nodes[i] = node\r
+       --- TODO: This could be shortened by checking `inverse` in the loop,\r
+       -- but that would have a speed penalty.  Is the penalty big enough\r
+       -- to matter?\r
+       if not inverse then\r
+               for i in area:iterp(pos1, pos2) do\r
+                       if data[i] == search_id then\r
+                               data[i] = replace_id\r
+                               count = count + 1\r
                        end\r
                end\r
-\r
-               --copy schematic to target\r
-               currentpos[axis] = currentpos[axis] + amount\r
-               place_schematic(currentpos, schematic)\r
-\r
-               --wip: copy meta\r
-\r
-               currentpos[axis] = currentpos[axis] + 1\r
-       end\r
-       return worldedit.volume(pos1, pos2)\r
-end\r
-\r
-worldedit.copy2 = function(pos1, pos2, direction, volume)\r
-       -- the overlap shouldn't matter as long as we\r
-       -- 1) start at the furthest separated corner\r
-       -- 2) complete an edge before moving inward, either edge works\r
-       -- 3) complete a face before moving inward, similarly\r
-       --\r
-       -- to do this I\r
-       -- 1) find the furthest destination in the direction, of each axis\r
-       -- 2) call those the furthest separated corner\r
-       -- 3) make sure to iterate inward from there\r
-       -- 4) nested loop to make sure complete edge, complete face, then complete cube.\r
-\r
-       local get_node, get_meta, add_node = minetest.get_node, minetest.get_meta, minetest.add_node\r
-       local somemeta = get_meta(pos1) -- hax lol\r
-       local to_table = somemeta.to_table\r
-       local from_table = somemeta.from_table\r
-       somemeta = nil\r
-\r
-       local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
-       local manip = minetest.get_voxel_manip()\r
-       manip:read_from_map(pos1, pos2)\r
-\r
-       local sx, sy, sz -- direction sign\r
-       local ix, iy, iz -- initial destination\r
-       local ex, ey, ez -- final destination\r
-       local originalx, originaly, originalz -- source\r
-       -- vim -> :'<,'>s/\<\([ioes]\?\)x\>/\1y/g\r
-       if direction.x > 0 then\r
-               originalx = pos2.x\r
-               ix = originalx + direction.x\r
-               ex = pos1.x + direction.x\r
-               sx = -1\r
-       elseif direction.x < 0 then\r
-               originalx = pos1.x\r
-               ix = originalx + direction.x\r
-               ex = pos2.x + direction.x\r
-               sx = 1\r
-       else\r
-               originalx = pos1.x\r
-               ix = originalx -- whatever\r
-               ex = pos2.x\r
-               sx = 1\r
-       end\r
-\r
-       if direction.y > 0 then\r
-               originaly = pos2.y\r
-               iy = originaly + direction.y\r
-               ey = pos1.y + direction.y\r
-               sy = -1\r
-       elseif direction.y < 0 then\r
-               originaly = pos1.y\r
-               iy = originaly + direction.y\r
-               ey = pos2.y + direction.y\r
-               sy = 1\r
-       else\r
-               originaly = pos1.y\r
-               iy = originaly -- whatever\r
-               ey = pos2.y\r
-               sy = 1\r
-       end\r
-\r
-       if direction.z > 0 then\r
-               originalz = pos2.z\r
-               iz = originalz + direction.z\r
-               ez = pos1.z + direction.z\r
-               sz = -1\r
-       elseif direction.z < 0 then\r
-               originalz = pos1.z\r
-               iz = originalz + direction.z\r
-               ez = pos2.z + direction.z\r
-               sz = 1\r
        else\r
-               originalz = pos1.z\r
-               iz = originalz -- whatever\r
-               ez = pos2.z\r
-               sz = 1\r
-       end\r
-       -- print('copy',originalx,ix,ex,sx,originaly,iy,ey,sy,originalz,iz,ez,sz)\r
-\r
-       local ox,oy,oz\r
-\r
-       ox = originalx\r
-       for x = ix, ex, sx do\r
-               oy = originaly\r
-               for y = iy, ey, sy do\r
-                       oz = originalz\r
-                       for z = iz, ez, sz do\r
-                               -- reusing pos1/pos2 as source/dest here\r
-                               pos1.x, pos1.y, pos1.z = ox, oy, oz\r
-                               pos2.x, pos2.y, pos2.z = x, y, z\r
-                               local node = get_node(pos1)\r
-                               local meta = to_table(get_meta(pos1)) --get meta of current node\r
-                               add_node(pos2,node)\r
-                               from_table(get_meta(pos2),meta)\r
-                               oz = oz + sz\r
+               for i in area:iterp(pos1, pos2) do\r
+                       if data[i] ~= search_id then\r
+                               data[i] = replace_id\r
+                               count = count + 1\r
                        end\r
-                       oy = oy + sy\r
                end\r
-               ox = ox + sx\r
        end\r
+\r
+       mh.finish(manip, data)\r
+\r
+       return count\r
 end\r
 \r
---duplicates the region defined by positions `pos1` and `pos2` `amount` times with offset vector `direction`, returning the number of nodes stacked\r
-worldedit.stack2 = function(pos1, pos2, direction, amount, finished)\r
+\r
+--- Duplicates a region `amount` times with offset vector `direction`.\r
+-- Stacking is spread across server steps, one copy per step.\r
+-- @return The number of nodes stacked.\r
+function worldedit.stack2(pos1, pos2, direction, amount, finished)\r
        local i = 0\r
-       local translated = {x=0,y=0,z=0}\r
-       local function nextone()\r
-               if i <= amount then\r
+       local translated = {x=0, y=0, z=0}\r
+       local function next_one()\r
+               if i < amount then\r
                        i = i + 1\r
                        translated.x = translated.x + direction.x\r
                        translated.y = translated.y + direction.y\r
                        translated.z = translated.z + direction.z\r
-                       worldedit.copy2(pos1, pos2, translated, volume)\r
-                       minetest.after(0, nextone)\r
+                       worldedit.copy2(pos1, pos2, translated)\r
+                       minetest.after(0, next_one)\r
                else\r
                        if finished then\r
                                finished()\r
                        end\r
                end\r
        end\r
-       nextone()\r
+       next_one()\r
        return worldedit.volume(pos1, pos2) * amount\r
 end\r
 \r
---copies the region defined by positions `pos1` and `pos2` along the `axis` axis ("x" or "y" or "z") by `amount` nodes, returning the number of nodes copied\r
-worldedit.copy = function(pos1, pos2, axis, amount)\r
+\r
+--- Copies a region along `axis` by `amount` nodes.\r
+-- @param pos1\r
+-- @param pos2\r
+-- @param axis Axis ("x", "y", or "z")\r
+-- @param amount\r
+-- @return The number of nodes copied.\r
+function worldedit.copy(pos1, pos2, axis, amount)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
-       manip:read_from_map(pos1, pos2)\r
+       worldedit.keep_loaded(pos1, pos2)\r
 \r
-       local get_node, get_meta, add_node = minetest.get_node, minetest.get_meta, minetest.add_node\r
+       local get_node, get_meta, set_node = minetest.get_node,\r
+                       minetest.get_meta, minetest.set_node\r
+       -- Copy things backwards when negative to avoid corruption.\r
+       -- FIXME: Lots of code duplication here.\r
        if amount < 0 then\r
-               local pos = {x=pos1.x, y=0, z=0}\r
+               local pos = {}\r
+               pos.x = pos1.x\r
                while pos.x <= pos2.x do\r
                        pos.y = pos1.y\r
                        while pos.y <= pos2.y do\r
                                pos.z = pos1.z\r
                                while pos.z <= pos2.z do\r
-                                       local node = get_node(pos) --obtain current node\r
-                                       local meta = get_meta(pos):to_table() --get meta of current node\r
-                                       local value = pos[axis] --store current position\r
-                                       pos[axis] = value + amount --move along axis\r
-                                       add_node(pos, node) --copy node to new position\r
-                                       get_meta(pos):from_table(meta) --set metadata of new node\r
-                                       pos[axis] = value --restore old position\r
+                                       local node = get_node(pos) -- Obtain current node\r
+                                       local meta = get_meta(pos):to_table() -- Get meta of current node\r
+                                       local value = pos[axis] -- Store current position\r
+                                       pos[axis] = value + amount -- Move along axis\r
+                                       set_node(pos, node) -- Copy node to new position\r
+                                       get_meta(pos):from_table(meta) -- Set metadata of new node\r
+                                       pos[axis] = value -- Restore old position\r
                                        pos.z = pos.z + 1\r
                                end\r
                                pos.y = pos.y + 1\r
@@ -331,19 +163,20 @@ worldedit.copy = function(pos1, pos2, axis, amount)
                        pos.x = pos.x + 1\r
                end\r
        else\r
-               local pos = {x=pos2.x, y=0, z=0}\r
+               local pos = {}\r
+               pos.x = pos2.x\r
                while pos.x >= pos1.x do\r
                        pos.y = pos2.y\r
                        while pos.y >= pos1.y do\r
                                pos.z = pos2.z\r
                                while pos.z >= pos1.z do\r
-                                       local node = get_node(pos) --obtain current node\r
-                                       local meta = get_meta(pos):to_table() --get meta of current node\r
-                                       local value = pos[axis] --store current position\r
-                                       pos[axis] = value + amount --move along axis\r
-                                       add_node(pos, node) --copy node to new position\r
-                                       get_meta(pos):from_table(meta) --set metadata of new node\r
-                                       pos[axis] = value --restore old position\r
+                                       local node = get_node(pos) -- Obtain current node\r
+                                       local meta = get_meta(pos):to_table() -- Get meta of current node\r
+                                       local value = pos[axis] -- Store current position\r
+                                       pos[axis] = value + amount -- Move along axis\r
+                                       set_node(pos, node) -- Copy node to new position\r
+                                       get_meta(pos):from_table(meta) -- Set metadata of new node\r
+                                       pos[axis] = value -- Restore old position\r
                                        pos.z = pos.z - 1\r
                                end\r
                                pos.y = pos.y - 1\r
@@ -354,31 +187,70 @@ worldedit.copy = function(pos1, pos2, axis, amount)
        return worldedit.volume(pos1, pos2)\r
 end\r
 \r
---moves the region defined by positions `pos1` and `pos2` along the `axis` axis ("x" or "y" or "z") by `amount` nodes, returning the number of nodes moved\r
-worldedit.move = function(pos1, pos2, axis, amount)\r
+--- Copies a region by offset vector `off`.\r
+-- @param pos1\r
+-- @param pos2\r
+-- @param off\r
+-- @return The number of nodes copied.\r
+function worldedit.copy2(pos1, pos2, off)\r
+       local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
+\r
+       worldedit.keep_loaded(pos1, pos2)\r
+\r
+       local get_node, get_meta, set_node = minetest.get_node,\r
+                       minetest.get_meta, minetest.set_node\r
+       local pos = {}\r
+       pos.x = pos2.x\r
+       while pos.x >= pos1.x do\r
+               pos.y = pos2.y\r
+               while pos.y >= pos1.y do\r
+                       pos.z = pos2.z\r
+                       while pos.z >= pos1.z do\r
+                               local node = get_node(pos) -- Obtain current node\r
+                               local meta = get_meta(pos):to_table() -- Get meta of current node\r
+                               local newpos = vector.add(pos, off) -- Calculate new position\r
+                               set_node(newpos, node) -- Copy node to new position\r
+                               get_meta(newpos):from_table(meta) -- Set metadata of new node\r
+                               pos.z = pos.z - 1\r
+                       end\r
+                       pos.y = pos.y - 1\r
+               end\r
+               pos.x = pos.x - 1\r
+       end\r
+       return worldedit.volume(pos1, pos2)\r
+end\r
+\r
+--- Moves a region along `axis` by `amount` nodes.\r
+-- @return The number of nodes moved.\r
+function worldedit.move(pos1, pos2, axis, amount)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
-       manip:read_from_map(pos1, pos2)\r
+       worldedit.keep_loaded(pos1, pos2)\r
 \r
-       --wip: move slice by slice using schematic method in the move axis and transfer metadata in separate loop (and if the amount is greater than the length in the axis, copy whole thing at a time and erase original after, using schematic method)\r
-       local get_node, get_meta, add_node, remove_node = minetest.get_node, minetest.get_meta, minetest.add_node, minetest.remove_node\r
+       --- TODO: Move slice by slice using schematic method in the move axis\r
+       -- and transfer metadata in separate loop (and if the amount is\r
+       -- greater than the length in the axis, copy whole thing at a time and\r
+       -- erase original after, using schematic method).\r
+       local get_node, get_meta, set_node, remove_node = minetest.get_node,\r
+                       minetest.get_meta, minetest.set_node, minetest.remove_node\r
+       -- Copy things backwards when negative to avoid corruption.\r
+       --- FIXME: Lots of code duplication here.\r
        if amount < 0 then\r
-               local pos = {x=pos1.x, y=0, z=0}\r
+               local pos = {}\r
+               pos.x = pos1.x\r
                while pos.x <= pos2.x do\r
                        pos.y = pos1.y\r
                        while pos.y <= pos2.y do\r
                                pos.z = pos1.z\r
                                while pos.z <= pos2.z do\r
-                                       local node = get_node(pos) --obtain current node\r
-                                       local meta = get_meta(pos):to_table() --get metadata of current node\r
-                                       remove_node(pos)\r
-                                       local value = pos[axis] --store current position\r
-                                       pos[axis] = value + amount --move along axis\r
-                                       add_node(pos, node) --move node to new position\r
-                                       get_meta(pos):from_table(meta) --set metadata of new node\r
-                                       pos[axis] = value --restore old position\r
+                                       local node = get_node(pos) -- Obtain current node\r
+                                       local meta = get_meta(pos):to_table() -- Get metadata of current node\r
+                                       remove_node(pos) -- Remove current node\r
+                                       local value = pos[axis] -- Store current position\r
+                                       pos[axis] = value + amount -- Move along axis\r
+                                       set_node(pos, node) -- Move node to new position\r
+                                       get_meta(pos):from_table(meta) -- Set metadata of new node\r
+                                       pos[axis] = value -- Restore old position\r
                                        pos.z = pos.z + 1\r
                                end\r
                                pos.y = pos.y + 1\r
@@ -386,20 +258,21 @@ worldedit.move = function(pos1, pos2, axis, amount)
                        pos.x = pos.x + 1\r
                end\r
        else\r
-               local pos = {x=pos2.x, y=0, z=0}\r
+               local pos = {}\r
+               pos.x = pos2.x\r
                while pos.x >= pos1.x do\r
                        pos.y = pos2.y\r
                        while pos.y >= pos1.y do\r
                                pos.z = pos2.z\r
                                while pos.z >= pos1.z do\r
-                                       local node = get_node(pos) --obtain current node\r
-                                       local meta = get_meta(pos):to_table() --get metadata of current node\r
-                                       remove_node(pos)\r
-                                       local value = pos[axis] --store current position\r
-                                       pos[axis] = value + amount --move along axis\r
-                                       add_node(pos, node) --move node to new position\r
-                                       get_meta(pos):from_table(meta) --set metadata of new node\r
-                                       pos[axis] = value --restore old position\r
+                                       local node = get_node(pos) -- Obtain current node\r
+                                       local meta = get_meta(pos):to_table() -- Get metadata of current node\r
+                                       remove_node(pos) -- Remove current node\r
+                                       local value = pos[axis] -- Store current position\r
+                                       pos[axis] = value + amount -- Move along axis\r
+                                       set_node(pos, node) -- Move node to new position\r
+                                       get_meta(pos):from_table(meta) -- Set metadata of new node\r
+                                       pos[axis] = value -- Restore old position\r
                                        pos.z = pos.z - 1\r
                                end\r
                                pos.y = pos.y - 1\r
@@ -410,8 +283,15 @@ worldedit.move = function(pos1, pos2, axis, amount)
        return worldedit.volume(pos1, pos2)\r
 end\r
 \r
---duplicates the region defined by positions `pos1` and `pos2` along the `axis` axis ("x" or "y" or "z") `count` times, returning the number of nodes stacked\r
-worldedit.stack = function(pos1, pos2, axis, count)\r
+\r
+--- Duplicates a region along `axis` `amount` times.\r
+-- Stacking is spread across server steps, one copy per step.\r
+-- @param pos1\r
+-- @param pos2\r
+-- @param axis Axis direction, "x", "y", or "z".\r
+-- @param count\r
+-- @return The number of nodes stacked.\r
+function worldedit.stack(pos1, pos2, axis, count)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
        local length = pos2[axis] - pos1[axis] + 1\r
        if count < 0 then\r
@@ -421,72 +301,85 @@ worldedit.stack = function(pos1, pos2, axis, count)
        local amount = 0\r
        local copy = worldedit.copy\r
        local i = 1\r
-       function nextone()\r
+       local function next_one()\r
                if i <= count then\r
                        i = i + 1\r
                        amount = amount + length\r
                        copy(pos1, pos2, axis, amount)\r
-                       minetest.after(0, nextone)\r
+                       minetest.after(0, next_one)\r
                end\r
        end\r
-       nextone()\r
+       next_one()\r
        return worldedit.volume(pos1, pos2) * count\r
 end\r
 \r
---stretches the region defined by positions `pos1` and `pos2` by an factor of positive integers `stretchx`, `stretchy`. and `stretchz` along the X, Y, and Z axes, respectively, with `pos1` as the origin, returning the number of nodes scaled, the new scaled position 1, and the new scaled position 2\r
-worldedit.stretch = function(pos1, pos2, stretchx, stretchy, stretchz) --wip: test this\r
+\r
+--- Stretches a region by a factor of positive integers along the X, Y, and Z\r
+-- axes, respectively, with `pos1` as the origin.\r
+-- @param pos1\r
+-- @param pos2\r
+-- @param stretch_x Amount to stretch along X axis.\r
+-- @param stretch_y Amount to stretch along Y axis.\r
+-- @param stretch_z Amount to stretch along Z axis.\r
+-- @return The number of nodes scaled.\r
+-- @return The new scaled position 1.\r
+-- @return The new scaled position 2.\r
+function worldedit.stretch(pos1, pos2, stretch_x, stretch_y, stretch_z)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       --prepare schematic of large node\r
-       local get_node, get_meta, place_schematic = minetest.get_node, minetest.get_meta, minetest.place_schematic\r
+       -- Prepare schematic of large node\r
+       local get_node, get_meta, place_schematic = minetest.get_node,\r
+                       minetest.get_meta, minetest.place_schematic\r
        local placeholder_node = {name="", param1=255, param2=0}\r
        local nodes = {}\r
-       for i = 1, stretchx * stretchy * stretchz do\r
+       for i = 1, stretch_x * stretch_y * stretch_z do\r
                nodes[i] = placeholder_node\r
        end\r
-       local schematic = {size={x=stretchx, y=stretchy, z=stretchz}, data=nodes}\r
+       local schematic = {size={x=stretch_x, y=stretch_y, z=stretch_z}, data=nodes}\r
 \r
-       local sizex, sizey, sizez = stretchx - 1, stretchy - 1, stretchz - 1\r
+       local size_x, size_y, size_z = stretch_x - 1, stretch_y - 1, stretch_z - 1\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
        local new_pos2 = {\r
-               x=pos1.x + (pos2.x - pos1.x) * stretchx + sizex,\r
-               y=pos1.y + (pos2.y - pos1.y) * stretchy + sizey,\r
-               z=pos1.z + (pos2.z - pos1.z) * stretchz + sizez,\r
+               x = pos1.x + (pos2.x - pos1.x) * stretch_x + size_x,\r
+               y = pos1.y + (pos2.y - pos1.y) * stretch_y + size_y,\r
+               z = pos1.z + (pos2.z - pos1.z) * stretch_z + size_z,\r
        }\r
-       manip:read_from_map(pos1, new_pos2)\r
+       worldedit.keep_loaded(pos1, new_pos2)\r
 \r
        local pos = {x=pos2.x, y=0, z=0}\r
-       local bigpos = {x=0, y=0, z=0}\r
+       local big_pos = {x=0, y=0, z=0}\r
        while pos.x >= pos1.x do\r
                pos.y = pos2.y\r
                while pos.y >= pos1.y do\r
                        pos.z = pos2.z\r
                        while pos.z >= pos1.z do\r
-                               local node = get_node(pos) --obtain current node\r
-                               local meta = get_meta(pos):to_table() --get meta of current node\r
+                               local node = get_node(pos) -- Get current node\r
+                               local meta = get_meta(pos):to_table() -- Get meta of current node\r
 \r
-                               --calculate far corner of the big node\r
-                               local posx = pos1.x + (pos.x - pos1.x) * stretchx\r
-                               local posy = pos1.y + (pos.y - pos1.y) * stretchy\r
-                               local posz = pos1.z + (pos.z - pos1.z) * stretchz\r
+                               -- Calculate far corner of the big node\r
+                               local pos_x = pos1.x + (pos.x - pos1.x) * stretch_x\r
+                               local pos_y = pos1.y + (pos.y - pos1.y) * stretch_y\r
+                               local pos_z = pos1.z + (pos.z - pos1.z) * stretch_z\r
 \r
-                               --create large node\r
+                               -- Create large node\r
                                placeholder_node.name = node.name\r
                                placeholder_node.param2 = node.param2\r
-                               bigpos.x, bigpos.y, bigpos.z = posx, posy, posz\r
-                               place_schematic(bigpos, schematic)\r
-\r
-                               --fill in large node meta\r
-                               if next(meta.fields) ~= nil or next(meta.inventory) ~= nil then --node has meta fields\r
-                                       for x = 0, sizex do\r
-                                               for y = 0, sizey do\r
-                                                       for z = 0, sizez do\r
-                                                               bigpos.x, bigpos.y, bigpos.z = posx + x, posy + y, posz + z\r
-                                                               get_meta(bigpos):from_table(meta) --set metadata of new node\r
-                                                       end\r
-                                               end\r
+                               big_pos.x, big_pos.y, big_pos.z = pos_x, pos_y, pos_z\r
+                               place_schematic(big_pos, schematic)\r
+\r
+                               -- Fill in large node meta\r
+                               if next(meta.fields) ~= nil or next(meta.inventory) ~= nil then\r
+                                       -- Node has meta fields\r
+                                       for x = 0, size_x do\r
+                                       for y = 0, size_y do\r
+                                       for z = 0, size_z do\r
+                                               big_pos.x = pos_x + x\r
+                                               big_pos.y = pos_y + y\r
+                                               big_pos.z = pos_z + z\r
+                                               -- Set metadata of new node\r
+                                               get_meta(big_pos):from_table(meta)\r
+                                       end\r
+                                       end\r
                                        end\r
                                end\r
                                pos.z = pos.z - 1\r
@@ -495,11 +388,15 @@ worldedit.stretch = function(pos1, pos2, stretchx, stretchy, stretchz) --wip: te
                end\r
                pos.x = pos.x - 1\r
        end\r
-       return worldedit.volume(pos1, pos2) * stretchx * stretchy * stretchz, pos1, new_pos2\r
+       return worldedit.volume(pos1, pos2) * stretch_x * stretch_y * stretch_z, pos1, new_pos2\r
 end\r
 \r
---transposes a region defined by the positions `pos1` and `pos2` between the `axis1` and `axis2` axes, returning the number of nodes transposed, the new transposed position 1, and the new transposed position 2\r
-worldedit.transpose = function(pos1, pos2, axis1, axis2)\r
+\r
+--- Transposes a region between two axes.\r
+-- @return The number of nodes transposed.\r
+-- @return The new transposed position 1.\r
+-- @return The new transposed position 2.\r
+function worldedit.transpose(pos1, pos2, axis1, axis2)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
        local compare\r
@@ -515,37 +412,36 @@ worldedit.transpose = function(pos1, pos2, axis1, axis2)
                end\r
        end\r
 \r
-       --calculate the new position 2 after transposition\r
+       -- Calculate the new position 2 after transposition\r
        local new_pos2 = {x=pos2.x, y=pos2.y, z=pos2.z}\r
        new_pos2[axis1] = pos1[axis1] + extent2\r
        new_pos2[axis2] = pos1[axis2] + extent1\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
-       local upperbound = {x=pos2.x, y=pos2.y, z=pos2.z}\r
-       if upperbound[axis1] < new_pos2[axis1] then upperbound[axis1] = new_pos2[axis1] end\r
-       if upperbound[axis2] < new_pos2[axis2] then upperbound[axis2] = new_pos2[axis2] end\r
-       manip:read_from_map(pos1, upperbound)\r
+       local upper_bound = {x=pos2.x, y=pos2.y, z=pos2.z}\r
+       if upper_bound[axis1] < new_pos2[axis1] then upper_bound[axis1] = new_pos2[axis1] end\r
+       if upper_bound[axis2] < new_pos2[axis2] then upper_bound[axis2] = new_pos2[axis2] end\r
+       worldedit.keep_loaded(pos1, upper_bound)\r
 \r
        local pos = {x=pos1.x, y=0, z=0}\r
-       local get_node, get_meta, add_node = minetest.get_node, minetest.get_meta, minetest.add_node\r
+       local get_node, get_meta, set_node = minetest.get_node,\r
+                       minetest.get_meta, minetest.set_node\r
        while pos.x <= pos2.x do\r
                pos.y = pos1.y\r
                while pos.y <= pos2.y do\r
                        pos.z = pos1.z\r
                        while pos.z <= pos2.z do\r
                                local extent1, extent2 = pos[axis1] - pos1[axis1], pos[axis2] - pos1[axis2]\r
-                               if compare(extent1, extent2) then --transpose only if below the diagonal\r
+                               if compare(extent1, extent2) then -- Transpose only if below the diagonal\r
                                        local node1 = get_node(pos)\r
                                        local meta1 = get_meta(pos):to_table()\r
-                                       local value1, value2 = pos[axis1], pos[axis2] --save position values\r
-                                       pos[axis1], pos[axis2] = pos1[axis1] + extent2, pos1[axis2] + extent1 --swap axis extents\r
+                                       local value1, value2 = pos[axis1], pos[axis2] -- Save position values\r
+                                       pos[axis1], pos[axis2] = pos1[axis1] + extent2, pos1[axis2] + extent1 -- Swap axis extents\r
                                        local node2 = get_node(pos)\r
                                        local meta2 = get_meta(pos):to_table()\r
-                                       add_node(pos, node1)\r
+                                       set_node(pos, node1)\r
                                        get_meta(pos):from_table(meta1)\r
-                                       pos[axis1], pos[axis2] = value1, value2 --restore position values\r
-                                       add_node(pos, node2)\r
+                                       pos[axis1], pos[axis2] = value1, value2 -- Restore position values\r
+                                       set_node(pos, node2)\r
                                        get_meta(pos):from_table(meta2)\r
                                end\r
                                pos.z = pos.z + 1\r
@@ -557,19 +453,20 @@ worldedit.transpose = function(pos1, pos2, axis1, axis2)
        return worldedit.volume(pos1, pos2), pos1, new_pos2\r
 end\r
 \r
---flips a region defined by the positions `pos1` and `pos2` along the `axis` axis ("x" or "y" or "z"), returning the number of nodes flipped\r
-worldedit.flip = function(pos1, pos2, axis)\r
+\r
+--- Flips a region along `axis`.\r
+-- @return The number of nodes flipped.\r
+function worldedit.flip(pos1, pos2, axis)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
-       manip:read_from_map(pos1, pos2)\r
+       worldedit.keep_loaded(pos1, pos2)\r
 \r
-       --wip: flip the region slice by slice along the flip axis using schematic method\r
+       --- TODO: Flip the region slice by slice along the flip axis using schematic method.\r
        local pos = {x=pos1.x, y=0, z=0}\r
        local start = pos1[axis] + pos2[axis]\r
        pos2[axis] = pos1[axis] + math.floor((pos2[axis] - pos1[axis]) / 2)\r
-       local get_node, get_meta, add_node = minetest.get_node, minetest.get_meta, minetest.add_node\r
+       local get_node, get_meta, set_node = minetest.get_node,\r
+                       minetest.get_meta, minetest.set_node\r
        while pos.x <= pos2.x do\r
                pos.y = pos1.y\r
                while pos.y <= pos2.y do\r
@@ -577,14 +474,14 @@ worldedit.flip = function(pos1, pos2, axis)
                        while pos.z <= pos2.z do\r
                                local node1 = get_node(pos)\r
                                local meta1 = get_meta(pos):to_table()\r
-                               local value = pos[axis]\r
-                               pos[axis] = start - value\r
+                               local value = pos[axis] -- Save position\r
+                               pos[axis] = start - value -- Shift position\r
                                local node2 = get_node(pos)\r
                                local meta2 = get_meta(pos):to_table()\r
-                               add_node(pos, node1)\r
+                               set_node(pos, node1)\r
                                get_meta(pos):from_table(meta1)\r
-                               pos[axis] = value\r
-                               add_node(pos, node2)\r
+                               pos[axis] = value -- Restore position\r
+                               set_node(pos, node2)\r
                                get_meta(pos):from_table(meta2)\r
                                pos.z = pos.z + 1\r
                        end\r
@@ -595,63 +492,74 @@ worldedit.flip = function(pos1, pos2, axis)
        return worldedit.volume(pos1, pos2)\r
 end\r
 \r
---rotates a region defined by the positions `pos1` and `pos2` by `angle` degrees clockwise around axis `axis` (90 degree increment), returning the number of nodes rotated\r
-worldedit.rotate = function(pos1, pos2, axis, angle)\r
+\r
+--- Rotates a region clockwise around an axis.\r
+-- @param pos1\r
+-- @param pos2\r
+-- @param axis Axis ("x", "y", or "z").\r
+-- @param angle Angle in degrees (90 degree increments only).\r
+-- @return The number of nodes rotated.\r
+-- @return The new first position.\r
+-- @return The new second position.\r
+function worldedit.rotate(pos1, pos2, axis, angle)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       local axis1, axis2\r
-       if axis == "x" then\r
-               axis1, axis2 = "z", "y"\r
-       elseif axis == "y" then\r
-               axis1, axis2 = "x", "z"\r
-       else --axis == "z"\r
-               axis1, axis2 = "y", "x"\r
-       end\r
+       local other1, other2 = worldedit.get_axis_others(axis)\r
        angle = angle % 360\r
 \r
        local count\r
        if angle == 90 then\r
-               worldedit.flip(pos1, pos2, axis1)\r
-               count, pos1, pos2 = worldedit.transpose(pos1, pos2, axis1, axis2)\r
+               worldedit.flip(pos1, pos2, other1)\r
+               count, pos1, pos2 = worldedit.transpose(pos1, pos2, other1, other2)\r
        elseif angle == 180 then\r
-               worldedit.flip(pos1, pos2, axis1)\r
-               count = worldedit.flip(pos1, pos2, axis2)\r
+               worldedit.flip(pos1, pos2, other1)\r
+               count = worldedit.flip(pos1, pos2, other2)\r
        elseif angle == 270 then\r
-               worldedit.flip(pos1, pos2, axis2)\r
-               count, pos1, pos2 = worldedit.transpose(pos1, pos2, axis1, axis2)\r
+               worldedit.flip(pos1, pos2, other2)\r
+               count, pos1, pos2 = worldedit.transpose(pos1, pos2, other1, other2)\r
+       else\r
+               error("Only 90 degree increments are supported!")\r
        end\r
        return count, pos1, pos2\r
 end\r
 \r
---rotates all oriented nodes in a region defined by the positions `pos1` and `pos2` by `angle` degrees clockwise (90 degree increment) around the Y axis, returning the number of nodes oriented\r
-worldedit.orient = function(pos1, pos2, angle) --wip: support 6D facedir rotation along arbitrary axis\r
+\r
+--- Rotates all oriented nodes in a region clockwise around the Y axis.\r
+-- @param pos1\r
+-- @param pos2\r
+-- @param angle Angle in degrees (90 degree increments only).\r
+-- @return The number of nodes oriented.\r
+-- TODO: Support 6D facedir rotation along arbitrary axis.\r
+function worldedit.orient(pos1, pos2, angle)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
        local registered_nodes = minetest.registered_nodes\r
 \r
        local wallmounted = {\r
-               [90]={[0]=0, [1]=1, [2]=5, [3]=4, [4]=2, [5]=3},\r
-               [180]={[0]=0, [1]=1, [2]=3, [3]=2, [4]=5, [5]=4},\r
-               [270]={[0]=0, [1]=1, [2]=4, [3]=5, [4]=3, [5]=2}\r
+               [90]  = {[0]=0, 1, 5, 4, 2, 3},\r
+               [180] = {[0]=0, 1, 3, 2, 5, 4},\r
+               [270] = {[0]=0, 1, 4, 5, 3, 2}\r
        }\r
        local facedir = {\r
-               [90]={[0]=1, [1]=2, [2]=3, [3]=0},\r
-               [180]={[0]=2, [1]=3, [2]=0, [3]=1},\r
-               [270]={[0]=3, [1]=0, [2]=1, [3]=2}\r
+               [90]  = {[0]=1, 2, 3, 0},\r
+               [180] = {[0]=2, 3, 0, 1},\r
+               [270] = {[0]=3, 0, 1, 2}\r
        }\r
 \r
        angle = angle % 360\r
        if angle == 0 then\r
                return 0\r
        end\r
+       if angle % 90 ~= 0 then\r
+               error("Only 90 degree increments are supported!")\r
+       end\r
        local wallmounted_substitution = wallmounted[angle]\r
        local facedir_substitution = facedir[angle]\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
-       manip:read_from_map(pos1, pos2)\r
+       worldedit.keep_loaded(pos1, pos2)\r
 \r
        local count = 0\r
-       local get_node, get_meta, add_node = minetest.get_node, minetest.get_meta, minetest.add_node\r
+       local set_node, get_node, get_meta, swap_node = minetest.set_node,\r
+                       minetest.get_node, minetest.get_meta, minetest.swap_node\r
        local pos = {x=pos1.x, y=0, z=0}\r
        while pos.x <= pos2.x do\r
                pos.y = pos1.y\r
@@ -664,13 +572,13 @@ worldedit.orient = function(pos1, pos2, angle) --wip: support 6D facedir rotatio
                                        if def.paramtype2 == "wallmounted" then\r
                                                node.param2 = wallmounted_substitution[node.param2]\r
                                                local meta = get_meta(pos):to_table()\r
-                                               add_node(pos, node)\r
+                                               set_node(pos, node)\r
                                                get_meta(pos):from_table(meta)\r
                                                count = count + 1\r
                                        elseif def.paramtype2 == "facedir" then\r
                                                node.param2 = facedir_substitution[node.param2]\r
                                                local meta = get_meta(pos):to_table()\r
-                                               add_node(pos, node)\r
+                                               set_node(pos, node)\r
                                                get_meta(pos):from_table(meta)\r
                                                count = count + 1\r
                                        end\r
@@ -684,42 +592,53 @@ worldedit.orient = function(pos1, pos2, angle) --wip: support 6D facedir rotatio
        return count\r
 end\r
 \r
---fixes the lighting in a region defined by positions `pos1` and `pos2`, returning the number of nodes updated\r
-worldedit.fixlight = function(pos1, pos2)\r
+\r
+--- Attempts to fix the lighting in a region.\r
+-- @return The number of nodes updated.\r
+function worldedit.fixlight(pos1, pos2)\r
        local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
-       manip:read_from_map(pos1, pos2)\r
+       local vmanip = minetest.get_voxel_manip(pos1, pos2)\r
+       vmanip:write_to_map()\r
+       vmanip:update_map() -- this updates the lighting\r
 \r
-       local nodes = minetest.find_nodes_in_area(pos1, pos2, "air")\r
-       local dig_node = minetest.dig_node\r
-       for _, pos in ipairs(nodes) do\r
-               dig_node(pos)\r
-       end\r
-       return #nodes\r
+       return worldedit.volume(pos1, pos2)\r
 end\r
 \r
---clears all objects in a region defined by the positions `pos1` and `pos2`, returning the number of objects cleared\r
-worldedit.clearobjects = function(pos1, pos2)\r
-       local pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
 \r
-       --make area stay loaded\r
-       local manip = minetest.get_voxel_manip()\r
-       manip:read_from_map(pos1, pos2)\r
+--- Clears all objects in a region.\r
+-- @return The number of objects cleared.\r
+function worldedit.clear_objects(pos1, pos2)\r
+       pos1, pos2 = worldedit.sort_pos(pos1, pos2)\r
+\r
+       worldedit.keep_loaded(pos1, pos2)\r
 \r
-       local pos1x, pos1y, pos1z = pos1.x, pos1.y, pos1.z\r
-       local pos2x, pos2y, pos2z = pos2.x + 1, pos2.y + 1, pos2.z + 1\r
-       local center = {x=(pos1x + pos2x) / 2, y=(pos1y + pos2y) / 2, z=(pos1z + pos2z) / 2} --center of region\r
-       local radius = ((center.x - pos1x + 0.5) + (center.y - pos1y + 0.5) + (center.z - pos1z + 0.5)) ^ 0.5 --bounding sphere radius\r
+       -- Offset positions to include full nodes (positions are in the center of nodes)\r
+       local pos1x, pos1y, pos1z = pos1.x - 0.5, pos1.y - 0.5, pos1.z - 0.5\r
+       local pos2x, pos2y, pos2z = pos2.x + 0.5, pos2.y + 0.5, pos2.z + 0.5\r
+\r
+       -- Center of region\r
+       local center = {\r
+               x = pos1x + ((pos2x - pos1x) / 2),\r
+               y = pos1y + ((pos2y - pos1y) / 2),\r
+               z = pos1z + ((pos2z - pos1z) / 2)\r
+       }\r
+       -- Bounding sphere radius\r
+       local radius = math.sqrt(\r
+                       (center.x - pos1x) ^ 2 +\r
+                       (center.y - pos1y) ^ 2 +\r
+                       (center.z - pos1z) ^ 2)\r
        local count = 0\r
-       for _, obj in pairs(minetest.get_objects_inside_radius(center, radius)) do --all objects in bounding sphere\r
+       for _, obj in pairs(minetest.get_objects_inside_radius(center, radius)) do\r
                local entity = obj:get_luaentity()\r
-               if not (entity and entity.name:find("^worldedit:")) then --avoid WorldEdit entities\r
+               -- Avoid players and WorldEdit entities\r
+               if not obj:is_player() and (not entity or\r
+                               not entity.name:find("^worldedit:")) then\r
                        local pos = obj:getpos()\r
-                       if pos.x >= pos1x and pos.x <= pos2x\r
-                       and pos.y >= pos1y and pos.y <= pos2y\r
-                       and pos.z >= pos1z and pos.z <= pos2z then --inside region\r
+                       if pos.x >= pos1x and pos.x <= pos2x and\r
+                                       pos.y >= pos1y and pos.y <= pos2y and\r
+                                       pos.z >= pos1z and pos.z <= pos2z then\r
+                               -- Inside region\r
                                obj:remove()\r
                                count = count + 1\r
                        end\r
@@ -727,3 +646,4 @@ worldedit.clearobjects = function(pos1, pos2)
        end\r
        return count\r
 end\r
+\r