Cleaner normalizePath regex

Signed-off-by: J0WI <J0WI@users.noreply.github.com>

lib/private/Files/Filesystem.php

@@ -831,10 +831,10 @@ public static function normalizePath($path, $stripTrailingSlash = true, $isAbsol
 		$path = '/' . $path;
 
 		$patterns = [
-			'/\\\\/s',          // no windows style slashes
-			'/\/\.(\/\.)?\//s', // remove '/./'
-			'/\/{2,}/s',        // remove sequence of slashes
-			'/\/\.$/s',         // remove trailing /.
+			'#\\\\#s',       // no windows style '\\' slashes
+			'#/\.(/\.)*/#s', // remove '/./'
+			'#\//+#s',       // remove sequence of slashes
+			'#/\.$#s',       // remove trailing '/.'
 		];
 
 		do {

tests/lib/Files/FilesystemTest.php

@@ -140,11 +140,26 @@ public function normalizePathData() {
 			['/foo/.bar', '/foo/.bar/'],
 			['/foo/.bar/', '/foo/.bar/', false],
 			['/foo/.bar/tee', '/foo/.bar/tee'],
-
-			['/foo/bar', '/.///././//./foo/.///././//./bar/./././.'],
-			['/foo/bar/', '/.///././//./foo/.///././//./bar/./././.', false],
-			['/foo/bar', '/.///././//./foo/.///././//./bar/././././'],
-			['/foo/bar/', '/.///././//./foo/.///././//./bar/././././', false],
+			['/foo/bar.', '/foo/bar./'],
+			['/foo/bar./', '/foo/bar./', false],
+			['/foo/bar./tee', '/foo/bar./tee'],
+			['/foo/.bar.', '/foo/.bar./'],
+			['/foo/.bar./', '/foo/.bar./', false],
+			['/foo/.bar./tee', '/foo/.bar./tee'],
+
+			['/foo/bar', '/.////././//./foo/.///././//./bar/././/./.'],
+			['/foo/bar/', '/.////././//./foo/.///././//./bar/./././.', false],
+			['/foo/bar', '/.////././//./foo/.///././//./bar/././/././'],
+			['/foo/bar/', '/.////././//./foo/.///././//./bar/././/././', false],
+			['/foo/.bar', '/.////././//./foo/./././/./.bar/././/././'],
+			['/foo/.bar/', '/.////././//./foo/./././/./.bar/././/././', false],
+			['/foo/.bar/tee.', '/.////././//./foo/./././/./.bar/tee././/././', false],
+			['/foo/bar.', '/.////././//./foo/./././/./bar./././/././'],
+			['/foo/bar./', '/.////././//./foo/./././/./bar./././/././', false],
+			['/foo/bar./tee./', '/.////././//./foo/./././/./bar./tee././/././', false],
+			['/foo/.bar.', '/.////././//./foo/./././/./.bar./././/././'],
+			['/foo/.bar./', '/.////././//./foo/./././/./.bar./././././', false],
+			['/foo/.bar./tee./', '/.////././//./foo/./././/./.bar./tee././././', false],
 
 			// Windows paths
 			['/', ''],
@@ -186,7 +201,9 @@ public function normalizePathData() {
 
 			// normalize does not resolve '..' (by design)
 			['/foo/..', '/foo/../'],
+			['/foo/../bar', '/foo/../bar/.'],
 			['/foo/..', '\\foo\\..\\'],
+			['/foo/../bar', '\\foo\\..\\bar'],
 		];
 	}