Changeset 9313

Timestamp:

12/05/2019 06:46:34 AM (5 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
             );