Changeset 9885 for code/branches/spacestationentry/src/modules/docking

Timestamp:

Dec 9, 2013, 3:57:28 PM (11 years ago)

Author:

agermann

Message:

The level EntryTest.oxw can be deleted. Finally a nice and working solution.

Location:

code/branches/spacestationentry/src/modules/docking

Files:

• Dock.cc (modified) (2 diffs)
• Dock.h (modified) (2 diffs)
• MoveToDockingTarget.cc (modified) (1 diff)

code/branches/spacestationentry/src/modules/docking/Dock.cc

@@ -85 +85 @@
     bool Dock::undocking(bool bTriggered, BaseObject* trigger)
     {
-        // Noch lange nicht fertig (leich veraenderte Kopie von execute())
+
         PlayerTrigger* pTrigger = orxonox_cast<PlayerTrigger*>(trigger);
-                PlayerInfo* player = NULL;
-
-                // Check whether it is a player trigger and extract pawn from it
-                if(pTrigger != NULL)
-                {
-                    if(!pTrigger->isForPlayer()) {  // The PlayerTrigger is not exclusively for Pawns which means we cannot extract one.
-                        orxout(verbose, context::docking) << "Docking:execute PlayerTrigger was not triggered by a player.." << endl;
-                        return false;
-                    }
-                    player = pTrigger->getTriggeringPlayer();
-                }
-                else
-                {
-                    orxout(verbose, context::docking) << "Docking::execute Not a player trigger, can't extract pawn from it.." << endl;
-                    return false;
-                }
-                if(player == NULL)
-                {
-                    orxout(verbose, context::docking) << "Docking::execute Can't retrieve PlayerInfo from Trigger. (" << trigger->getIdentifier()->getName() << ")" << endl;
-                    return false;
-                }
-
-                if(bTriggered)
-                {
-                    cmdUndock();
-                }
-                else
-                {
-                    // Remove player from candidates list
-                    candidates_.erase(player);
-                }
-
-                return true;
+        PlayerInfo* player = NULL;
+
+        // Check whether it is a player trigger and extract pawn from it
+        if(pTrigger != NULL)
+        {
+              if(!pTrigger->isForPlayer()) {  // The PlayerTrigger is not exclusively for Pawns which means we cannot extract one.
+              orxout(verbose, context::docking) << "Docking:execute PlayerTrigger was not triggered by a player.." << endl;
+              return false;
+              }
+              player = pTrigger->getTriggeringPlayer();
+        }
+        else
+        {
+              orxout(verbose, context::docking) << "Docking::execute Not a player trigger, can't extract pawn from it.." << endl;
+              return false;
+        }
+        if(player == NULL)
+        {
+              orxout(verbose, context::docking) << "Docking::execute Can't retrieve PlayerInfo from Trigger. (" << trigger->getIdentifier()->getName() << ")" << endl;
+              return false;
+        }
+
+            if(bTriggered)
+        {
+              // Add player to this Docks candidates
+              candidates_.insert(player);
+
+              // Show docking dialog
+              this->showUndockingDialogHelper(player);
+        }
+        else
+        {
+              // Remove player from candidates list
+              candidates_.erase(player);
+        }
+
+        return true;
     }
 
@@ -166 +170 @@
     }
 
+
+    void Dock::showUndockingDialogHelper(PlayerInfo* player)
+        {
+            assert(player);
+
+            if(!player->isHumanPlayer())
+                return;
+
+            if(GameMode::isMaster())
+            {
+                if(GameMode::showsGraphics())
+                    GUIManager::showGUI("UndockingDialog");
+            }
+            else
+                callStaticNetworkFunction(Dock::showDockingDialog, player->getClientID());
+
+        }
 
     void Dock::showDockingDialogHelper(PlayerInfo* player)

code/branches/spacestationentry/src/modules/docking/Dock.h

@@ -86 +86 @@
             void dock()
                 { this->dock(HumanController::getLocalControllerSingleton()->getPlayer()); }
+            void undock()
+                { this->undock(HumanController::getLocalControllerSingleton()->getPlayer()); }
             static unsigned int getNumberOfActiveDocks();
             static Dock* getActiveDockAtIndex(unsigned int index);
@@ -96 +98 @@
             // Network functions
             void showDockingDialogHelper(PlayerInfo* player);
+            void showUndockingDialogHelper(PlayerInfo* player);
             static void showDockingDialog();
 

code/branches/spacestationentry/src/modules/docking/MoveToDockingTarget.cc

@@ -66 +66 @@
     {
         //TODO: Investigate strange things...
-        this->parent_->detach((WorldEntity*)player->getControllableEntity());
+        //this->parent_->detach((WorldEntity*)player->getControllableEntity());
 
+        //TODO: Check the issue with this detach call.
+        //I have removed the line because the detach call only caused a warning and terminated. And because I didn't find a attach call either.
+        //Didn't find the need for the line.
         this->parent_->undockingAnimationFinished(player);
         return true;