mirror of
git://develop.git.wordpress.org/
synced 2025-03-15 01:19:51 +01:00
I18N: Cache list of language file paths in WP_Textdomain_Registry
.
Loading a list of language file paths using `glob()` can be expensive if involving thousands of files. Expands scope of `WP_Textdomain_Registry` to cache list of language file paths in object cache and provides a way to invalidate that cache upon translation updates. Plugins can clear the cache using calls such as `wp_cache_delete( 'cached_mo_files_' . md5( $path ), 'translations' );` Props mreishus, swissspidy Fixes #58919 git-svn-id: https://develop.svn.wordpress.org/trunk@57287 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
parent
0d109bda84
commit
24531a9afc
@ -51,8 +51,11 @@ class WP_Textdomain_Registry {
|
||||
* Holds a cached list of available .mo files to improve performance.
|
||||
*
|
||||
* @since 6.1.0
|
||||
* @since 6.5.0 This property is no longer used.
|
||||
*
|
||||
* @var array
|
||||
*
|
||||
* @deprecated
|
||||
*/
|
||||
protected $cached_mo_files = array();
|
||||
|
||||
@ -65,6 +68,18 @@ class WP_Textdomain_Registry {
|
||||
*/
|
||||
protected $domains_with_translations = array();
|
||||
|
||||
/**
|
||||
* Initializes the registry.
|
||||
*
|
||||
* Hooks into the {@see 'upgrader_process_complete'} filter
|
||||
* to invalidate MO files caches.
|
||||
*
|
||||
* @since 6.5.0
|
||||
*/
|
||||
public function init() {
|
||||
add_action( 'upgrader_process_complete', array( $this, 'invalidate_mo_files_cache' ), 10, 2 );
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns the languages directory path for a specific domain and locale.
|
||||
*
|
||||
@ -134,6 +149,106 @@ class WP_Textdomain_Registry {
|
||||
$this->custom_paths[ $domain ] = rtrim( $path, '/' );
|
||||
}
|
||||
|
||||
/**
|
||||
* Retrieves .mo files from the specified path.
|
||||
*
|
||||
* Allows early retrieval through the {@see 'pre_get_mo_files_from_path'} filter to optimize
|
||||
* performance, especially in directories with many files.
|
||||
*
|
||||
* @since 6.5.0
|
||||
*
|
||||
* @param string $path The directory path to search for .mo files.
|
||||
* @return array Array of .mo file paths.
|
||||
*/
|
||||
public function get_language_files_from_path( $path ) {
|
||||
$path = trailingslashit( $path );
|
||||
|
||||
/**
|
||||
* Filters the .mo files retrieved from a specified path before the actual lookup.
|
||||
*
|
||||
* Returning a non-null value from the filter will effectively short-circuit
|
||||
* the MO files lookup, returning that value instead.
|
||||
*
|
||||
* This can be useful in situations where the directory contains a large number of files
|
||||
* and the default glob() function becomes expensive in terms of performance.
|
||||
*
|
||||
* @since 6.5.0
|
||||
*
|
||||
* @param null|array $mo_files List of .mo files. Default null.
|
||||
* @param string $path The path from which .mo files are being fetched.
|
||||
**/
|
||||
$mo_files = apply_filters( 'pre_get_language_files_from_path', null, $path );
|
||||
|
||||
if ( null !== $mo_files ) {
|
||||
return $mo_files;
|
||||
}
|
||||
|
||||
$cache_key = 'cached_mo_files_' . md5( $path );
|
||||
$mo_files = wp_cache_get( $cache_key, 'translations' );
|
||||
|
||||
if ( false === $mo_files ) {
|
||||
$mo_files = glob( $path . '*.mo' );
|
||||
if ( false === $mo_files ) {
|
||||
$mo_files = array();
|
||||
}
|
||||
wp_cache_set( $cache_key, $mo_files, 'translations' );
|
||||
}
|
||||
|
||||
return $mo_files;
|
||||
}
|
||||
|
||||
/**
|
||||
* Invalidate the cache for .mo files.
|
||||
*
|
||||
* This function deletes the cache entries related to .mo files when triggered
|
||||
* by specific actions, such as the completion of an upgrade process.
|
||||
*
|
||||
* @since 6.5.0
|
||||
*
|
||||
* @param WP_Upgrader $upgrader Unused. WP_Upgrader instance. In other contexts this might be a
|
||||
* Theme_Upgrader, Plugin_Upgrader, Core_Upgrade, or Language_Pack_Upgrader instance.
|
||||
* @param array $hook_extra {
|
||||
* Array of bulk item update data.
|
||||
*
|
||||
* @type string $action Type of action. Default 'update'.
|
||||
* @type string $type Type of update process. Accepts 'plugin', 'theme', 'translation', or 'core'.
|
||||
* @type bool $bulk Whether the update process is a bulk update. Default true.
|
||||
* @type array $plugins Array of the basename paths of the plugins' main files.
|
||||
* @type array $themes The theme slugs.
|
||||
* @type array $translations {
|
||||
* Array of translations update data.
|
||||
*
|
||||
* @type string $language The locale the translation is for.
|
||||
* @type string $type Type of translation. Accepts 'plugin', 'theme', or 'core'.
|
||||
* @type string $slug Text domain the translation is for. The slug of a theme/plugin or
|
||||
* 'default' for core translations.
|
||||
* @type string $version The version of a theme, plugin, or core.
|
||||
* }
|
||||
* }
|
||||
* @return void
|
||||
*/
|
||||
public function invalidate_mo_files_cache( $upgrader, $hook_extra ) {
|
||||
if ( 'translation' !== $hook_extra['type'] || array() === $hook_extra['translations'] ) {
|
||||
return;
|
||||
}
|
||||
|
||||
$translation_types = array_unique( wp_list_pluck( $hook_extra['translations'], 'type' ) );
|
||||
|
||||
foreach ( $translation_types as $type ) {
|
||||
switch ( $type ) {
|
||||
case 'plugin':
|
||||
wp_cache_delete( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/plugins/' ), 'translations' );
|
||||
break;
|
||||
case 'theme':
|
||||
wp_cache_delete( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/themes/' ), 'translations' );
|
||||
break;
|
||||
default:
|
||||
wp_cache_delete( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) ), 'translations' );
|
||||
break;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Returns possible language directory paths for a given text domain.
|
||||
*
|
||||
@ -156,7 +271,7 @@ class WP_Textdomain_Registry {
|
||||
}
|
||||
|
||||
/**
|
||||
* Gets the path to the language directory for the current locale.
|
||||
* Gets the path to the language directory for the current domain and locale.
|
||||
*
|
||||
* Checks the plugins and themes language directories as well as any
|
||||
* custom directory set via {@see load_plugin_textdomain()} or {@see load_theme_textdomain()}.
|
||||
@ -175,13 +290,11 @@ class WP_Textdomain_Registry {
|
||||
$found_location = false;
|
||||
|
||||
foreach ( $locations as $location ) {
|
||||
if ( ! isset( $this->cached_mo_files[ $location ] ) ) {
|
||||
$this->set_cached_mo_files( $location );
|
||||
}
|
||||
$files = $this->get_language_files_from_path( $location );
|
||||
|
||||
$path = "$location/$domain-$locale.mo";
|
||||
|
||||
foreach ( $this->cached_mo_files[ $location ] as $mo_path ) {
|
||||
foreach ( $files as $mo_path ) {
|
||||
if (
|
||||
! in_array( $domain, $this->domains_with_translations, true ) &&
|
||||
str_starts_with( str_replace( "$location/", '', $mo_path ), "$domain-" )
|
||||
@ -215,21 +328,4 @@ class WP_Textdomain_Registry {
|
||||
|
||||
return false;
|
||||
}
|
||||
|
||||
/**
|
||||
* Reads and caches all available MO files from a given directory.
|
||||
*
|
||||
* @since 6.1.0
|
||||
*
|
||||
* @param string $path Language directory path.
|
||||
*/
|
||||
private function set_cached_mo_files( $path ) {
|
||||
$this->cached_mo_files[ $path ] = array();
|
||||
|
||||
$mo_files = glob( $path . '/*.mo' );
|
||||
|
||||
if ( $mo_files ) {
|
||||
$this->cached_mo_files[ $path ] = $mo_files;
|
||||
}
|
||||
}
|
||||
}
|
||||
|
@ -1398,15 +1398,21 @@ function translate_user_role( $name, $domain = 'default' ) {
|
||||
* @since 3.0.0
|
||||
* @since 4.7.0 The results are now filterable with the {@see 'get_available_languages'} filter.
|
||||
*
|
||||
* @global WP_Textdomain_Registry $wp_textdomain_registry WordPress Textdomain Registry.
|
||||
*
|
||||
* @param string $dir A directory to search for language files.
|
||||
* Default WP_LANG_DIR.
|
||||
* @return string[] An array of language codes or an empty array if no languages are present.
|
||||
* Language codes are formed by stripping the .mo extension from the language file names.
|
||||
*/
|
||||
function get_available_languages( $dir = null ) {
|
||||
global $wp_textdomain_registry;
|
||||
|
||||
$languages = array();
|
||||
|
||||
$lang_files = glob( ( is_null( $dir ) ? WP_LANG_DIR : $dir ) . '/*.mo' );
|
||||
$path = is_null( $dir ) ? WP_LANG_DIR : $dir;
|
||||
$lang_files = $wp_textdomain_registry->get_language_files_from_path( $path );
|
||||
|
||||
if ( $lang_files ) {
|
||||
foreach ( $lang_files as $lang_file ) {
|
||||
$lang_file = basename( $lang_file, '.mo' );
|
||||
|
@ -380,6 +380,7 @@ $GLOBALS['wp_embed'] = new WP_Embed();
|
||||
* @global WP_Textdomain_Registry $wp_textdomain_registry WordPress Textdomain Registry.
|
||||
*/
|
||||
$GLOBALS['wp_textdomain_registry'] = new WP_Textdomain_Registry();
|
||||
$GLOBALS['wp_textdomain_registry']->init();
|
||||
|
||||
// Load multisite-specific files.
|
||||
if ( is_multisite() ) {
|
||||
|
@ -18,21 +18,21 @@ class Tests_L10n_wpTextdomainRegistry extends WP_UnitTestCase {
|
||||
$this->instance = new WP_Textdomain_Registry();
|
||||
}
|
||||
|
||||
public function tear_down() {
|
||||
wp_cache_delete( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/foobar/' ), 'translations' );
|
||||
wp_cache_delete( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/plugins/' ), 'translations' );
|
||||
wp_cache_delete( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/themes/' ), 'translations' );
|
||||
wp_cache_delete( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) ), 'translations' );
|
||||
|
||||
parent::tear_down();
|
||||
}
|
||||
|
||||
/**
|
||||
* @covers ::has
|
||||
* @covers ::get
|
||||
* @covers ::set_custom_path
|
||||
*/
|
||||
public function test_set_custom_path() {
|
||||
$reflection = new ReflectionClass( $this->instance );
|
||||
$reflection_property = $reflection->getProperty( 'cached_mo_files' );
|
||||
$reflection_property->setAccessible( true );
|
||||
|
||||
$this->assertEmpty(
|
||||
$reflection_property->getValue( $this->instance ),
|
||||
'Cache not empty by default'
|
||||
);
|
||||
|
||||
$this->instance->set_custom_path( 'foo', WP_LANG_DIR . '/bar' );
|
||||
|
||||
$this->assertTrue(
|
||||
@ -48,10 +48,9 @@ class Tests_L10n_wpTextdomainRegistry extends WP_UnitTestCase {
|
||||
$this->instance->get( 'foo', 'de_DE' ),
|
||||
'Custom path for textdomain not returned'
|
||||
);
|
||||
$this->assertArrayHasKey(
|
||||
WP_LANG_DIR . '/bar',
|
||||
$reflection_property->getValue( $this->instance ),
|
||||
'Custom path missing from cache'
|
||||
$this->assertNotFalse(
|
||||
wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . 'bar/' ), 'translations' ),
|
||||
'List of files in custom path not cached'
|
||||
);
|
||||
}
|
||||
|
||||
@ -60,22 +59,12 @@ class Tests_L10n_wpTextdomainRegistry extends WP_UnitTestCase {
|
||||
* @dataProvider data_domains_locales
|
||||
*/
|
||||
public function test_get( $domain, $locale, $expected ) {
|
||||
$reflection = new ReflectionClass( $this->instance );
|
||||
$reflection_property = $reflection->getProperty( 'cached_mo_files' );
|
||||
$reflection_property->setAccessible( true );
|
||||
|
||||
$actual = $this->instance->get( $domain, $locale );
|
||||
$this->assertSame(
|
||||
$expected,
|
||||
$actual,
|
||||
'Expected languages directory path not matching actual one'
|
||||
);
|
||||
|
||||
$this->assertArrayHasKey(
|
||||
WP_LANG_DIR . '/plugins',
|
||||
$reflection_property->getValue( $this->instance ),
|
||||
'Default plugins path missing from cache'
|
||||
);
|
||||
}
|
||||
|
||||
/**
|
||||
@ -91,6 +80,75 @@ class Tests_L10n_wpTextdomainRegistry extends WP_UnitTestCase {
|
||||
);
|
||||
}
|
||||
|
||||
/**
|
||||
* @covers ::get_language_files_from_path
|
||||
*/
|
||||
public function test_get_language_files_from_path_caches_results() {
|
||||
$this->instance->get_language_files_from_path( trailingslashit( WP_LANG_DIR ) . '/foobar/' );
|
||||
$this->instance->get_language_files_from_path( trailingslashit( WP_LANG_DIR ) . '/plugins/' );
|
||||
$this->instance->get_language_files_from_path( trailingslashit( WP_LANG_DIR ) . '/themes/' );
|
||||
$this->instance->get_language_files_from_path( trailingslashit( WP_LANG_DIR ) );
|
||||
|
||||
$this->assertNotFalse( wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/plugins/' ), 'translations' ) );
|
||||
$this->assertNotFalse( wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/themes/' ), 'translations' ) );
|
||||
$this->assertNotFalse( wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/foobar/' ), 'translations' ) );
|
||||
$this->assertNotFalse( wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) ), 'translations' ) );
|
||||
}
|
||||
|
||||
/**
|
||||
* @covers ::get_language_files_from_path
|
||||
*/
|
||||
public function test_get_language_files_from_path_short_circuit() {
|
||||
add_filter( 'pre_get_language_files_from_path', '__return_empty_array' );
|
||||
$result = $this->instance->get_language_files_from_path( trailingslashit( WP_LANG_DIR ) . '/plugins/' );
|
||||
remove_filter( 'pre_get_language_files_from_path', '__return_empty_array' );
|
||||
|
||||
$cache = wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/plugins/' ), 'translations' );
|
||||
|
||||
$this->assertEmpty( $result );
|
||||
$this->assertFalse( $cache );
|
||||
}
|
||||
|
||||
/**
|
||||
* @covers ::invalidate_mo_files_cache
|
||||
*/
|
||||
public function test_invalidate_mo_files_cache() {
|
||||
$this->instance->get_language_files_from_path( trailingslashit( WP_LANG_DIR ) . '/plugins/' );
|
||||
$this->instance->get_language_files_from_path( trailingslashit( WP_LANG_DIR ) . '/themes/' );
|
||||
$this->instance->get_language_files_from_path( trailingslashit( WP_LANG_DIR ) );
|
||||
|
||||
$this->instance->invalidate_mo_files_cache(
|
||||
null,
|
||||
array(
|
||||
'type' => 'translation',
|
||||
'translations' => array(
|
||||
(object) array(
|
||||
'type' => 'plugin',
|
||||
'slug' => 'internationalized-plugin',
|
||||
'language' => 'de_DE',
|
||||
'version' => '99.9.9',
|
||||
),
|
||||
(object) array(
|
||||
'type' => 'theme',
|
||||
'slug' => 'internationalized-theme',
|
||||
'language' => 'de_DE',
|
||||
'version' => '99.9.9',
|
||||
),
|
||||
(object) array(
|
||||
'type' => 'core',
|
||||
'slug' => 'default',
|
||||
'language' => 'es_ES',
|
||||
'version' => '99.9.9',
|
||||
),
|
||||
),
|
||||
)
|
||||
);
|
||||
|
||||
$this->assertFalse( wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/plugins/' ), 'translations' ) );
|
||||
$this->assertFalse( wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) . '/themes/' ), 'translations' ) );
|
||||
$this->assertFalse( wp_cache_get( 'cached_mo_files_' . md5( trailingslashit( WP_LANG_DIR ) ), 'translations' ) );
|
||||
}
|
||||
|
||||
public function data_domains_locales() {
|
||||
return array(
|
||||
'Non-existent plugin' => array(
|
||||
|
Loading…
x
Reference in New Issue
Block a user