CommonTasks.MountUp exiting early

CommonTasks.MountUp is exiting early I think.

I don't really have a way to post a log, but something like

Code:

await CommonTasks.MountUp(1);

MovementManager.MoveForwardStart();

will start casting the chocobo mount but also immediately start moving forward and canceling the mount cast.

 

Compiler wouldn't let you await outside of an async block, but here's the method that I was running into the problem:

Code:

        public async Task<bool> Fly() {
            bool moving = false;

            // can't mount continue on tree
            if (Core.Me.InCombat && !Core.Me.IsMounted) return false;

            await CommonTasks.MountUp(Convert.ToUInt32(MountId));

            MovementManager.SetFacing2D(Location);

            await CommonTasks.TakeOff();

            if (Altitude > float.MinValue) {
                if (Core.Me.Location.Y > Altitude + 1f) {
                    await CommonTasks.DescendTo(Altitude);
                } else if (Core.Me.Location.Y < Altitude - 1f) {
                    await CommonTasks.AscendTo(Altitude);
                }
            }

            while (Core.Me.Location.Distance2D(Location) > Radius) {
                if (!moving) {
                    MovementManager.MoveForwardStart();
                    moving = true;
                }
                MovementManager.SetFacing2D(Location);
                await Coroutine.Yield();
            }

            MovementManager.MoveForwardStop();

            if (LandAfter)
                await CommonTasks.Land();

            _done = true;
            return true;
        }

replacing it with a custom mount method:

Code:

        public static async Task<bool> Mount(int MountId) {
            if (MountId == 0) return true;

            if (!Core.Player.IsMounted) {
                Actionmanager.Mount(Convert.ToUInt32(MountId));
                await Coroutine.Wait(3000, () => Core.Me.IsMounted);
            }

            return true;
        }

worked