Changeset 9313

Timestamp:

12/05/2019 06:46:34 AM (7 years ago)

Author:

dd32

Message:

Translate: Only query for this locale.. not all of them.

This is the "more optimized" version that i intended to commit originally..

See r9312.
See #2768.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/inc/class-plugin.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/inc/class-plugin.php

@@ -370 +370 @@
          * @return string Filtered $content where any WordPress.org links have been replaced with $wp_locale subdomain links.
          */
-        function localize_links( $content, $wp_locale = false ) {
+        function localize_links( $content, $wp_locale ) {
                 global $wpdb;
 
-                static $subdomains = null;
-                if ( is_null( $subdomains ) && $wp_locale ) {
-                        $subdomains = $wpdb->get_results( 'SELECT locale, subdomain FROM wporg_locales', OBJECT_K );
-                }
-
-                if ( $wp_locale && isset( $subdomains[ $wp_locale ] ) ) {
+                static $subdomains = [];
+                if ( ! isset( $subdomains[ $wp_locale ] ) ) {
+                        $subdomain[ $wp_locale ] = $wpdb->get_var( $wpdb->prepare( 'SELECT subdomain FROM wporg_locales WHERE locale = %s LIMIT 1', $wp_locale ) );
+                }
+
+                if ( $subdomains[ $wp_locale ] ) {
                         $content = preg_replace(
                                 '!(?<=[\'"])https?://wordpress.org/!i', // Only match when it's a url within an attribute.
-                                'https://' . $subdomains[ $wp_locale ]->subdomain . '.wordpress.org/',
+                                'https://' . $subdomains[ $wp_locale ] . '.wordpress.org/',
                                 $content
                         );