From 65133a89c169021497e92017e6aab8abe1b02f4a Mon Sep 17 00:00:00 2001 From: Drew Jaynes Date: Tue, 6 Oct 2015 04:35:25 +0000 Subject: [PATCH] Multisite: Introduce `get_subdirectory_reserved_names()`, which returns a filterable list of reserved subdirectory site names. The function encapsulates the existing `subdirectory_reserved_names` filter and reduces the maintenance burden of keeping the value of (currently) two instances of the same hook in sync. See #33615. Built from https://develop.svn.wordpress.org/trunk@34854 git-svn-id: http://core.svn.wordpress.org/trunk@34819 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-admin/network/site-new.php | 9 ++++---- wp-includes/ms-functions.php | 39 +++++++++++++++++++++++------------ wp-includes/version.php | 2 +- 3 files changed, 32 insertions(+), 18 deletions(-) diff --git a/wp-admin/network/site-new.php b/wp-admin/network/site-new.php index bd3216bc75..7f91f9abb0 100644 --- a/wp-admin/network/site-new.php +++ b/wp-admin/network/site-new.php @@ -46,10 +46,11 @@ if ( isset($_REQUEST['action']) && 'add-site' == $_REQUEST['action'] ) { // If not a subdomain install, make sure the domain isn't a reserved word if ( ! is_subdomain_install() ) { - /** This filter is documented in wp-includes/ms-functions.php */ - $subdirectory_reserved_names = apply_filters( 'subdirectory_reserved_names', array( 'page', 'comments', 'blog', 'embed', 'files', 'feed', 'wp-admin', 'wp-content', 'wp-includes', 'wp-json' ) ); - if ( in_array( $domain, $subdirectory_reserved_names ) ) - wp_die( sprintf( __('The following words are reserved for use by WordPress functions and cannot be used as blog names: %s' ), implode( ', ', $subdirectory_reserved_names ) ) ); + $subdirectory_reserved_names = get_subdirectory_reserved_names(); + + if ( in_array( $domain, $subdirectory_reserved_names ) ) { + wp_die( sprintf( __( 'The following words are reserved for use by WordPress functions and cannot be used as blog names: %s' ), implode( ', ', $subdirectory_reserved_names ) ) ); + } } $title = $blog['title']; diff --git a/wp-includes/ms-functions.php b/wp-includes/ms-functions.php index b066a27c96..8c748e686f 100644 --- a/wp-includes/ms-functions.php +++ b/wp-includes/ms-functions.php @@ -555,19 +555,7 @@ function wpmu_validate_blog_signup( $blogname, $blog_title, $user = '' ) { * spring them from jail. */ if ( ! is_subdomain_install() ) { - $illegal_names = array_merge( - $illegal_names, - /** - * Filter reserved site names on a sub-directory Multisite install. - * - * @since 3.0.0 - * @since 4.4.0 'wp-admin', 'wp-content', 'wp-includes', 'wp-json', and 'embed' were added - * to the reserved names list. - * - * @param array $subdirectory_reserved_names Array of reserved names. - */ - apply_filters( 'subdirectory_reserved_names', array( 'page', 'comments', 'blog', 'files', 'feed', 'wp-admin', 'wp-content', 'wp-includes', 'wp-json', 'embed' ) ) - ); + $illegal_names = array_merge( $illegal_names, get_subdirectory_reserved_names() ); } if ( empty( $blogname ) ) @@ -2468,3 +2456,28 @@ function wp_get_sites( $args = array() ) { return $site_results; } + +/** + * Retrieves a list of reserved site on a sub-directory Multisite install. + * + * @since 4.4.0 + * + * @return array $names Array of reserved subdirectory names. + */ +function get_subdirectory_reserved_names() { + $names = array( + 'page', 'comments', 'blog', 'files', 'feed', 'wp-admin', + 'wp-content', 'wp-includes', 'wp-json', 'embed' + ); + + /** + * Filter reserved site names on a sub-directory Multisite install. + * + * @since 3.0.0 + * @since 4.4.0 'wp-admin', 'wp-content', 'wp-includes', 'wp-json', and 'embed' were added + * to the reserved names list. + * + * @param array $subdirectory_reserved_names Array of reserved names. + */ + return apply_filters( 'subdirectory_reserved_names', $names ); +} diff --git a/wp-includes/version.php b/wp-includes/version.php index 5f6f740f52..193623e15e 100644 --- a/wp-includes/version.php +++ b/wp-includes/version.php @@ -4,7 +4,7 @@ * * @global string $wp_version */ -$wp_version = '4.4-alpha-34853'; +$wp_version = '4.4-alpha-34854'; /** * Holds the WordPress DB revision, increments when changes are made to the WordPress DB schema.