Changeset 44418

Timestamp:

01/07/2019 01:59:01 PM (8 years ago)

Author:

ocean90

Message:

I18n: Expose JSON translation file paths in load_script_textdomain().

Removes file_exist() checks before calling load_script_translations() to let the determined paths be passed to load_script_translations() which provides its own file check and the possibility to filter the path.

Props swissspidy, johnbillion, ocean90.
See #45769.

File:

• trunk/src/wp-includes/l10n.php (modified) (2 diffs)

trunk/src/wp-includes/l10n.php

@@ -923 +923 @@
         $file_base       = $domain === 'default' ? $locale : $domain . '-' . $locale;
         $handle_filename = $file_base . '-' . $handle . '.json';
-        if ( $path && file_exists( $path . '/' . $handle_filename ) ) {
-                return load_script_translations( $path . '/' . $handle_filename, $handle, $domain );
+
+        if ( $path ) {
+                $translations = load_script_translations( $path . '/' . $handle_filename, $handle, $domain );
+
+                if ( $translations ) {
+                        return $translations;
+                }
         }
 
@@ -984 +989 @@
 
         $md5_filename = $file_base . '-' . md5( $relative ) . '.json';
-        if ( $path && file_exists( $path . '/' . $md5_filename ) ) {
-                return load_script_translations( $path . '/' . $md5_filename, $handle, $domain );
-        }
-        if ( file_exists( $languages_path . '/' . $md5_filename ) ) {
-                return load_script_translations( $languages_path . '/' . $md5_filename, $handle, $domain );
+
+        if ( $path ) {
+                $translations = load_script_translations( $path . '/' . $md5_filename, $handle, $domain );
+
+                if ( $translations ) {
+                        return $translations;
+                }
+        }
+
+        $translations = load_script_translations( $languages_path . '/' . $md5_filename, $handle, $domain );
+
+        if ( $translations ) {
+                return $translations;
         }