Changeset 38364

Timestamp:

08/26/2016 10:19:26 AM (9 years ago)

Author:

wonderboymusic

Message:

i18n: move is_rtl() to l10n.php (which loads way earlier). Load WP_Locale file in wp-settings.php. Retire wp-includes/locale.php - it only loaded the class and the one function, is_rtl(). If someone loaded this file for fun somewhere else, it would be a fatal error.

See #37827.

Location:

trunk/src

Files:

• wp-admin/includes/update-core.php (modified) (1 diff)
• wp-includes/l10n.php (modified) (1 diff)
• wp-includes/locale.php (deleted)
• wp-settings.php (modified) (2 diffs)

trunk/src/wp-admin/includes/update-core.php

@@ -706 +706 @@
 // 4.6
 'wp-admin/includes/class-wp-automatic-upgrader.php', // Wrong file name, see #37628.
+// 4.7
+'wp-includes/locale.php',
 );
 

trunk/src/wp-includes/l10n.php

@@ -1133 +1133 @@
     return $output;
 }
+
+/**
+ * Checks if current locale is RTL.
+ *
+ * @since 3.0.0
+ *
+ * @global WP_Locale $wp_locale
+ *
+ * @return bool Whether locale is RTL.
+ */
+function is_rtl() {
+    global $wp_locale;
+    return $wp_locale->is_rtl();
+}

trunk/src/wp-settings.php

@@ -214 +214 @@
 require( ABSPATH . WPINC . '/rest-api/class-wp-rest-response.php' );
 require( ABSPATH . WPINC . '/rest-api/class-wp-rest-request.php' );
+require( ABSPATH . WPINC . '/class-wp-locale.php' );
 
 $GLOBALS['wp_embed'] = new WP_Embed();
@@ -374 +375 @@
 unset( $locale_file );
 
-// Pull in locale data after loading text domain.
-require_once( ABSPATH . WPINC . '/locale.php' );
-
 /**
  * WordPress Locale object for loading locale domain date and various strings.