Rename CSM flavours to restrictions
[oweals/minetest.git] / src / server.cpp
index bd90afb323674ff90619d709987071e8eeae8543..753c71701139d02a01543a8f990064a39dbfb9f7 100644 (file)
@@ -44,13 +44,12 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "itemdef.h"
 #include "craftdef.h"
 #include "emerge.h"
-#include "mapgen.h"
-#include "mg_biome.h"
+#include "mapgen/mapgen.h"
+#include "mapgen/mg_biome.h"
 #include "content_mapnode.h"
 #include "content_nodemeta.h"
 #include "content_sao.h"
-#include "mods.h"
-#include "event_manager.h"
+#include "content/mods.h"
 #include "modchannels.h"
 #include "serverlist.h"
 #include "util/string.h"
@@ -58,10 +57,11 @@ with this program; if not, write to the Free Software Foundation, Inc.,
 #include "util/serialize.h"
 #include "util/thread.h"
 #include "defaultsettings.h"
+#include "server/mods.h"
 #include "util/base64.h"
 #include "util/sha1.h"
 #include "util/hex.h"
-#include "database.h"
+#include "database/database.h"
 #include "chatmessage.h"
 #include "chat_interface.h"
 #include "remoteplayer.h"
@@ -139,7 +139,60 @@ v3f ServerSoundParams::getPos(ServerEnvironment *env, bool *pos_exists) const
        return v3f(0,0,0);
 }
 
+void Server::ShutdownState::reset()
+{
+       m_timer = 0.0f;
+       message.clear();
+       should_reconnect = false;
+       is_requested = false;
+}
+
+void Server::ShutdownState::trigger(float delay, const std::string &msg, bool reconnect)
+{
+       m_timer = delay;
+       message = msg;
+       should_reconnect = reconnect;
+}
+
+void Server::ShutdownState::tick(float dtime, Server *server)
+{
+       if (m_timer <= 0.0f)
+               return;
+
+       // Timed shutdown
+       static const float shutdown_msg_times[] =
+       {
+               1, 2, 3, 4, 5, 10, 20, 40, 60, 120, 180, 300, 600, 1200, 1800, 3600
+       };
+
+       // Automated messages
+       if (m_timer < shutdown_msg_times[ARRLEN(shutdown_msg_times) - 1]) {
+               for (float t : shutdown_msg_times) {
+                       // If shutdown timer matches an automessage, shot it
+                       if (m_timer > t && m_timer - dtime < t) {
+                               std::wstring periodicMsg = getShutdownTimerMessage();
 
+                               infostream << wide_to_utf8(periodicMsg).c_str() << std::endl;
+                               server->SendChatMessage(PEER_ID_INEXISTENT, periodicMsg);
+                               break;
+                       }
+               }
+       }
+
+       m_timer -= dtime;
+       if (m_timer < 0.0f) {
+               m_timer = 0.0f;
+               is_requested = true;
+       }
+}
+
+std::wstring Server::ShutdownState::getShutdownTimerMessage() const
+{
+       std::wstringstream ws;
+       ws << L"*** Server shutting down in "
+               << duration_to_string(myround(m_timer)).c_str() << ".";
+       return ws.str();
+}
 
 /*
        Server
@@ -167,7 +220,6 @@ Server::Server(
        m_itemdef(createItemDefManager()),
        m_nodedef(createNodeDefManager()),
        m_craftdef(createCraftDefManager()),
-       m_event(new EventManager()),
        m_uptime(0),
        m_clients(m_con),
        m_admin_chat(iface),
@@ -175,22 +227,96 @@ Server::Server(
 {
        m_lag = g_settings->getFloat("dedicated_server_step");
 
-       if (path_world.empty())
+       if (m_path_world.empty())
                throw ServerError("Supplied empty world path");
 
-       if(!gamespec.isValid())
+       if (!gamespec.isValid())
                throw ServerError("Supplied invalid gamespec");
+}
+
+Server::~Server()
+{
+       infostream << "Server destructing" << std::endl;
 
-       infostream<<"Server created for gameid \""<<m_gamespec.id<<"\"";
-       if(m_simple_singleplayer_mode)
-               infostream<<" in simple singleplayer mode"<<std::endl;
+       // Send shutdown message
+       SendChatMessage(PEER_ID_INEXISTENT, ChatMessage(CHATMESSAGE_TYPE_ANNOUNCE,
+                       L"*** Server shutting down"));
+
+       if (m_env) {
+               MutexAutoLock envlock(m_env_mutex);
+
+               infostream << "Server: Saving players" << std::endl;
+               m_env->saveLoadedPlayers();
+
+               infostream << "Server: Kicking players" << std::endl;
+               std::string kick_msg;
+               bool reconnect = false;
+               if (isShutdownRequested()) {
+                       reconnect = m_shutdown_state.should_reconnect;
+                       kick_msg = m_shutdown_state.message;
+               }
+               if (kick_msg.empty()) {
+                       kick_msg = g_settings->get("kick_msg_shutdown");
+               }
+               m_env->kickAllPlayers(SERVER_ACCESSDENIED_SHUTDOWN,
+                       kick_msg, reconnect);
+       }
+
+       // Do this before stopping the server in case mapgen callbacks need to access
+       // server-controlled resources (like ModStorages). Also do them before
+       // shutdown callbacks since they may modify state that is finalized in a
+       // callback.
+       if (m_emerge)
+               m_emerge->stopThreads();
+
+       if (m_env) {
+               MutexAutoLock envlock(m_env_mutex);
+
+               // Execute script shutdown hooks
+               infostream << "Executing shutdown hooks" << std::endl;
+               m_script->on_shutdown();
+
+               infostream << "Server: Saving environment metadata" << std::endl;
+               m_env->saveMeta();
+       }
+
+       // Stop threads
+       if (m_thread) {
+               stop();
+               delete m_thread;
+       }
+
+       // Delete things in the reverse order of creation
+       delete m_emerge;
+       delete m_env;
+       delete m_rollback;
+       delete m_banmanager;
+       delete m_itemdef;
+       delete m_nodedef;
+       delete m_craftdef;
+
+       // Deinitialize scripting
+       infostream << "Server: Deinitializing scripting" << std::endl;
+       delete m_script;
+
+       // Delete detached inventories
+       for (auto &detached_inventory : m_detached_inventories) {
+               delete detached_inventory.second;
+       }
+}
+
+void Server::init()
+{
+       infostream << "Server created for gameid \"" << m_gamespec.id << "\"";
+       if (m_simple_singleplayer_mode)
+               infostream << " in simple singleplayer mode" << std::endl;
        else
-               infostream<<std::endl;
-       infostream<<"- world:  "<<m_path_world<<std::endl;
-       infostream<<"- game:   "<<m_gamespec.path<<std::endl;
+               infostream << std::endl;
+       infostream << "- world:  " << m_path_world << std::endl;
+       infostream << "- game:   " << m_gamespec.path << std::endl;
 
        // Create world if it doesn't exist
-       if(!loadGameConfAndInitWorld(m_path_world, m_gamespec))
+       if (!loadGameConfAndInitWorld(m_path_world, m_gamespec))
                throw ServerError("Failed to initialize world");
 
        // Create server thread
@@ -203,48 +329,27 @@ Server::Server(
        std::string ban_path = m_path_world + DIR_DELIM "ipban.txt";
        m_banmanager = new BanManager(ban_path);
 
-       ServerModConfiguration modconf(m_path_world);
-       m_mods = modconf.getMods();
-       std::vector<ModSpec> unsatisfied_mods = modconf.getUnsatisfiedMods();
+       m_modmgr = std::unique_ptr<ServerModManager>(new ServerModManager(m_path_world));
+       std::vector<ModSpec> unsatisfied_mods = m_modmgr->getUnsatisfiedMods();
        // complain about mods with unsatisfied dependencies
-       if (!modconf.isConsistent()) {
-               modconf.printUnsatisfiedModsError();
+       if (!m_modmgr->isConsistent()) {
+               m_modmgr->printUnsatisfiedModsError();
        }
 
        //lock environment
        MutexAutoLock envlock(m_env_mutex);
 
        // Create the Map (loads map_meta.txt, overriding configured mapgen params)
-       ServerMap *servermap = new ServerMap(path_world, this, m_emerge);
+       ServerMap *servermap = new ServerMap(m_path_world, this, m_emerge);
 
        // Initialize scripting
-       infostream<<"Server: Initializing Lua"<<std::endl;
+       infostream << "Server: Initializing Lua" << std::endl;
 
        m_script = new ServerScripting(this);
 
        m_script->loadMod(getBuiltinLuaPath() + DIR_DELIM "init.lua", BUILTIN_MOD_NAME);
 
-       // Print mods
-       infostream << "Server: Loading mods: ";
-       for (std::vector<ModSpec>::const_iterator i = m_mods.begin();
-                       i != m_mods.end(); ++i) {
-               infostream << (*i).name << " ";
-       }
-       infostream << std::endl;
-       // Load and run "mod" scripts
-       for (std::vector<ModSpec>::const_iterator it = m_mods.begin();
-                       it != m_mods.end(); ++it) {
-               const ModSpec &mod = *it;
-               if (!string_allowed(mod.name, MODNAME_ALLOWED_CHARS)) {
-                       throw ModError("Error loading mod \"" + mod.name +
-                               "\": Mod name does not follow naming conventions: "
-                               "Only characters [a-z0-9_] are allowed.");
-               }
-               std::string script_path = mod.path + DIR_DELIM + "init.lua";
-               infostream << "  [" << padStringRight(mod.name, 12) << "] [\""
-                               << script_path << "\"]" << std::endl;
-               m_script->loadMod(script_path, mod.name);
-       }
+       m_modmgr->loadMods(m_script);
 
        // Read Textures and calculate sha1 sums
        fillMediaCache();
@@ -253,9 +358,11 @@ Server::Server(
        m_nodedef->updateAliases(m_itemdef);
 
        // Apply texture overrides from texturepack/override.txt
-       std::string texture_path = g_settings->get("texture_path");
-       if (!texture_path.empty() && fs::IsDir(texture_path))
-               m_nodedef->applyTextureOverrides(texture_path + DIR_DELIM + "override.txt");
+       std::vector<std::string> paths;
+       fs::GetRecursiveDirs(paths, g_settings->get("texture_path"));
+       fs::GetRecursiveDirs(paths, m_gamespec.path + DIR_DELIM + "textures");
+       for (const std::string &path : paths)
+               m_nodedef->applyTextureOverrides(path + DIR_DELIM + "override.txt");
 
        m_nodedef->setNodeRegistrationStatus(true);
 
@@ -279,8 +386,7 @@ Server::Server(
        // Initialize mapgens
        m_emerge->initMapgens(servermap->getMapgenParams());
 
-       m_enable_rollback_recording = g_settings->getBool("enable_rollback_recording");
-       if (m_enable_rollback_recording) {
+       if (g_settings->getBool("enable_rollback_recording")) {
                // Create rollback manager
                m_rollback = new RollbackManager(m_path_world, this);
        }
@@ -291,80 +397,12 @@ Server::Server(
        // Register us to receive map edit events
        servermap->addEventReceiver(this);
 
-       // If file exists, load environment metadata
-       if (fs::PathExists(m_path_world + DIR_DELIM "env_meta.txt")) {
-               infostream << "Server: Loading environment metadata" << std::endl;
-               m_env->loadMeta();
-       } else {
-               m_env->loadDefaultMeta();
-       }
+       m_env->loadMeta();
 
        m_liquid_transform_every = g_settings->getFloat("liquid_update");
        m_max_chatmessage_length = g_settings->getU16("chat_message_max_size");
-       m_csm_flavour_limits = g_settings->getU64("csm_flavour_limits");
-       m_csm_noderange_limit = g_settings->getU32("csm_flavour_noderange_limit");
-}
-
-Server::~Server()
-{
-       infostream<<"Server destructing"<<std::endl;
-
-       // Send shutdown message
-       SendChatMessage(PEER_ID_INEXISTENT, ChatMessage(CHATMESSAGE_TYPE_ANNOUNCE,
-                       L"*** Server shutting down"));
-
-       {
-               MutexAutoLock envlock(m_env_mutex);
-
-               // Execute script shutdown hooks
-               m_script->on_shutdown();
-
-               infostream << "Server: Saving players" << std::endl;
-               m_env->saveLoadedPlayers();
-
-               infostream << "Server: Kicking players" << std::endl;
-               std::string kick_msg;
-               bool reconnect = false;
-               if (getShutdownRequested()) {
-                       reconnect = m_shutdown_ask_reconnect;
-                       kick_msg = m_shutdown_msg;
-               }
-               if (kick_msg.empty()) {
-                       kick_msg = g_settings->get("kick_msg_shutdown");
-               }
-               m_env->kickAllPlayers(SERVER_ACCESSDENIED_SHUTDOWN,
-                       kick_msg, reconnect);
-
-               infostream << "Server: Saving environment metadata" << std::endl;
-               m_env->saveMeta();
-       }
-
-       // Stop threads
-       stop();
-       delete m_thread;
-
-       // stop all emerge threads before deleting players that may have
-       // requested blocks to be emerged
-       m_emerge->stopThreads();
-
-       // Delete things in the reverse order of creation
-       delete m_emerge;
-       delete m_env;
-       delete m_rollback;
-       delete m_banmanager;
-       delete m_event;
-       delete m_itemdef;
-       delete m_nodedef;
-       delete m_craftdef;
-
-       // Deinitialize scripting
-       infostream<<"Server: Deinitializing scripting"<<std::endl;
-       delete m_script;
-
-       // Delete detached inventories
-       for (auto &detached_inventory : m_detached_inventories) {
-               delete detached_inventory.second;
-       }
+       m_csm_restriction_flags = g_settings->getU64("csm_restriction_flags");
+       m_csm_restriction_noderange = g_settings->getU32("csm_restriction_noderange");
 }
 
 void Server::start()
@@ -383,7 +421,7 @@ void Server::start()
        m_thread->start();
 
        // ASCII art for the win!
-       actionstream
+       std::cerr
                << "        .__               __                   __   " << std::endl
                << "  _____ |__| ____   _____/  |_  ____   _______/  |_ " << std::endl
                << " /     \\|  |/    \\_/ __ \\   __\\/ __ \\ /  ___/\\   __\\" << std::endl
@@ -549,8 +587,7 @@ void Server::AsyncRunStep(bool initial_step)
                /*
                        Set the modified blocks unsent for all the clients
                */
-               if(!modified_blocks.empty())
-               {
+               if (!modified_blocks.empty()) {
                        SetBlocksNotSent(modified_blocks);
                }
        }
@@ -572,7 +609,7 @@ void Server::AsyncRunStep(bool initial_step)
                                        m_lag,
                                        m_gamespec.id,
                                        Mapgen::getMapgenName(m_emerge->mgparams->mgtype),
-                                       m_mods,
+                                       m_modmgr->getMods(),
                                        m_dedicated);
                        counter = 0.01;
                }
@@ -848,13 +885,13 @@ void Server::AsyncRunStep(bool initial_step)
                        case MEET_BLOCK_NODE_METADATA_CHANGED:
                                infostream << "Server: MEET_BLOCK_NODE_METADATA_CHANGED" << std::endl;
                                                prof.add("MEET_BLOCK_NODE_METADATA_CHANGED", 1);
-                                               setBlockNotSent(event->p);
+                                               m_clients.markBlockposAsNotSent(event->p);
                                break;
                        case MEET_OTHER:
                                infostream << "Server: MEET_OTHER" << std::endl;
                                prof.add("MEET_OTHER", 1);
                                for (const v3s16 &modified_block : event->modified_blocks) {
-                                       setBlockNotSent(modified_block);
+                                       m_clients.markBlockposAsNotSent(modified_block);
                                }
                                break;
                        default:
@@ -937,38 +974,7 @@ void Server::AsyncRunStep(bool initial_step)
                }
        }
 
-       // Timed shutdown
-       static const float shutdown_msg_times[] =
-       {
-               1, 2, 3, 4, 5, 10, 15, 20, 25, 30, 45, 60, 120, 180, 300, 600, 1200, 1800, 3600
-       };
-
-       if (m_shutdown_timer > 0.0f) {
-               // Automated messages
-               if (m_shutdown_timer < shutdown_msg_times[ARRLEN(shutdown_msg_times) - 1]) {
-                       for (u16 i = 0; i < ARRLEN(shutdown_msg_times) - 1; i++) {
-                               // If shutdown timer matches an automessage, shot it
-                               if (m_shutdown_timer > shutdown_msg_times[i] &&
-                                       m_shutdown_timer - dtime < shutdown_msg_times[i]) {
-                                       std::wstringstream ws;
-
-                                       ws << L"*** Server shutting down in "
-                                               << duration_to_string(myround(m_shutdown_timer - dtime)).c_str()
-                                               << ".";
-
-                                       infostream << wide_to_utf8(ws.str()).c_str() << std::endl;
-                                       SendChatMessage(PEER_ID_INEXISTENT, ws.str());
-                                       break;
-                               }
-                       }
-               }
-
-               m_shutdown_timer -= dtime;
-               if (m_shutdown_timer < 0.0f) {
-                       m_shutdown_timer = 0.0f;
-                       m_shutdown_requested = true;
-               }
-       }
+       m_shutdown_state.tick(dtime, this);
 }
 
 void Server::Receive()
@@ -1015,7 +1021,7 @@ PlayerSAO* Server::StageTwoClientInit(session_t peer_id)
 
        // If failed, cancel
        if (!playersao || !player) {
-               if (player && player->peer_id != 0) {
+               if (player && player->getPeerId() != PEER_ID_INEXISTENT) {
                        actionstream << "Server: Failed to emerge player \"" << playername
                                        << "\" (player allocated to an another client)" << std::endl;
                        DenyAccess_Legacy(peer_id, L"Another client is connected with this "
@@ -1047,7 +1053,8 @@ PlayerSAO* Server::StageTwoClientInit(session_t peer_id)
        if (playersao->isDead())
                SendDeathscreen(peer_id, false, v3f(0,0,0));
        else
-               SendPlayerHPOrDie(playersao);
+               SendPlayerHPOrDie(playersao,
+                               PlayerHPChangeReason(PlayerHPChangeReason::SET_HP));
 
        // Send Breath
        SendPlayerBreath(playersao);
@@ -1057,7 +1064,7 @@ PlayerSAO* Server::StageTwoClientInit(session_t peer_id)
                // Send information about server to player in chat
                SendChatMessage(peer_id, ChatMessage(CHATMESSAGE_TYPE_SYSTEM, getStatusString()));
        }
-       Address addr = getPeerAddress(player->peer_id);
+       Address addr = getPeerAddress(player->getPeerId());
        std::string ip_str = addr.serializeString();
        actionstream<<player->getName() <<" [" << ip_str << "] joins game. " << std::endl;
        /*
@@ -1177,9 +1184,7 @@ void Server::setTimeOfDay(u32 time)
 
 void Server::onMapEditEvent(MapEditEvent *event)
 {
-       if(m_ignore_map_edit_events)
-               return;
-       if(m_ignore_map_edit_events_area.contains(event->getArea()))
+       if (m_ignore_map_edit_events_area.contains(event->getArea()))
                return;
        MapEditEvent *e = event->clone();
        m_unsent_map_edit_queue.push(e);
@@ -1253,7 +1258,7 @@ void Server::setInventoryModified(const InventoryLocation &loc, bool playerSend)
                if (block)
                        block->raiseModified(MOD_STATE_WRITE_NEEDED);
 
-               setBlockNotSent(blockpos);
+               m_clients.markBlockposAsNotSent(blockpos);
        }
                break;
        case InventoryLocation::DETACHED:
@@ -1410,7 +1415,7 @@ void Server::SendMovement(session_t peer_id)
        Send(&pkt);
 }
 
-void Server::SendPlayerHPOrDie(PlayerSAO *playersao)
+void Server::SendPlayerHPOrDie(PlayerSAO *playersao, const PlayerHPChangeReason &reason)
 {
        if (!g_settings->getBool("enable_damage"))
                return;
@@ -1421,7 +1426,7 @@ void Server::SendPlayerHPOrDie(PlayerSAO *playersao)
        if (is_alive)
                SendPlayerHP(peer_id);
        else
-               DiePlayer(peer_id);
+               DiePlayer(peer_id, reason);
 }
 
 void Server::SendHP(session_t peer_id, u16 hp)
@@ -1492,7 +1497,7 @@ void Server::SendItemDef(session_t peer_id,
 }
 
 void Server::SendNodeDef(session_t peer_id,
-               INodeDefManager *nodedef, u16 protocol_version)
+       const NodeDefManager *nodedef, u16 protocol_version)
 {
        NetworkPacket pkt(TOCLIENT_NODEDEF, 0, peer_id);
 
@@ -1562,8 +1567,10 @@ void Server::SendShowFormspecMessage(session_t peer_id, const std::string &forms
        NetworkPacket pkt(TOCLIENT_SHOW_FORMSPEC, 0 , peer_id);
        if (formspec.empty()){
                //the client should close the formspec
+               m_formspec_state_data.erase(peer_id);
                pkt.putLongString("");
        } else {
+               m_formspec_state_data[peer_id] = formname;
                pkt.putLongString(FORMSPEC_VERSION_STRING + formspec);
        }
        pkt << formname;
@@ -1764,17 +1771,11 @@ void Server::SendSetSky(session_t peer_id, const video::SColor &bgcolor,
        Send(&pkt);
 }
 
-void Server::SendCloudParams(session_t peer_id, float density,
-               const video::SColor &color_bright,
-               const video::SColor &color_ambient,
-               float height,
-               float thickness,
-               const v2f &speed)
+void Server::SendCloudParams(session_t peer_id, const CloudParams &params)
 {
        NetworkPacket pkt(TOCLIENT_CLOUD_PARAMS, 0, peer_id);
-       pkt << density << color_bright << color_ambient
-                       << height << thickness << speed;
-
+       pkt << params.density << params.color_bright << params.color_ambient
+                       << params.height << params.thickness << params.speed;
        Send(&pkt);
 }
 
@@ -1872,7 +1873,7 @@ void Server::SendPlayerPrivileges(session_t peer_id)
 {
        RemotePlayer *player = m_env->getPlayer(peer_id);
        assert(player);
-       if(player->peer_id == PEER_ID_INEXISTENT)
+       if(player->getPeerId() == PEER_ID_INEXISTENT)
                return;
 
        std::set<std::string> privs;
@@ -1892,7 +1893,7 @@ void Server::SendPlayerInventoryFormspec(session_t peer_id)
 {
        RemotePlayer *player = m_env->getPlayer(peer_id);
        assert(player);
-       if(player->peer_id == PEER_ID_INEXISTENT)
+       if (player->getPeerId() == PEER_ID_INEXISTENT)
                return;
 
        NetworkPacket pkt(TOCLIENT_INVENTORY_FORMSPEC, 0, peer_id);
@@ -1900,6 +1901,18 @@ void Server::SendPlayerInventoryFormspec(session_t peer_id)
        Send(&pkt);
 }
 
+void Server::SendPlayerFormspecPrepend(session_t peer_id)
+{
+       RemotePlayer *player = m_env->getPlayer(peer_id);
+       assert(player);
+       if (player->getPeerId() == PEER_ID_INEXISTENT)
+               return;
+
+       NetworkPacket pkt(TOCLIENT_FORMSPEC_PREPEND, 0, peer_id);
+       pkt << FORMSPEC_VERSION_STRING + player->formspec_prepend;
+       Send(&pkt);
+}
+
 u32 Server::SendActiveObjectRemoveAdd(session_t peer_id, const std::string &datas)
 {
        NetworkPacket pkt(TOCLIENT_ACTIVE_OBJECT_REMOVE_ADD, datas.size(), peer_id);
@@ -1921,11 +1934,11 @@ void Server::SendActiveObjectMessages(session_t peer_id, const std::string &data
                        &pkt, reliable);
 }
 
-void Server::SendCSMFlavourLimits(session_t peer_id)
+void Server::SendCSMRestrictionFlags(session_t peer_id)
 {
-       NetworkPacket pkt(TOCLIENT_CSM_FLAVOUR_LIMITS,
-               sizeof(m_csm_flavour_limits) + sizeof(m_csm_noderange_limit), peer_id);
-       pkt << m_csm_flavour_limits << m_csm_noderange_limit;
+       NetworkPacket pkt(TOCLIENT_CSM_RESTRICTION_FLAGS,
+               sizeof(m_csm_restriction_flags) + sizeof(m_csm_restriction_noderange), peer_id);
+       pkt << m_csm_restriction_flags << m_csm_restriction_noderange;
        Send(&pkt);
 }
 
@@ -1948,12 +1961,12 @@ s32 Server::playSound(const SimpleSoundSpec &spec,
                                        <<"\" not found"<<std::endl;
                        return -1;
                }
-               if(player->peer_id == PEER_ID_INEXISTENT){
+               if (player->getPeerId() == PEER_ID_INEXISTENT) {
                        infostream<<"Server::playSound: Player \""<<params.to_player
                                        <<"\" not connected"<<std::endl;
                        return -1;
                }
-               dst_clients.push_back(player->peer_id);
+               dst_clients.push_back(player->getPeerId());
        } else {
                std::vector<session_t> clients = m_clients.getClientIDs();
 
@@ -2142,22 +2155,9 @@ void Server::sendAddNode(v3s16 p, MapNode n, u16 ignore_id,
        }
 }
 
-void Server::setBlockNotSent(v3s16 p)
-{
-       std::vector<session_t> clients = m_clients.getClientIDs();
-       m_clients.lock();
-       for (const session_t i : clients) {
-               RemoteClient *client = m_clients.lockedGetClientNoEx(i);
-               client->SetBlockNotSent(p);
-       }
-       m_clients.unlock();
-}
-
 void Server::SendBlockNoLock(session_t peer_id, MapBlock *block, u8 ver,
                u16 net_proto_version)
 {
-       v3s16 p = block->getPos();
-
        /*
                Create a packet with the block in the right format
        */
@@ -2169,7 +2169,7 @@ void Server::SendBlockNoLock(session_t peer_id, MapBlock *block, u8 ver,
 
        NetworkPacket pkt(TOCLIENT_BLOCKDATA, 2 + 2 + 2 + 2 + s.size(), peer_id);
 
-       pkt << p;
+       pkt << block->getPos();
        pkt.putRawString(s.c_str(), s.size());
        Send(&pkt);
 }
@@ -2246,14 +2246,9 @@ void Server::fillMediaCache()
 
        // Collect all media file paths
        std::vector<std::string> paths;
-       for (const ModSpec &mod : m_mods) {
-               paths.push_back(mod.path + DIR_DELIM + "textures");
-               paths.push_back(mod.path + DIR_DELIM + "sounds");
-               paths.push_back(mod.path + DIR_DELIM + "media");
-               paths.push_back(mod.path + DIR_DELIM + "models");
-               paths.push_back(mod.path + DIR_DELIM + "locale");
-       }
-       paths.push_back(porting::path_user + DIR_DELIM + "textures" + DIR_DELIM + "server");
+       m_modmgr->getModsMediaPaths(paths);
+       fs::GetRecursiveDirs(paths, m_gamespec.path + DIR_DELIM + "textures");
+       fs::GetRecursiveDirs(paths, porting::path_user + DIR_DELIM + "textures" + DIR_DELIM + "server");
 
        // Collect media file information from paths into cache
        for (const std::string &mediapath : paths) {
@@ -2501,7 +2496,7 @@ void Server::sendDetachedInventory(const std::string &name, session_t peer_id)
                        return m_clients.sendToAll(&pkt);
                RemotePlayer *p = m_env->getPlayer(check.c_str());
                if (p)
-                       m_clients.send(p->peer_id, 0, &pkt, true);
+                       m_clients.send(p->getPeerId(), 0, &pkt, true);
        } else {
                if (check.empty() || getPlayerName(peer_id) == check)
                        Send(&pkt);
@@ -2521,7 +2516,7 @@ void Server::sendDetachedInventories(session_t peer_id)
        Something random
 */
 
-void Server::DiePlayer(session_t peer_id)
+void Server::DiePlayer(session_t peer_id, const PlayerHPChangeReason &reason)
 {
        PlayerSAO *playersao = getPlayerSAO(peer_id);
        // In some rare cases this can be NULL -- if the player is disconnected
@@ -2533,10 +2528,11 @@ void Server::DiePlayer(session_t peer_id)
                        << playersao->getPlayer()->getName()
                        << " dies" << std::endl;
 
-       playersao->setHP(0);
+       playersao->setHP(0, reason);
+       playersao->clearParentAttachment();
 
        // Trigger scripted stuff
-       m_script->on_dieplayer(playersao);
+       m_script->on_dieplayer(playersao, reason);
 
        SendPlayerHP(peer_id);
        SendDeathscreen(peer_id, false, v3f(0,0,0));
@@ -2551,7 +2547,8 @@ void Server::RespawnPlayer(session_t peer_id)
                        << playersao->getPlayer()->getName()
                        << " respawns" << std::endl;
 
-       playersao->setHP(playersao->accessObjectProperties()->hp_max);
+       playersao->setHP(playersao->accessObjectProperties()->hp_max,
+                       PlayerHPChangeReason(PlayerHPChangeReason::RESPAWN));
        playersao->setBreath(playersao->accessObjectProperties()->breath_max);
 
        bool repositioned = m_script->on_respawnplayer(playersao);
@@ -2650,6 +2647,9 @@ void Server::DeleteClient(session_t peer_id, ClientDeletionReason reason)
                                ++i;
                }
 
+               // clear formspec info so the next client can't abuse the current state
+               m_formspec_state_data.erase(peer_id);
+
                RemotePlayer *player = m_env->getPlayer(peer_id);
 
                /* Run scripts and remove from environment */
@@ -2657,6 +2657,9 @@ void Server::DeleteClient(session_t peer_id, ClientDeletionReason reason)
                        PlayerSAO *playersao = player->getPlayerSAO();
                        assert(playersao);
 
+                       playersao->clearChildAttachments();
+                       playersao->clearParentAttachment();
+
                        // inform connected clients
                        NetworkPacket notice(TOCLIENT_UPDATE_PLAYER_LIST, 0, PEER_ID_INEXISTENT);
                        // (u16) 1 + std::string represents a vector serialization representation
@@ -2710,6 +2713,10 @@ void Server::DeleteClient(session_t peer_id, ClientDeletionReason reason)
 
 void Server::UpdateCrafting(RemotePlayer *player)
 {
+       InventoryList *clist = player->inventory.getList("craft");
+       if (!clist || clist->getSize() == 0)
+               return;
+
        // Get a preview for crafting
        ItemStack preview;
        InventoryLocation loc;
@@ -2717,13 +2724,13 @@ void Server::UpdateCrafting(RemotePlayer *player)
        std::vector<ItemStack> output_replacements;
        getCraftingResult(&player->inventory, preview, output_replacements, false, this);
        m_env->getScriptIface()->item_CraftPredict(preview, player->getPlayerSAO(),
-                       (&player->inventory)->getList("craft"), loc);
+                       clist, loc);
 
-       // Put the new preview in
        InventoryList *plist = player->inventory.getList("craftpreview");
-       sanity_check(plist);
-       sanity_check(plist->getSize() >= 1);
-       plist->changeItem(0, preview);
+       if (plist && plist->getSize() >= 1) {
+               // Put the new preview in
+               plist->changeItem(0, preview);
+       }
 }
 
 void Server::handleChatInterfaceEvent(ChatEvent *evt)
@@ -2765,7 +2772,7 @@ std::wstring Server::handleChat(const std::string &name, const std::wstring &wna
                                return ws.str();
                        }
                        case RPLAYER_CHATRESULT_KICK:
-                               DenyAccess_Legacy(player->peer_id,
+                               DenyAccess_Legacy(player->getPeerId(),
                                                L"You have been kicked due to message flooding.");
                                return L"";
                        case RPLAYER_CHATRESULT_OK:
@@ -2818,7 +2825,9 @@ std::wstring Server::handleChat(const std::string &name, const std::wstring &wna
                if they are using protocol version >= 29
        */
 
-       session_t peer_id_to_avoid_sending = (player ? player->peer_id : PEER_ID_INEXISTENT);
+       session_t peer_id_to_avoid_sending =
+               (player ? player->getPeerId() : PEER_ID_INEXISTENT);
+
        if (player && player->protocol_version >= 29)
                peer_id_to_avoid_sending = PEER_ID_INEXISTENT;
 
@@ -2935,7 +2944,7 @@ void Server::reportPrivsModified(const std::string &name)
                RemotePlayer *player = m_env->getPlayer(name.c_str());
                if (!player)
                        return;
-               SendPlayerPrivileges(player->peer_id);
+               SendPlayerPrivileges(player->getPeerId());
                PlayerSAO *sao = player->getPlayerSAO();
                if(!sao)
                        return;
@@ -2950,7 +2959,15 @@ void Server::reportInventoryFormspecModified(const std::string &name)
        RemotePlayer *player = m_env->getPlayer(name.c_str());
        if (!player)
                return;
-       SendPlayerInventoryFormspec(player->peer_id);
+       SendPlayerInventoryFormspec(player->getPeerId());
+}
+
+void Server::reportFormspecPrependModified(const std::string &name)
+{
+       RemotePlayer *player = m_env->getPlayer(name.c_str());
+       if (!player)
+               return;
+       SendPlayerFormspecPrepend(player->getPeerId());
 }
 
 void Server::setIpBanned(const std::string &ip, const std::string &name)
@@ -2983,10 +3000,10 @@ void Server::notifyPlayer(const char *name, const std::wstring &msg)
                return;
        }
 
-       if (player->peer_id == PEER_ID_INEXISTENT)
+       if (player->getPeerId() == PEER_ID_INEXISTENT)
                return;
 
-       SendChatMessage(player->peer_id, ChatMessage(msg));
+       SendChatMessage(player->getPeerId(), ChatMessage(msg));
 }
 
 bool Server::showFormspec(const char *playername, const std::string &formspec,
@@ -3000,7 +3017,7 @@ bool Server::showFormspec(const char *playername, const std::string &formspec,
        if (!player)
                return false;
 
-       SendShowFormspecMessage(player->peer_id, formspec, formname);
+       SendShowFormspecMessage(player->getPeerId(), formspec, formname);
        return true;
 }
 
@@ -3011,7 +3028,7 @@ u32 Server::hudAdd(RemotePlayer *player, HudElement *form)
 
        u32 id = player->addHud(form);
 
-       SendHUDAdd(player->peer_id, id, form);
+       SendHUDAdd(player->getPeerId(), id, form);
 
        return id;
 }
@@ -3027,7 +3044,7 @@ bool Server::hudRemove(RemotePlayer *player, u32 id) {
 
        delete todel;
 
-       SendHUDRemove(player->peer_id, id);
+       SendHUDRemove(player->getPeerId(), id);
        return true;
 }
 
@@ -3036,7 +3053,7 @@ bool Server::hudChange(RemotePlayer *player, u32 id, HudElementStat stat, void *
        if (!player)
                return false;
 
-       SendHUDChange(player->peer_id, id, stat, data);
+       SendHUDChange(player->getPeerId(), id, stat, data);
        return true;
 }
 
@@ -3045,7 +3062,7 @@ bool Server::hudSetFlags(RemotePlayer *player, u32 flags, u32 mask)
        if (!player)
                return false;
 
-       SendHUDSetFlags(player->peer_id, flags, mask);
+       SendHUDSetFlags(player->getPeerId(), flags, mask);
        player->hud_flags &= ~mask;
        player->hud_flags |= flags;
 
@@ -3069,29 +3086,17 @@ bool Server::hudSetHotbarItemcount(RemotePlayer *player, s32 hotbar_itemcount)
        player->setHotbarItemcount(hotbar_itemcount);
        std::ostringstream os(std::ios::binary);
        writeS32(os, hotbar_itemcount);
-       SendHUDSetParam(player->peer_id, HUD_PARAM_HOTBAR_ITEMCOUNT, os.str());
+       SendHUDSetParam(player->getPeerId(), HUD_PARAM_HOTBAR_ITEMCOUNT, os.str());
        return true;
 }
 
-s32 Server::hudGetHotbarItemcount(RemotePlayer *player) const
-{
-       return player->getHotbarItemcount();
-}
-
 void Server::hudSetHotbarImage(RemotePlayer *player, std::string name)
 {
        if (!player)
                return;
 
        player->setHotbarImage(name);
-       SendHUDSetParam(player->peer_id, HUD_PARAM_HOTBAR_IMAGE, name);
-}
-
-std::string Server::hudGetHotbarImage(RemotePlayer *player)
-{
-       if (!player)
-               return "";
-       return player->getHotbarImage();
+       SendHUDSetParam(player->getPeerId(), HUD_PARAM_HOTBAR_IMAGE, name);
 }
 
 void Server::hudSetHotbarSelectedImage(RemotePlayer *player, std::string name)
@@ -3100,12 +3105,7 @@ void Server::hudSetHotbarSelectedImage(RemotePlayer *player, std::string name)
                return;
 
        player->setHotbarSelectedImage(name);
-       SendHUDSetParam(player->peer_id, HUD_PARAM_HOTBAR_SELECTED_IMAGE, name);
-}
-
-const std::string& Server::hudGetHotbarSelectedImage(RemotePlayer *player) const
-{
-       return player->getHotbarSelectedImage();
+       SendHUDSetParam(player->getPeerId(), HUD_PARAM_HOTBAR_SELECTED_IMAGE, name);
 }
 
 Address Server::getPeerAddress(session_t peer_id)
@@ -3113,54 +3113,36 @@ Address Server::getPeerAddress(session_t peer_id)
        return m_con->GetPeerAddress(peer_id);
 }
 
-bool Server::setLocalPlayerAnimations(RemotePlayer *player,
+void Server::setLocalPlayerAnimations(RemotePlayer *player,
                v2s32 animation_frames[4], f32 frame_speed)
 {
-       if (!player)
-               return false;
-
+       sanity_check(player);
        player->setLocalAnimations(animation_frames, frame_speed);
-       SendLocalPlayerAnimations(player->peer_id, animation_frames, frame_speed);
-       return true;
+       SendLocalPlayerAnimations(player->getPeerId(), animation_frames, frame_speed);
 }
 
-bool Server::setPlayerEyeOffset(RemotePlayer *player, v3f first, v3f third)
+void Server::setPlayerEyeOffset(RemotePlayer *player, const v3f &first, const v3f &third)
 {
-       if (!player)
-               return false;
-
+       sanity_check(player);
        player->eye_offset_first = first;
        player->eye_offset_third = third;
-       SendEyeOffset(player->peer_id, first, third);
-       return true;
+       SendEyeOffset(player->getPeerId(), first, third);
 }
 
-bool Server::setSky(RemotePlayer *player, const video::SColor &bgcolor,
+void Server::setSky(RemotePlayer *player, const video::SColor &bgcolor,
        const std::string &type, const std::vector<std::string> &params,
        bool &clouds)
 {
-       if (!player)
-               return false;
-
+       sanity_check(player);
        player->setSky(bgcolor, type, params, clouds);
-       SendSetSky(player->peer_id, bgcolor, type, params, clouds);
-       return true;
+       SendSetSky(player->getPeerId(), bgcolor, type, params, clouds);
 }
 
-bool Server::setClouds(RemotePlayer *player, float density,
-       const video::SColor &color_bright,
-       const video::SColor &color_ambient,
-       float height,
-       float thickness,
-       const v2f &speed)
+void Server::setClouds(RemotePlayer *player, const CloudParams &params)
 {
-       if (!player)
-               return false;
-
-       SendCloudParams(player->peer_id, density,
-                       color_bright, color_ambient, height,
-                       thickness, speed);
-       return true;
+       sanity_check(player);
+       player->setCloudParams(params);
+       SendCloudParams(player->getPeerId(), params);
 }
 
 bool Server::overrideDayNightRatio(RemotePlayer *player, bool do_override,
@@ -3170,7 +3152,7 @@ bool Server::overrideDayNightRatio(RemotePlayer *player, bool do_override,
                return false;
 
        player->overrideDayNightRatio(do_override, ratio);
-       SendOverrideDayNightRatio(player->peer_id, do_override, ratio);
+       SendOverrideDayNightRatio(player->getPeerId(), do_override, ratio);
        return true;
 }
 
@@ -3196,7 +3178,7 @@ void Server::spawnParticle(const std::string &playername, v3f pos,
                RemotePlayer *player = m_env->getPlayer(playername.c_str());
                if (!player)
                        return;
-               peer_id = player->peer_id;
+               peer_id = player->getPeerId();
                proto_ver = player->protocol_version;
        }
 
@@ -3223,7 +3205,7 @@ u32 Server::addParticleSpawner(u16 amount, float spawntime,
                RemotePlayer *player = m_env->getPlayer(playername.c_str());
                if (!player)
                        return -1;
-               peer_id = player->peer_id;
+               peer_id = player->getPeerId();
                proto_ver = player->protocol_version;
        }
 
@@ -3255,7 +3237,7 @@ void Server::deleteParticleSpawner(const std::string &playername, u32 id)
                RemotePlayer *player = m_env->getPlayer(playername.c_str());
                if (!player)
                        return;
-               peer_id = player->peer_id;
+               peer_id = player->getPeerId();
        }
 
        m_env->deleteParticleSpawner(id);
@@ -3288,7 +3270,8 @@ bool Server::rollbackRevertActions(const std::list<RollbackAction> &actions,
        ServerMap *map = (ServerMap*)(&m_env->getMap());
 
        // Fail if no actions to handle
-       if(actions.empty()){
+       if (actions.empty()) {
+               assert(log);
                log->push_back("Nothing to do.");
                return false;
        }
@@ -3329,7 +3312,7 @@ IItemDefManager *Server::getItemDefManager()
        return m_itemdef;
 }
 
-INodeDefManager *Server::getNodeDefManager()
+const NodeDefManager *Server::getNodeDefManager()
 {
        return m_nodedef;
 }
@@ -3344,17 +3327,12 @@ u16 Server::allocateUnknownNodeId(const std::string &name)
        return m_nodedef->allocateDummy(name);
 }
 
-MtEventManager *Server::getEventManager()
-{
-       return m_event;
-}
-
 IWritableItemDefManager *Server::getWritableItemDefManager()
 {
        return m_itemdef;
 }
 
-IWritableNodeDefManager *Server::getWritableNodeDefManager()
+NodeDefManager *Server::getWritableNodeDefManager()
 {
        return m_nodedef;
 }
@@ -3364,22 +3342,19 @@ IWritableCraftDefManager *Server::getWritableCraftDefManager()
        return m_craftdef;
 }
 
+const std::vector<ModSpec> & Server::getMods() const
+{
+       return m_modmgr->getMods();
+}
+
 const ModSpec *Server::getModSpec(const std::string &modname) const
 {
-       std::vector<ModSpec>::const_iterator it;
-       for (it = m_mods.begin(); it != m_mods.end(); ++it) {
-               const ModSpec &mod = *it;
-               if (mod.name == modname)
-                       return &mod;
-       }
-       return NULL;
+       return m_modmgr->getModSpec(modname);
 }
 
 void Server::getModNames(std::vector<std::string> &modlist)
 {
-       std::vector<ModSpec>::iterator it;
-       for (it = m_mods.begin(); it != m_mods.end(); ++it)
-               modlist.push_back(it->name);
+       m_modmgr->getModNames(modlist);
 }
 
 std::string Server::getBuiltinLuaPath()
@@ -3446,39 +3421,36 @@ v3f Server::findSpawnPos()
 
 void Server::requestShutdown(const std::string &msg, bool reconnect, float delay)
 {
-       m_shutdown_timer = delay;
-       m_shutdown_msg = msg;
-       m_shutdown_ask_reconnect = reconnect;
-
        if (delay == 0.0f) {
        // No delay, shutdown immediately
-               m_shutdown_requested = true;
+               m_shutdown_state.is_requested = true;
                // only print to the infostream, a chat message saying
                // "Server Shutting Down" is sent when the server destructs.
                infostream << "*** Immediate Server shutdown requested." << std::endl;
-       } else if (delay < 0.0f && m_shutdown_timer > 0.0f) {
-       // Negative delay, cancel shutdown if requested
-               m_shutdown_timer = 0.0f;
-               m_shutdown_msg = "";
-               m_shutdown_ask_reconnect = false;
-               m_shutdown_requested = false;
+       } else if (delay < 0.0f && m_shutdown_state.isTimerRunning()) {
+               // Negative delay, cancel shutdown if requested
+               m_shutdown_state.reset();
                std::wstringstream ws;
 
                ws << L"*** Server shutdown canceled.";
 
                infostream << wide_to_utf8(ws.str()).c_str() << std::endl;
                SendChatMessage(PEER_ID_INEXISTENT, ws.str());
+               // m_shutdown_* are already handled, skip.
+               return;
        } else if (delay > 0.0f) {
        // Positive delay, tell the clients when the server will shut down
                std::wstringstream ws;
 
                ws << L"*** Server shutting down in "
-                               << duration_to_string(myround(m_shutdown_timer)).c_str()
+                               << duration_to_string(myround(delay)).c_str()
                                << ".";
 
                infostream << wide_to_utf8(ws.str()).c_str() << std::endl;
                SendChatMessage(PEER_ID_INEXISTENT, ws.str());
        }
+
+       m_shutdown_state.trigger(delay, msg, reconnect);
 }
 
 PlayerSAO* Server::emergePlayer(const char *name, session_t peer_id, u16 proto_version)
@@ -3489,7 +3461,7 @@ PlayerSAO* Server::emergePlayer(const char *name, session_t peer_id, u16 proto_v
        RemotePlayer *player = m_env->getPlayer(name);
 
        // If player is already connected, cancel
-       if (player && player->peer_id != 0) {
+       if (player && player->getPeerId() != PEER_ID_INEXISTENT) {
                infostream<<"emergePlayer(): Player already connected"<<std::endl;
                return NULL;
        }
@@ -3566,7 +3538,7 @@ void dedicated_server_loop(Server &server, bool &kill)
                }
                server.step(steplen);
 
-               if (server.getShutdownRequested() || kill)
+               if (server.isShutdownRequested() || kill)
                        break;
 
                /*