Changeset 29249 for trunk/tests/phpunit/tests/theme/themeDir.php

Timestamp:

07/19/2014 11:06:54 PM (12 years ago)

Author:

wonderboymusic

Message:

Call untrailingslashit() when adding a theme directory in register_theme_directory(). This prevents double-slashing in generated URLs. Don't add a directory if it is already present.

Adds unit tests.
Props obenland, wonderboymusic.
Fixes #28662.

File:

• trunk/tests/phpunit/tests/theme/themeDir.php (modified) (1 diff)

trunk/tests/phpunit/tests/theme/themeDir.php

@@ -235 +235 @@
     }
 
+    /**
+     * @ticket 28662
+     */
+    function test_theme_dir_slashes() {
+        $size = count( $GLOBALS['wp_theme_directories'] );
+
+        @mkdir( WP_CONTENT_DIR . '/themes/foo' );
+        @mkdir( WP_CONTENT_DIR . '/themes/foo-themes' );
+
+        $this->assertTrue( file_exists( WP_CONTENT_DIR . '/themes/foo' ) );
+        $this->assertTrue( file_exists( WP_CONTENT_DIR . '/themes/foo-themes') );
+
+        register_theme_directory( '/' );
+
+        $this->assertCount( $size, $GLOBALS['wp_theme_directories'] );
+
+        register_theme_directory( 'themes/' );
+
+        $this->assertCount( $size, $GLOBALS['wp_theme_directories'] );
+
+        register_theme_directory( '/foo/' );
+
+        $this->assertCount( $size, $GLOBALS['wp_theme_directories'] );
+
+        register_theme_directory( 'foo/' );
+
+        $this->assertCount( $size, $GLOBALS['wp_theme_directories'] );
+
+        register_theme_directory( 'themes/foo/' );
+
+        $this->assertCount( $size + 1, $GLOBALS['wp_theme_directories'] );
+
+        register_theme_directory( WP_CONTENT_DIR . '/foo-themes/' );
+
+        $this->assertCount( $size + 1, $GLOBALS['wp_theme_directories'] );
+
+        foreach ( $GLOBALS['wp_theme_directories'] as $dir ) {
+            $this->assertNotEquals( '/', substr( $dir, -1 ) );
+        }
+
+        rmdir( WP_CONTENT_DIR . '/themes/foo' );
+        rmdir( WP_CONTENT_DIR . '/themes/foo-themes' );
+    }
 }