vhad/wordpress-develop
1
0
Fork 0
mirror of https://github.com/gosticks/wordpress-develop.git synced 2026-07-01 07:40:07 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Sitemaps: Correctly enforce maximum number of sitemaps in index.

Browse Source 
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.

git-svn-id: https://develop.svn.wordpress.org/trunk@48532 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Pascal Birchler
2020-07-21 13:55:45 +00:00
parent fd9f7232e5
commit 38cade3b07
5 changed files with 98 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
tests/phpunit/includes/bootstrap.php
View File

@@ -162,6 +162,7 @@ require __DIR__ . '/class-wp-rest-test-configurable-controller.php';
require __DIR__ . '/class-wp-fake-block-type.php';
require __DIR__ . '/class-wp-sitemaps-test-provider.php';
require __DIR__ . '/class-wp-sitemaps-empty-test-provider.php';
require __DIR__ . '/class-wp-sitemaps-large-test-provider.php';
/**
* A class to handle additional command line arguments passed to the script.

59
tests/phpunit/includes/class-wp-sitemaps-large-test-provider.php Normal file
View File

@@ -0,0 +1,59 @@
<?php
/**
* Class WP_Sitemaps_Large_Test_Provider.
*
* Provides test data for additional registered providers.
*/
class WP_Sitemaps_Large_Test_Provider extends WP_Sitemaps_Provider {
/**
* Number of entries in the sitemap the provider produces.
*
* @var integer
*/
public $num_entries = 1;
/**
* WP_Sitemaps_Large_Test_Provider constructor.
*
* @param int $num_entries Optional. Number of entries in in the sitemap.
*/
public function __construct( $num_entries = 50001 ) {
$this->name = 'tests';
$this->object_type = 'test';
$this->num_entries = $num_entries;
}
/**
* Gets a URL list for a sitemap.
*
* @param int $page_num Page of results.
* @param string $object_subtype Optional. Object subtype name. Default empty.
* @return array List of URLs for a sitemap.
*/
public function get_url_list( $page_num, $object_subtype = '' ) {
return array_fill( 0, $this->num_entries, array( 'loc' => home_url( '/' ) ) );
}
/**
* Lists sitemap pages exposed by this provider.
*
* The returned data is used to populate the sitemap entries of the index.
*
* @return array[] Array of sitemap entries.
*/
public function get_sitemap_entries() {
return array_fill( 0, $this->num_entries, array( 'loc' => home_url( '/' ) ) );
}
/**
* Query for determining the number of pages.
*
* @param string $object_subtype Optional. Object subtype. Default empty.
* @return int Total number of pages.
*/
public function get_max_num_pages( $object_subtype = '' ) {
return $this->num_entries;
}
}

23
tests/phpunit/tests/sitemaps/sitemaps-index.php
View File

@@ -20,6 +20,29 @@ class Test_WP_Sitemaps_Index extends WP_UnitTestCase {
$this->assertCount( 24, $sitemap_index->get_sitemap_list() );
}
/**
* Test that a sitemap index won't contain more than 50000 sitemaps.
*
* @ticket 50666
*/
public function test_get_sitemap_list_limit() {
$registry = new WP_Sitemaps_Registry();
// add 3 providers, which combined produce more than the maximum 50000 sitemaps in the index.
$registry->add_sitemap( 'provider_1', new WP_Sitemaps_Large_Test_Provider( 25000 ) );
$registry->add_sitemap( 'provider_2', new WP_Sitemaps_Large_Test_Provider( 25000 ) );
$registry->add_sitemap( 'provider_3', new WP_Sitemaps_Large_Test_Provider( 25000 ) );
$count = 0;
foreach ( $registry->get_sitemaps() as $provider ) {
$count += count( $provider->get_url_list( 1 ) );
}
$this->assertGreaterThan( 50000, $count );
$sitemap_index = new WP_Sitemaps_Index( $registry );
$this->assertCount( 50000, $sitemap_index->get_sitemap_list() );
}
public function test_get_sitemap_list_no_entries() {
$registry = new WP_Sitemaps_Registry();