From 2aef21a55b2bf1d8ce6f96a88840384b9f9f49e8 Mon Sep 17 00:00:00 2001 From: John Blackbourn Date: Thu, 8 Sep 2016 22:53:57 +0000 Subject: [PATCH] Themes: Improve child theme file inheritance by introducing functions for locating and fetching the URL or path to files within child and parent themes. The most useful function this introduces is `get_theme_file_uri()`, which returns the URL to the specified file in the child theme if it exists, and falls back to the URL to the specified file in the parent theme. This allows parent themes to reference files (including enqueuing CSS and JavaScript files) that can be overridden by the child theme simply by existing. This change also introduces `get_theme_file_path()`, which is the file path equivalent of `get_theme_file_uri()`. Finally, `get_parent_theme_file_uri()` and `get_parent_theme_file_path()` are also introduced, which allow a theme to specifically reference a file URL or file path in the parent theme. These can be used as replacements for `get_template_directory_uri()` and `get_template_directory()` respectively, for consistency. Props johnbillion, georgestephanis, gma992. Fixes #18302 git-svn-id: https://develop.svn.wordpress.org/trunk@38578 602fd350-edb4-49c9-b593-d223f7449a82 --- src/wp-includes/link-template.php | 127 ++++++++++++++- .../data/theme-file-child/child-only.php | 1 + .../theme-file-child/parent-and-child.php | 1 + tests/phpunit/data/theme-file-child/style.css | 4 + .../theme-file-parent/parent-and-child.php | 1 + .../data/theme-file-parent/parent-only.php | 1 + .../phpunit/data/theme-file-parent/style.css | 3 + tests/phpunit/tests/link/themeFile.php | 146 ++++++++++++++++++ 8 files changed, 282 insertions(+), 2 deletions(-) create mode 100644 tests/phpunit/data/theme-file-child/child-only.php create mode 100644 tests/phpunit/data/theme-file-child/parent-and-child.php create mode 100644 tests/phpunit/data/theme-file-child/style.css create mode 100644 tests/phpunit/data/theme-file-parent/parent-and-child.php create mode 100644 tests/phpunit/data/theme-file-parent/parent-only.php create mode 100644 tests/phpunit/data/theme-file-parent/style.css create mode 100644 tests/phpunit/tests/link/themeFile.php diff --git a/src/wp-includes/link-template.php b/src/wp-includes/link-template.php index 2c80eb3068..555c50949c 100644 --- a/src/wp-includes/link-template.php +++ b/src/wp-includes/link-template.php @@ -930,7 +930,7 @@ function get_edit_term_link( $term_id, $taxonomy = '', $object_type = '' ) { } $tax = get_taxonomy( $term->taxonomy ); - if ( ! $tax || ! current_user_can( $tax->cap->edit_terms ) ) { + if ( ! $tax || ! current_user_can( 'edit_term', $term->term_id ) ) { return; } @@ -984,8 +984,9 @@ function edit_term_link( $link = '', $before = '', $after = '', $term = null, $e return; $tax = get_taxonomy( $term->taxonomy ); - if ( ! current_user_can( $tax->cap->edit_terms ) ) + if ( ! current_user_can( 'edit_term', $term->term_id ) ) { return; + } if ( empty( $link ) ) $link = __('Edit This'); @@ -4023,3 +4024,125 @@ function get_avatar_data( $id_or_email, $args = null ) { */ return apply_filters( 'get_avatar_data', $args, $id_or_email ); } + +/** + * Retrieve the URL of a file in the theme. + * + * Searches in the stylesheet directory before the template directory so themes + * which inherit from a parent theme can just override one file. + * + * @since 4.7.0 + * + * @param string $file Optional. File to search for in the stylesheet directory. + * @return string The URL of the file. + */ +function get_theme_file_uri( $file = '' ) { + $file = ltrim( $file, '/' ); + + if ( empty( $file ) ) { + $url = get_stylesheet_directory_uri(); + } elseif ( file_exists( get_stylesheet_directory() . '/' . $file ) ) { + $url = get_stylesheet_directory_uri() . '/' . $file; + } else { + $url = get_template_directory_uri() . '/' . $file; + } + + /** + * Filter the URL to a file in the theme. + * + * @since 4.7.0 + * + * @param string $url The file URL. + * @param string $file The requested file to search for. + */ + return apply_filters( 'theme_file_uri', $url, $file ); +} + +/** + * Retrieve the URL of a file in the parent theme. + * + * @since 4.7.0 + * + * @param string $file Optional. File to return the URL for in the template directory. + * @return string The URL of the file. + */ +function get_parent_theme_file_uri( $file = '' ) { + $file = ltrim( $file, '/' ); + + if ( empty( $file ) ) { + $url = get_template_directory_uri(); + } else { + $url = get_template_directory_uri() . '/' . $file; + } + + /** + * Filter the URL to a file in the parent theme. + * + * @since 4.7.0 + * + * @param string $url The file URL. + * @param string $file The requested file to search for. + */ + return apply_filters( 'parent_theme_file_uri', $url, $file ); +} + +/** + * Retrieve the path of a file in the theme. + * + * Searches in the stylesheet directory before the template directory so themes + * which inherit from a parent theme can just override one file. + * + * @since 4.7.0 + * + * @param string $file Optional. File to search for in the stylesheet directory. + * @return string The path of the file. + */ +function get_theme_file_path( $file = '' ) { + $file = ltrim( $file, '/' ); + + if ( empty( $file ) ) { + $path = get_stylesheet_directory(); + } elseif ( file_exists( get_stylesheet_directory() . '/' . $file ) ) { + $path = get_stylesheet_directory() . '/' . $file; + } else { + $path = get_template_directory() . '/' . $file; + } + + /** + * Filter the path to a file in the theme. + * + * @since 4.7.0 + * + * @param string $path The file path. + * @param string $file The requested file to search for. + */ + return apply_filters( 'theme_file_path', $path, $file ); +} + +/** + * Retrieve the path of a file in the parent theme. + * + * @since 4.7.0 + * + * @param string $file Optional. File to return the path for in the template directory. + * @return string The path of the file. + */ +function get_parent_theme_file_path( $file = '' ) { + $file = ltrim( $file, '/' ); + + if ( empty( $file ) ) { + $path = get_template_directory(); + } else { + $path = get_template_directory() . '/' . $file; + } + + /** + * Filter the path to a file in the parent theme. + * + * @since 4.7.0 + * + * @param string $path The file path. + * @param string $file The requested file to search for. + */ + return apply_filters( 'parent_theme_file_path', $path, $file ); +} diff --git a/tests/phpunit/data/theme-file-child/child-only.php b/tests/phpunit/data/theme-file-child/child-only.php new file mode 100644 index 0000000000..9065825c33 --- /dev/null +++ b/tests/phpunit/data/theme-file-child/child-only.php @@ -0,0 +1 @@ + diff --git a/tests/phpunit/data/theme-file-child/parent-and-child.php b/tests/phpunit/data/theme-file-child/parent-and-child.php new file mode 100644 index 0000000000..fb1203accf --- /dev/null +++ b/tests/phpunit/data/theme-file-child/parent-and-child.php @@ -0,0 +1 @@ + diff --git a/tests/phpunit/data/theme-file-child/style.css b/tests/phpunit/data/theme-file-child/style.css new file mode 100644 index 0000000000..2b0e145eda --- /dev/null +++ b/tests/phpunit/data/theme-file-child/style.css @@ -0,0 +1,4 @@ +/* +Theme Name: Child Theme +Template: theme-file-parent +*/ diff --git a/tests/phpunit/data/theme-file-parent/parent-and-child.php b/tests/phpunit/data/theme-file-parent/parent-and-child.php new file mode 100644 index 0000000000..fb1203accf --- /dev/null +++ b/tests/phpunit/data/theme-file-parent/parent-and-child.php @@ -0,0 +1 @@ + diff --git a/tests/phpunit/data/theme-file-parent/parent-only.php b/tests/phpunit/data/theme-file-parent/parent-only.php new file mode 100644 index 0000000000..22a963c4e6 --- /dev/null +++ b/tests/phpunit/data/theme-file-parent/parent-only.php @@ -0,0 +1 @@ + diff --git a/tests/phpunit/data/theme-file-parent/style.css b/tests/phpunit/data/theme-file-parent/style.css new file mode 100644 index 0000000000..418cb056f7 --- /dev/null +++ b/tests/phpunit/data/theme-file-parent/style.css @@ -0,0 +1,3 @@ +/* +Theme Name: Parent Theme +*/ diff --git a/tests/phpunit/tests/link/themeFile.php b/tests/phpunit/tests/link/themeFile.php new file mode 100644 index 0000000000..53408aeab7 --- /dev/null +++ b/tests/phpunit/tests/link/themeFile.php @@ -0,0 +1,146 @@ + +assertSame( content_url( "themes/theme-file-parent/{$file}" ), get_theme_file_uri( $file ) ); + $this->assertSame( content_url( "themes/theme-file-parent/{$file}" ), get_parent_theme_file_uri( $file ) ); + } + + /** + * @ticket 18302 + * + * @dataProvider data_theme_files + */ + public function test_theme_file_uri_with_child_theme( $file, $expected_theme, $existence ) { + switch_theme( 'theme-file-child' ); + + // Ensure the returned URL uses the expected theme: + $this->assertSame( content_url( "themes/{$expected_theme}/{$file}" ), get_theme_file_uri( $file ) ); + + // Ensure the returned URL always uses the parent theme: + $this->assertSame( content_url( "themes/theme-file-parent/{$file}" ), get_parent_theme_file_uri( $file ) ); + } + + /** + * @ticket 18302 + * + * @dataProvider data_theme_files + */ + public function test_theme_file_path_with_parent_theme( $file, $expected_theme, $existence ) { + switch_theme( 'theme-file-parent' ); + + // Ensure the returned path always uses the parent theme: + $this->assertSame( WP_CONTENT_DIR . "/themes/theme-file-parent/{$file}", get_theme_file_path( $file ) ); + $this->assertSame( WP_CONTENT_DIR . "/themes/theme-file-parent/{$file}", get_parent_theme_file_path( $file ) ); + } + + /** + * @ticket 18302 + * + * @dataProvider data_theme_files + */ + public function test_theme_file_path_with_child_theme( $file, $expected_theme, $existence ) { + switch_theme( 'theme-file-child' ); + + // Ensure the returned path uses the expected theme: + $this->assertSame( WP_CONTENT_DIR . "/themes/{$expected_theme}/{$file}", get_theme_file_path( $file ) ); + + // Ensure the returned path always uses the parent theme: + $this->assertSame( WP_CONTENT_DIR . "/themes/theme-file-parent/{$file}", get_parent_theme_file_path( $file ) ); + } + + /** + * Test the tests. + * + * @ticket 18302 + * + * @dataProvider data_theme_files + */ + public function test_theme_file_existance( $file, $expected_theme, $existence ) { + + if ( in_array( 'theme-file-child', $existence, true ) ) { + $this->assertFileExists( WP_CONTENT_DIR . "/themes/theme-file-child/{$file}" ); + } else { + $this->assertFileNotExists( WP_CONTENT_DIR . "/themes/theme-file-child/{$file}" ); + } + + if ( in_array( 'theme-file-parent', $existence, true ) ) { + $this->assertFileExists( WP_CONTENT_DIR . "/themes/theme-file-parent/{$file}" ); + } else { + $this->assertFileNotExists( WP_CONTENT_DIR . "/themes/theme-file-parent/{$file}" ); + } + + } + + /** + * @ticket 18302 + * + * @dataProvider data_theme_files + */ + public function test_theme_file_uri_returns_valid_uri( $file, $expected_theme, $existence ) { + $uri = get_theme_file_uri( $file ); + $parent_uri = get_parent_theme_file_uri( $file ); + + $this->assertSame( esc_url_raw( $uri ), $uri ); + $this->assertSame( esc_url_raw( $parent_uri ), $parent_uri ); + } + + public function data_theme_files() { + $parent = 'theme-file-parent'; + $child = 'theme-file-child'; + + return array( + array( + 'parent-only.php', + $parent, + array( + $parent, + ), + ), + array( + 'child-only.php', + $child, + array( + $child, + ), + ), + array( + 'parent-and-child.php', + $child, + array( + $parent, + $child, + ), + ), + array( + 'neither.php', + $parent, + array( + ), + ), + ); + } + +}