Changeset 57831 for trunk/tests/phpunit/tests/l10n/wpTextdomainRegistry.php

Timestamp:

03/14/2024 09:04:00 AM (11 months ago)

Author:

swissspidy

Message:

I18N: Improve translation file cache group & expiration.

Adds an explicit 1 hour expiration for the translation file cache introduced in [57287] / #58919.
This prevents stale caches when a site does not use the regular way of installing language packs, for example when an atomic filesystem is involved.
Also configures the translation_files group as a global cache group on multisite.

Props dd32.
Fixes #60764.

File:

• trunk/tests/phpunit/tests/l10n/wpTextdomainRegistry.php (modified) (5 diffs)

trunk/tests/phpunit/tests/l10n/wpTextdomainRegistry.php

@@ -20 +20 @@
 
     public function tear_down() {
-        wp_cache_delete( 'cached_mo_files_' . md5( WP_LANG_DIR . '/foobar/' ), 'translations' );
-        wp_cache_delete( 'cached_mo_files_' . md5( WP_LANG_DIR . '/plugins/' ), 'translations' );
-        wp_cache_delete( 'cached_mo_files_' . md5( WP_LANG_DIR . '/themes/' ), 'translations' );
-        wp_cache_delete( 'cached_mo_files_' . md5( WP_LANG_DIR . '/' ), 'translations' );
+        wp_cache_delete( md5( WP_LANG_DIR . '/foobar/' ), 'translation_files' );
+        wp_cache_delete( md5( WP_LANG_DIR . '/plugins/' ), 'translation_files' );
+        wp_cache_delete( md5( WP_LANG_DIR . '/themes/' ), 'translation_files' );
+        wp_cache_delete( md5( WP_LANG_DIR . '/' ), 'translation_files' );
 
         parent::tear_down();
@@ -50 +50 @@
         );
         $this->assertNotFalse(
-            wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/bar/' ), 'translations' ),
+            wp_cache_get( md5( WP_LANG_DIR . '/bar/' ), 'translation_files' ),
             'List of files in custom path not cached'
         );
@@ -90 +90 @@
         $this->instance->get_language_files_from_path( WP_LANG_DIR . '/' );
 
-        $this->assertNotFalse( wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/plugins/' ), 'translations' ) );
-        $this->assertNotFalse( wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/themes/' ), 'translations' ) );
-        $this->assertNotFalse( wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/foobar/' ), 'translations' ) );
-        $this->assertNotFalse( wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/' ), 'translations' ) );
+        $this->assertNotFalse( wp_cache_get( md5( WP_LANG_DIR . '/plugins/' ), 'translation_files' ) );
+        $this->assertNotFalse( wp_cache_get( md5( WP_LANG_DIR . '/themes/' ), 'translation_files' ) );
+        $this->assertNotFalse( wp_cache_get( md5( WP_LANG_DIR . '/foobar/' ), 'translation_files' ) );
+        $this->assertNotFalse( wp_cache_get( md5( WP_LANG_DIR . '/' ), 'translation_files' ) );
     }
 
@@ -104 +104 @@
         remove_filter( 'pre_get_language_files_from_path', '__return_empty_array' );
 
-        $cache = wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/plugins/' ), 'translations' );
+        $cache = wp_cache_get( md5( WP_LANG_DIR . '/plugins/' ), 'translation_files' );
 
         $this->assertEmpty( $result );
@@ -145 +145 @@
         );
 
-        $this->assertFalse( wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/plugins/' ), 'translations' ) );
-        $this->assertFalse( wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/themes/' ), 'translations' ) );
-        $this->assertFalse( wp_cache_get( 'cached_mo_files_' . md5( WP_LANG_DIR . '/' ), 'translations' ) );
+        $this->assertFalse( wp_cache_get( md5( WP_LANG_DIR . '/plugins/' ), 'translation_files' ) );
+        $this->assertFalse( wp_cache_get( md5( WP_LANG_DIR . '/themes/' ), 'translation_files' ) );
+        $this->assertFalse( wp_cache_get( md5( WP_LANG_DIR . '/' ), 'translation_files' ) );
     }