From 1dbcf701858ed68515559d31dd46d4dfb3d43d27 Mon Sep 17 00:00:00 2001
From: kilbith <jeanpatrick.guerrero@gmail.com>
Date: Tue, 3 May 2016 01:58:02 +0200
Subject: [PATCH] Painting: Use on_place for random placing

---
 mailbox.lua |  2 +-
 nodes.lua   | 23 +++++++++++++++++------
 2 files changed, 18 insertions(+), 7 deletions(-)

diff --git a/mailbox.lua b/mailbox.lua
index dc02521..0411333 100644
--- a/mailbox.lua
+++ b/mailbox.lua
@@ -94,7 +94,7 @@ function mailbox.put(pos, listname, _, stack, player)
 		if inv:room_for_item("mailbox", stack) then
 			return -1
 		else
-			minetest.chat_send_player(player:get_player_name(), "[!] The mailbox is full")
+			minetest.chat_send_player(player:get_player_name(), "The mailbox is full")
 		end
 	end
 	return 0
diff --git a/nodes.lua b/nodes.lua
index fdd6fdb..83f44d6 100644
--- a/nodes.lua
+++ b/nodes.lua
@@ -54,7 +54,7 @@ xdecor.register("baricade", {
 	groups = {choppy=2, oddly_breakable_by_hand=1, flammable=2},
 	damage_per_second = 4,
 	selection_box = xdecor.nodebox.slab_y(0.3),
-	collision_box = xdecor.pixelbox(2, {{0,0,1,2,2,0}})
+	collision_box = xdecor.pixelbox(2, {{0, 0, 1, 2, 2, 0}})
 })
 
 xdecor.register("barrel", {
@@ -359,11 +359,22 @@ xdecor.register("painting_1", {
 	groups = {choppy=3, oddly_breakable_by_hand=2, flammable=2, attached_node=1},
 	sounds = default.node_sound_wood_defaults(),
 	node_box = painting_box,
-	on_construct = function(pos)
-		local node = minetest.get_node(pos)
-		local random = math.random(4)
-		if random == 1 then return end
-		minetest.set_node(pos, {name="xdecor:painting_"..random, param2=node.param2})
+	node_placement_prediction = "",
+	on_place = function(itemstack, placer, pointed_thing)
+		local player_name = placer:get_player_name()
+		local pos = pointed_thing.above
+
+		if not minetest.is_protected(pos, player_name) then
+			local num = math.random(4)
+			local dir = minetest.dir_to_wallmounted(placer:get_look_dir())
+			minetest.set_node(pos, {name="xdecor:painting_"..num, param2=dir})
+		else
+			minetest.chat_send_player(player_name, "This area is protected")
+		end
+		if not minetest.setting_getbool("creative_mode") then
+			itemstack:take_item()
+			return itemstack
+		end
 	end
 })