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.

1 files changed, 19 insertions, 7 deletions

@@ -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,