It would appear to me that the logic in this method was backwards.

There is no recent (~years) modification of this line, but at the same
time it appears that none of our products use the LocationProvider.moveBody()
method, so maybe this bug has just never been tickled.


git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@4549 542714f4-19e9-0310-aa3c-eee0fc999fb1

src/as/com/threerings/crowd/client/LocationDirector.as

@@ -443,7 +443,7 @@ public class LocationDirector extends BasicDirector
     {
         log.info("Moving at request of server [placeId=" + placeId + "].");
 
-        if (movePending()) {
+        if (!movePending()) {
             // clear out our old place information
             mayLeavePlace();
             didLeavePlace();

src/java/com/threerings/crowd/client/LocationDirector.java

@@ -459,7 +459,7 @@ public class LocationDirector extends BasicDirector
     {
         Log.info("Moving at request of server [placeId=" + placeId + "].");
 
-        if (movePending()) {
+        if (!movePending()) {
             // clear out our old place information
             mayLeavePlace();
             didLeavePlace();