Typo in getOutgoingSequenceNumber => successfull to successful

[oweals/minetest.git] / src / network / serverpackethandler.cpp

diff --git a/src/network/serverpackethandler.cpp b/src/network/serverpackethandler.cpp
index fd3b0ac0e6543b34e28c40516cf66decfba7bb98..e01aeaa55e0d36b2058f884528a6be84fb533dae 100644 (file)
--- a/src/network/serverpackethandler.cpp
+++ b/src/network/serverpackethandler.cpp
@@ -343,7 +343,7 @@ void Server::handleCommand_Init_Legacy(NetworkPacket* pkt)
                 * respond for some time, your server was overloaded or
                 * things like that.
                 */
-               infostream << "Server::ProcessData(): Cancelling: peer "
+               infostream << "Server::ProcessData(): Canceling: peer "
                                << pkt->getPeerId() << " not found" << std::endl;
                return;
        }
@@ -388,7 +388,7 @@ void Server::handleCommand_Init_Legacy(NetworkPacket* pkt)
                DenyAccess_Legacy(pkt->getPeerId(), std::wstring(
                                L"Your client's version is not supported.\n"
                                L"Server version is ")
-                               + narrow_to_wide(minetest_version_simple) + L"."
+                               + narrow_to_wide(g_version_string) + L"."
                );
                return;
        }
@@ -436,7 +436,7 @@ void Server::handleCommand_Init_Legacy(NetworkPacket* pkt)
                DenyAccess_Legacy(pkt->getPeerId(), std::wstring(
                                L"Your client's version is not supported.\n"
                                L"Server version is ")
-                               + narrow_to_wide(minetest_version_simple) + L",\n"
+                               + narrow_to_wide(g_version_string) + L",\n"
                                + L"server's PROTOCOL_VERSION is "
                                + narrow_to_wide(itos(SERVER_PROTOCOL_VERSION_MIN))
                                + L"..."
@@ -456,7 +456,7 @@ void Server::handleCommand_Init_Legacy(NetworkPacket* pkt)
                        DenyAccess_Legacy(pkt->getPeerId(), std::wstring(
                                        L"Your client's version is not supported.\n"
                                        L"Server version is ")
-                                       + narrow_to_wide(minetest_version_simple) + L",\n"
+                                       + narrow_to_wide(g_version_string) + L",\n"
                                        + L"server's PROTOCOL_VERSION (strict) is "
                                        + narrow_to_wide(itos(LATEST_PROTOCOL_VERSION))
                                        + L", client's PROTOCOL_VERSION is "
@@ -826,7 +826,7 @@ void Server::handleCommand_PlayerPos(NetworkPacket* pkt)
 
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -842,7 +842,7 @@ void Server::handleCommand_PlayerPos(NetworkPacket* pkt)
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player object for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -904,7 +904,7 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
 {
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -913,7 +913,7 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player object for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -952,8 +952,8 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
                ma->from_inv.applyCurrentPlayer(player->getName());
                ma->to_inv.applyCurrentPlayer(player->getName());
 
-               setInventoryModified(ma->from_inv);
-               setInventoryModified(ma->to_inv);
+               setInventoryModified(ma->from_inv, false);
+               setInventoryModified(ma->to_inv, false);
 
                bool from_inv_is_current_player =
                        (ma->from_inv.type == InventoryLocation::PLAYER) &&
@@ -1006,7 +1006,7 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
 
                da->from_inv.applyCurrentPlayer(player->getName());
 
-               setInventoryModified(da->from_inv);
+               setInventoryModified(da->from_inv, false);
 
                /*
                        Disable dropping items out of craftpreview
@@ -1033,7 +1033,7 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
 
                ca->craft_inv.applyCurrentPlayer(player->getName());
 
-               setInventoryModified(ca->craft_inv);
+               setInventoryModified(ca->craft_inv, false);
 
                //bool craft_inv_is_current_player =
                //      (ca->craft_inv.type == InventoryLocation::PLAYER) &&
@@ -1052,6 +1052,8 @@ void Server::handleCommand_InventoryAction(NetworkPacket* pkt)
        a->apply(this, playersao, this);
        // Eat the action
        delete a;
+
+       SendInventory(playersao);
 }
 
 void Server::handleCommand_ChatMessage(NetworkPacket* pkt)
@@ -1074,7 +1076,7 @@ void Server::handleCommand_ChatMessage(NetworkPacket* pkt)
 
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1155,7 +1157,7 @@ void Server::handleCommand_Damage(NetworkPacket* pkt)
 
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1164,7 +1166,7 @@ void Server::handleCommand_Damage(NetworkPacket* pkt)
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player object for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1189,7 +1191,7 @@ void Server::handleCommand_Breath(NetworkPacket* pkt)
 
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1209,7 +1211,7 @@ void Server::handleCommand_Breath(NetworkPacket* pkt)
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player object for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1254,7 +1256,7 @@ void Server::handleCommand_Password(NetworkPacket* pkt)
 
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1291,7 +1293,7 @@ void Server::handleCommand_Password(NetworkPacket* pkt)
        } else {
                actionstream << player->getName() << " tries to change password but "
                                << "it fails" << std::endl;
-               SendChatMessage(pkt->getPeerId(), L"Password change failed or inavailable.");
+               SendChatMessage(pkt->getPeerId(), L"Password change failed or unavailable.");
        }
 }
 
@@ -1302,7 +1304,7 @@ void Server::handleCommand_PlayerItem(NetworkPacket* pkt)
 
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1311,7 +1313,7 @@ void Server::handleCommand_PlayerItem(NetworkPacket* pkt)
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player object for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1329,7 +1331,7 @@ void Server::handleCommand_Respawn(NetworkPacket* pkt)
 {
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1345,7 +1347,7 @@ void Server::handleCommand_Respawn(NetworkPacket* pkt)
                        << PP(player->getPosition()/BS) << std::endl;
 
        // ActiveObject is added to environment in AsyncRunStep after
-       // the previous addition has been succesfully removed
+       // the previous addition has been successfully removed
 }
 
 void Server::handleCommand_Interact(NetworkPacket* pkt)
@@ -1377,7 +1379,7 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
 
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1386,7 +1388,7 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player object for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1724,7 +1726,9 @@ void Server::handleCommand_Interact(NetworkPacket* pkt)
                if (m_script->item_OnUse(
                                item, playersao, pointed)) {
                        // Apply returned ItemStack
-                       playersao->setWieldedItem(item);
+                       if (playersao->setWieldedItem(item)) {
+                               SendInventory(playersao);
+                       }
                }
 
        } // action == 4
@@ -1778,7 +1782,7 @@ void Server::handleCommand_NodeMetaFields(NetworkPacket* pkt)
 
        Player *player = m_env->getPlayer(pkt->getPeerId());
        if (player == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!" << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());
@@ -1787,7 +1791,7 @@ void Server::handleCommand_NodeMetaFields(NetworkPacket* pkt)
 
        PlayerSAO *playersao = player->getPlayerSAO();
        if (playersao == NULL) {
-               errorstream << "Server::ProcessData(): Cancelling: "
+               errorstream << "Server::ProcessData(): Canceling: "
                                "No player object for peer_id=" << pkt->getPeerId()
                                << " disconnecting peer!"  << std::endl;
                m_con.DisconnectPeer(pkt->getPeerId());