Merge pull request #51869 from nextcloud/backport/51866/stable31

[stable31] fix: use proper migration sorting when checking if a migration needs to be executed

Author: AndyScherzinger

lib/private/DB/MigrationService.php

@@ -199,9 +199,9 @@ protected function sortMigrations(string $a, string $b): int {
 			if ($versionA !== $versionB) {
 				return ($versionA < $versionB) ? -1 : 1;
 			}
-			return ($matchA[2] < $matchB[2]) ? -1 : 1;
+			return strnatcmp($matchA[2], $matchB[2]);
 		}
-		return (basename($a) < basename($b)) ? -1 : 1;
+		return strnatcmp(basename($a), basename($b));
 	}
 
 	/**
@@ -250,7 +250,7 @@ private function getMigrationsToExecute($to) {
 
 		$toBeExecuted = [];
 		foreach ($availableMigrations as $v) {
-			if ($to !== 'latest' && $v > $to) {
+			if ($to !== 'latest' && ($this->sortMigrations($v, $to) > 0)) {
 				continue;
 			}
 			if ($this->shallBeExecuted($v, $knownMigrations)) {