Changeset 9017

Timestamp:

07/02/2019 05:56:38 AM (6 years ago)

Author:

dd32

Message:

Plugin Directory: Cache the available language packs for a plugin. This doesn't affect the Language pack API's, only the public facing data in the directory.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory

Files:

• class-plugin-directory.php (modified) (3 diffs)
• class-plugin-i18n.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php

@@ -972 +972 @@
         $post = get_post( $post_id );
 
-        // Only translate Plugin post objects
+        // Only translate Plugin post objects.
         if ( $post && 'plugin' === $post->post_type ) {
             return Plugin_I18n::instance()->translate( $section, $content, [ 'post_id' => $post_id ] );
@@ -990 +990 @@
         $post = get_post( $post_id );
 
-        // Only translate Post type items.
+        // Only translate Plugin post objects.
         if ( $post && $post->post_type === 'plugin' ) {
             return Plugin_I18n::instance()->translate( 'title', $title, [ 'post_id' => $post ] );
@@ -1008 +1008 @@
         $post = get_post( $post );
 
-        // Only translate Post type items.
+        // Only translate Plugin post objects.
         if ( $post && $post->post_type === 'plugin' ) {
             return Plugin_I18n::instance()->translate( 'excerpt', $excerpt, [ 'post_id' => $post ] );

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-i18n.php

@@ -539 +539 @@
 
         // Get the active language packs of the plugin.
-        $locales = $wpdb->get_col( $wpdb->prepare( "
-            SELECT `language`
-            FROM language_packs
-            WHERE
-                type = 'plugin' AND
-                domain = %s AND
-                active = 1
-            GROUP BY `language`",
-            $plugin_slug
-        ) );
+        $locales = wp_cache_get( $plugin_slug, 'language-pack-locales' );
+        if ( $locales === false ) {
+            $locales = $wpdb->get_col( $wpdb->prepare( "
+                SELECT `language`
+                FROM language_packs
+                WHERE
+                    type = 'plugin' AND
+                    domain = %s AND
+                    active = 1
+                GROUP BY `language`",
+                $plugin_slug
+            ) );
+
+            wp_cache_set( $plugin_slug, $locales, 'language-pack-locales', HOUR_IN_SECONDS );
+        }
 
         $translations = [];