Changeset 7862

Timestamp:

Feb 12, 2011, 11:57:28 AM (14 years ago)

Author:

landauf

Message:

added feature to Spectator which changes the keybind mode of the fire command to OnPress if the player controls a Spectator. changes mode back to OnHold if the player stops control of the Spectator (e.g. spawns) by using a helper function.

This avoids accidentally firing the weapons right after spawn.

Location:

code/trunk/src/orxonox

Files:

• controllers/HumanController.cc (modified) (2 diffs)
• worldentities/pawns/Spectator.cc (modified) (3 diffs)
• worldentities/pawns/Spectator.h (modified) (2 diffs)

code/trunk/src/orxonox/controllers/HumanController.cc

@@ -40 +40 @@
 namespace orxonox
 {
+    extern const std::string __CC_fire_name = "fire";
+
     SetConsoleCommand("HumanController", "moveFrontBack",          &HumanController::moveFrontBack ).addShortcut().setAsInputCommand();
     SetConsoleCommand("HumanController", "moveRightLeft",          &HumanController::moveRightLeft ).addShortcut().setAsInputCommand();
@@ -46 +48 @@
     SetConsoleCommand("HumanController", "rotatePitch",            &HumanController::rotatePitch   ).addShortcut().setAsInputCommand();
     SetConsoleCommand("HumanController", "rotateRoll",             &HumanController::rotateRoll    ).addShortcut().setAsInputCommand();
-    SetConsoleCommand("HumanController", "fire",                   &HumanController::fire          ).addShortcut().keybindMode(KeybindMode::OnHold);
+    SetConsoleCommand("HumanController", __CC_fire_name,           &HumanController::fire          ).addShortcut().keybindMode(KeybindMode::OnHold);
     SetConsoleCommand("HumanController", "reload",                 &HumanController::reload        ).addShortcut();
     SetConsoleCommand("HumanController", "boost",                  &HumanController::boost         ).addShortcut().keybindMode(KeybindMode::OnHold);

code/trunk/src/orxonox/worldentities/pawns/Spectator.cc

@@ -29 +29 @@
 #include "Spectator.h"
 
+#include "util/Convert.h"
 #include "core/CoreIncludes.h"
 #include "core/ConfigValueIncludes.h"
 #include "core/GameMode.h"
+#include "core/command/CommandExecutor.h"
+#include "core/command/ConsoleCommand.h"
 
 #include "tools/BillboardSet.h"
@@ -39 +42 @@
 namespace orxonox
 {
+    extern const std::string __CC_fire_name;
+
     CreateFactory(Spectator);
 
@@ -148 +153 @@
     }
 
+    /**
+        @brief Changes the keybind mode of the fire command to OnPress.
+    */
     void Spectator::startLocalHumanControl()
     {
         ControllableEntity::startLocalHumanControl();
+
+        // change keybind mode of fire command to OnPress to avoid firing after respawn
+        ModifyConsoleCommand(__CC_fire_name).keybindMode(KeybindMode::OnPress);
+    }
+
+    /**
+        @brief Changes the keybind mode of the fire command back to OnHold.
+    */
+    void Spectator::stopLocalHumanControl()
+    {
+        ControllableEntity::stopLocalHumanControl();
+
+        // change fire command to a helper function and change keybind mode to OnPress
+        // as soon as the player releases and presses the button again, the helper function will be called which changes the keybind mode back to OnHold
+        ModifyConsoleCommand(__CC_fire_name).pushFunction(&Spectator::resetFireCommand).keybindMode(KeybindMode::OnPress);
+    }
+
+    /**
+        @brief Helper function which changes the fire command back to the original function and keybind mode to OnHold.
+    */
+    void Spectator::resetFireCommand(unsigned int firemode)
+    {
+        ModifyConsoleCommand(__CC_fire_name).popFunction().keybindMode(KeybindMode::OnHold); // pop this helper function and change keybind mode
+
+        CommandExecutor::execute(__CC_fire_name + " " + multi_cast<std::string>(firemode)); // call the fire command again, this time with the real function
     }
 

code/trunk/src/orxonox/worldentities/pawns/Spectator.h

@@ -61 +61 @@
             virtual void setPlayer(PlayerInfo* player);
             virtual void startLocalHumanControl();
+            virtual void stopLocalHumanControl();
 
         private:
@@ -66 +67 @@
             void changedGreeting();
             void changedFlareVisibility();
+
+            static void resetFireCommand(unsigned int firemode);
 
             BillboardSet* greetingFlare_;