From 74770ab71818a16b49274ac2261c0577cbdccfce Mon Sep 17 00:00:00 2001
From: Perttu Ahola <celeron55@gmail.com>
Date: Fri, 25 Nov 2011 17:42:41 +0200
Subject: [PATCH] Don't allow placing CONTENT_IGNORE with Map::setNode()
 because it is never useful and is only caused by bugs.

---
 src/map.cpp | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/map.cpp b/src/map.cpp
index 822b3f65..3f858ed6 100644
--- a/src/map.cpp
+++ b/src/map.cpp
@@ -208,6 +208,15 @@ void Map::setNode(v3s16 p, MapNode & n)
 	v3s16 blockpos = getNodeBlockPos(p);
 	MapBlock *block = getBlockNoCreate(blockpos);
 	v3s16 relpos = p - blockpos*MAP_BLOCKSIZE;
+	// Never allow placing CONTENT_IGNORE, it fucks up stuff
+	if(n.getContent() == CONTENT_IGNORE){
+		errorstream<<"Map::setNode(): Not allowing to place CONTENT_IGNORE"
+				<<" while trying to replace \""
+				<<m_gamedef->ndef()->get(block->getNodeNoCheck(relpos)).name
+				<<"\" at "<<PP(p)<<" (block "<<PP(blockpos)<<")"<<std::endl;
+		debug_stacks_print_to(errorstream);
+		return;
+	}
 	block->setNodeNoCheck(relpos, n);
 }