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

Timestamp:

07/21/2020 01:55:45 PM (4 years ago)

Author:

swissspidy

Message:

Sitemaps: Correctly enforce maximum number of sitemaps in index.

Before this change, the limit of 50k entries was enforced for the number of providers, not the amount of sitemaps all providers add to the index in total.

Props pbiron, swissspidy.
Fixes #50666.

File:

• trunk/src/wp-includes/sitemaps/class-wp-sitemaps-registry.php (modified) (3 diffs)

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

@@ -24 +24 @@
      */
     private $sitemaps = array();
-
-    /**
-     * Maximum number of sitemaps to include in an index.
-     *
-     * @sincee 5.5.0
-     *
-     * @var int Maximum number of sitemaps.
-     */
-    private $max_sitemaps = 50000;
 
     /**
@@ -70 +61 @@
 
     /**
-     * Lists all registered sitemaps.
+     * Returns all registered sitemap providers.
      *
      * @since 5.5.0
@@ -77 +68 @@
      */
     public function get_sitemaps() {
-        $total_sitemaps = count( $this->sitemaps );
-
-        if ( $total_sitemaps > $this->max_sitemaps ) {
-            return array_slice( $this->sitemaps, 0, $this->max_sitemaps, true );
-        }
-
         return $this->sitemaps;
     }