only crop the palleteidx if param2 is colorfacedir

(prevent a potential bug later: some day, maybe there
will be more than 89 colors in UD)
This commit is contained in:
Vanessa Ezekowitz 2017-01-28 18:19:07 -05:00
parent aef03c1932
commit 966166b1b9

@ -300,7 +300,9 @@ function unifieddyes.on_rightclick(pos, node, player, stack, pointed_thing, newn
newnode = string.gsub(newnode, "_grey", "_"..HUES[hue]) newnode = string.gsub(newnode, "_grey", "_"..HUES[hue])
end end
end end
node.param2 = paletteidx + (minetest.get_node(pos).param2 % 32) node.param2 = paletteidx + (minetest.get_node(pos).param2 % 32)
else
node.param2 = paletteidx
end end
node.name = newnode node.name = newnode
minetest.swap_node(pos, node) minetest.swap_node(pos, node)
@ -316,8 +318,10 @@ function unifieddyes.on_rightclick(pos, node, player, stack, pointed_thing, newn
elseif string.find(minetest.get_node(pos).name, "_grey") and hue ~= 0 then elseif string.find(minetest.get_node(pos).name, "_grey") and hue ~= 0 then
newnode.name = string.gsub(newnode.name, "_grey", "_"..HUES[hue]) newnode.name = string.gsub(newnode.name, "_grey", "_"..HUES[hue])
end end
newnode.param2 = paletteidx + (minetest.get_node(pos).param2 % 32)
else
newnode.param2 = paletteidx
end end
newnode.param2 = paletteidx + (minetest.get_node(pos).param2 % 32)
minetest.swap_node(pos, newnode) minetest.swap_node(pos, newnode)
end end
else -- here is where a node is just being placed, not something being colored else -- here is where a node is just being placed, not something being colored