Changeset 48543 for trunk/src/wp-includes/sitemaps/class-wp-sitemaps.php

Timestamp:

07/21/2020 04:12:49 PM (5 years ago)

Author:

swissspidy

Message:

Sitemaps: Replace wp_sitemaps_register_providers filter with more suitable wp_sitemaps_add_provider filter.

The previous filter failed the goal of allowing developers to filter all providers before they are registered, since it only filtered the built-in ones.

The more specific wp_sitemaps_add_provider filter enables exactly that, as it filters every sitemap provider right before it is added to the sitemaps registry.

Props pbiron, pfefferle, Chouby, swissspidy.
Fixes #50660.

File:

• trunk/src/wp-includes/sitemaps/class-wp-sitemaps.php (modified) (1 diff)

trunk/src/wp-includes/sitemaps/class-wp-sitemaps.php

@@ -111 +111 @@
      */
     public function register_sitemaps() {
-        /**
-         * Filters the list of registered sitemap providers.
-         *
-         * @since 5.5.0
-         *
-         * @param array $providers {
-         *     Array of WP_Sitemaps_Provider objects keyed by their name.
-         *
-         *     @type WP_Sitemaps_Posts      $posts      The WP_Sitemaps_Posts object.
-         *     @type WP_Sitemaps_Taxonomies $taxonomies The WP_Sitemaps_Taxonomies object.
-         *     @type WP_Sitemaps_Users      $users      The WP_Sitemaps_Users object.
-         * }
-         */
-        $providers = apply_filters(
-            'wp_sitemaps_register_providers',
-            array(
-                'posts'      => new WP_Sitemaps_Posts(),
-                'taxonomies' => new WP_Sitemaps_Taxonomies(),
-                'users'      => new WP_Sitemaps_Users(),
-            )
+        $providers = array(
+            'posts'      => new WP_Sitemaps_Posts(),
+            'taxonomies' => new WP_Sitemaps_Taxonomies(),
+            'users'      => new WP_Sitemaps_Users(),
         );
 
-        // Register each supported provider.
         /* @var WP_Sitemaps_Provider $provider */
         foreach ( $providers as $name => $provider ) {