Various fixes

1 files changed, 36 insertions, 32 deletions

diff --git a/src/koopatlas/pathmanager.cpp b/src/koopatlas/pathmanager.cpp
index 300e628..4ae0b44 100644
--- a/src/koopatlas/pathmanager.cpp
+++ b/src/koopatlas/pathmanager.cpp
@@ -11,7 +11,6 @@ void dWMPathManager_c::setup() {
 	timer = 0.0;
 	currentPath = 0;
 	reverseThroughPath = false;
-	beginWait = 120;
 
 	pathLayer = dScKoopatlas_c::instance->mapData.pathLayer;
 
@@ -285,11 +284,6 @@ bool dWMPathManager_c::evaluateUnlockCondition(u8 *&in, SaveBlock *save, int sta
 
 
 void dWMPathManager_c::execute() {
-	if (beginWait > 0) {
-		beginWait--;
-		return;
-	}
-
 	if (isEnteringLevel) {
 		if (levelStartWait > 0) {
 			levelStartWait--;
@@ -300,37 +294,14 @@ void dWMPathManager_c::execute() {
 		return;
 	}
 
-	int nowPressed = Remocon_GetPressed(GetActiveRemocon());
-
-	if (isMoving) {
-		moveThroughPath();
-	} else {
-		// Left, right, up, down
-		int pressedDir = -1;
-		if (nowPressed & WPAD_LEFT) pressedDir = 0;
-		else if (nowPressed & WPAD_RIGHT) pressedDir = 1;
-		else if (nowPressed & WPAD_UP) pressedDir = 2;
-		else if (nowPressed & WPAD_DOWN) pressedDir = 3;
-		else if (nowPressed & WPAD_TWO)
-			activatePoint();
-
-		if (pressedDir >= 0) {
-			if (canUseExit(currentNode->exits[pressedDir])) {
-				startMovementTo(currentNode->exits[pressedDir]);
-			} else {
-				// TODO: maybe remove this? got to see how it looks
-				static u16 directions[] = {-0x4000,0x4000,-0x7FFF,0};
-				daWMPlayer_c::instance->rot.y = directions[pressedDir];
-			}
-		}
-	}
-
 	// handle path fading
 	if (countdownToFadeIn > 0) {
 		countdownToFadeIn--;
 		if (countdownToFadeIn <= 0) {
 			unlockingAlpha = 0;
 			MapSoundPlayer(SoundRelatedClass, SE_SYS_NEW_POINT, 1);
+		} else {
+			return;
 		}
 	}
 
@@ -355,6 +326,39 @@ void dWMPathManager_c::execute() {
 			// we've reached the end
 			unlockingAlpha = -1;
 			MapSoundPlayer(SoundRelatedClass, SE_SYS_NEW_POINT_END, 1);
+			waitAfterUnlock = 15;
+		}
+
+		return;
+	}
+
+	if (waitAfterUnlock > 0) {
+		waitAfterUnlock--;
+		return;
+	}
+
+	int nowPressed = Remocon_GetPressed(GetActiveRemocon());
+
+	if (isMoving) {
+		moveThroughPath();
+	} else {
+		// Left, right, up, down
+		int pressedDir = -1;
+		if (nowPressed & WPAD_LEFT) pressedDir = 0;
+		else if (nowPressed & WPAD_RIGHT) pressedDir = 1;
+		else if (nowPressed & WPAD_UP) pressedDir = 2;
+		else if (nowPressed & WPAD_DOWN) pressedDir = 3;
+		else if (nowPressed & WPAD_TWO)
+			activatePoint();
+
+		if (pressedDir >= 0) {
+			if (canUseExit(currentNode->exits[pressedDir])) {
+				startMovementTo(currentNode->exits[pressedDir]);
+			} else {
+				// TODO: maybe remove this? got to see how it looks
+				static u16 directions[] = {-0x4000,0x4000,-0x7FFF,0};
+				daWMPlayer_c::instance->rot.y = directions[pressedDir];
+			}
 		}
 	}
 }
@@ -456,7 +460,7 @@ void dWMPathManager_c::startMovementTo(dKPPath_s *path) {
 
 	player->visible = (path->animation != dKPPath_s::INVISIBLE);
 
-	int id = (path->animation < dKPPath_s::MAX_ANIM) ? 0 : (int)path->animation;
+	int id = (path->animation >= dKPPath_s::MAX_ANIM) ? 0 : (int)path->animation;
 
 	player->startAnimation(Animations[id].anim, Animations[id].animParam1, Animations[id].animParam2, 0.0f);