Unferried players while changing maps

This fixes the third problem described in 8f1c4f1 -- that if a ferried body's transform is modified outside of the PonderingSystem, it will not be unferried as appropriate. This does still require that any future code that modifies a body's transform also unferries the body first.

3 files changed, 36 insertions, 12 deletions

@@ -79,11 +79,7 @@ void PlayingSystem::changeMap(
   auto& transformable = game_.getEntityManager().
     getComponent<TransformableComponent>(player);
 
-  auto& animatable = game_.getEntityManager().
-    getComponent<AnimatableComponent>(player);
-
-  auto& ponderable = game_.getEntityManager().
-    getComponent<PonderableComponent>(player);
+  auto& pondering = game_.getSystemManager().getSystem<PonderingSystem>();
 
   auto& realizing = game_.getSystemManager().getSystem<RealizingSystem>();
 
@@ -105,6 +101,8 @@ void PlayingSystem::changeMap(
     playable.mapId = newMapEntity;
   }
 
+  pondering.unferry(player);
+
   transformable.x = x;
   transformable.y = y;
 
@@ -655,7 +655,7 @@ void PonderingSystem::tick(double dt)
       }
     }
 
-    // Handle ferry passengers
+    // Ferry or unferry as necessary
     if ((ponderable.type == PonderableComponent::Type::freefalling) &&
         (ponderable.grounded != oldGrounded))
     {
@@ -674,15 +674,11 @@ void PonderingSystem::tick(double dt)
       } else if (ponderable.ferried)
       {
         // The body is no longer being ferried
-        ponderable.ferried = false;
-
-        auto& ferryPonder = game_.getEntityManager().
-          getComponent<PonderableComponent>(ponderable.ferry);
-
-        ferryPonder.passengers.erase(entity);
+        unferry(entity);
       }
     }
 
+    // Update a ferry passenger's relative position
     if (ponderable.ferried)
     {
       auto& ferryTrans = game_.getEntityManager().
@@ -768,6 +764,22 @@ void PonderingSystem::initPrototype(id_type prototype)
   ponderable.passengers.clear();
 }
 
+void PonderingSystem::unferry(id_type entity)
+{
+  auto& ponderable = game_.getEntityManager().
+    getComponent<PonderableComponent>(entity);
+
+  if (ponderable.ferried)
+  {
+    ponderable.ferried = false;
+
+    auto& ferryPonder = game_.getEntityManager().
+      getComponent<PonderableComponent>(ponderable.ferry);
+
+    ferryPonder.passengers.erase(entity);
+  }
+}
+
 void PonderingSystem::processCollision(
   id_type entity,
   id_type collider,
@@ -16,8 +16,22 @@ public:
 
   void initializeBody(id_type entity, PonderableComponent::Type type);
 
+  /**
+   * Initializes a ponderable map object from its prototype data.
+   *
+   * @requires entity is ponderable
+   * @requires entity is a map object
+   */
   void initPrototype(id_type prototype);
 
+  /**
+   * Unferries an entity if it is a passenger of another entity. Use before
+   * moving a ponderable entity outside the PonderingSystem.
+   *
+   * @requires entity is ponderable
+   */
+  void unferry(id_type entity);
+
 private:
 
   struct CollisionResult