Changeset 60411 for branches/6.8/tests/phpunit/tests/formatting/sanitizeFileName.php

Timestamp:

07/05/2025 06:33:45 AM (14 months ago)

Author:

audrasjb

Message:

Media: Normalize all space characters to a space in sanitize_file_name().

The sanitize_file_name() function normalizes the no-break space to a normal space (U+0020) in order to prevent issues saving files with the no-break space in it.

This patch expands the replacement to all space characters, since it’s known that macOS stores a NARROW NO-BREAK SPACE (U+202F) in screenshot filenames between the time and the am/pm indicator.

There are deeper issues with the way this function works, but this patch resolves a known and common problem without raising any of the deeper refactoring questions.

Reviewed by audrasjb, youknowriad.
Merges [60399] to the 6.8 branch.
Props audrasjb, desrosj, dmsnell, jonsurrell, matt, room34, siliconforks, zieladam, annezazu.
Fixes #62995.

Location:

branches/6.8

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/formatting/sanitizeFileName.php (modified) (1 diff)

branches/6.8/tests/phpunit/tests/formatting/sanitizeFileName.php

@@ -36 +36 @@
          *
          * @ticket 16330
+         * @ticket 62995
          */
         public function test_replaces_spaces() {
                 $urls = array(
-                        'unencoded space.png'  => 'unencoded-space.png',
-                        'encoded-space.jpg'    => 'encoded-space.jpg',
-                        'plus+space.jpg'       => 'plusspace.jpg',
-                        'multi %20 +space.png' => 'multi-20-space.png',
+                        'unencoded space.png'                         => 'unencoded-space.png',
+                        'encoded-space.jpg'                           => 'encoded-space.jpg',
+                        'plus+space.jpg'                              => 'plusspace.jpg',
+                        'multi %20 +space.png'                        => 'multi-20-space.png',
+                        "Screenshot 2025-02-19 at 2.17.33\u{202F}PM.png" => 'Screenshot-2025-02-19-at-2.17.33-PM.png',
+                        "Filename with non-breaking\u{00A0}space.txt" => 'Filename-with-non-breaking-space.txt',
+                        "Filename with thin\u{2009}space.txt"         => 'Filename-with-thin-space.txt',
                 );