Changeset 57287 for trunk/src/wp-includes/class-wp-textdomain-registry.php

Timestamp:

01/15/2024 07:03:27 PM (3 years ago)

Author:

swissspidy

Message:

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

File:

• trunk/src/wp-includes/class-wp-textdomain-registry.php (modified) (6 diffs)

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

@@ -52 +52 @@
          *
          * @since 6.1.0
+         * @since 6.5.0 This property is no longer used.
          *
          * @var array
+         *
+         * @deprecated
          */
         protected $cached_mo_files = array();
@@ -65 +68 @@
          */
         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 );
+        }
 
         /**
@@ -136 +151 @@
 
         /**
+         * 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.
          *
@@ -157 +272 @@
 
         /**
-         * 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
@@ -176 +291 @@
 
                 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 ) &&
@@ -216 +329 @@
                 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;
-                }
-        }
 }