Changeset 44418

Timestamp:

01/07/2019 01:59:01 PM (6 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;
     }