Prevent Tux from using item pocket while deactivated

fixes #3189

Author: MatusGuy

src/object/player.hpp

@@ -297,6 +297,7 @@ class Player final : public MovingObject
   inline bool is_swimboosting() const { return m_swimboosting; }
   inline bool is_water_jumping() const { return m_water_jump; }
   inline bool is_skidding() const { return m_skidding_timer.started(); }
+  inline bool is_scripting_activated() const { return !m_deactivated; }
   inline float get_swimming_angle() const { return m_swimming_angle; }
 
   /**

src/supertux/game_session.cpp

@@ -622,7 +622,9 @@ GameSession::update(float dt_sec, const Controller& controller)
 
       for (Player* player : m_currentsector->get_players())
       {
-        if (player->get_controller().pressed(Control::ITEM) && m_savegame.get_player_status().m_item_pockets.size() > 0)
+        if (player->is_active() && player->is_scripting_activated() &&
+            player->get_controller().pressed(Control::ITEM) &&
+            m_savegame.get_player_status().m_item_pockets.size() > 0)
         {
           player->get_status().give_item_from_pocket(player);
         }