Changeset 10100

Timestamp:

07/27/2020 06:14:28 AM (5 years ago)

Author:

tellyworth

Message:

Plugin dir: include translation data in API response for block plugins.

This is needed for one potential solution to #50732-core. Might be reverted in future if a different solution is used.

Props tellyworth, ryelle, dd32.

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory

Files:

• api/routes/class-plugin.php (modified) (1 diff)
• standalone/class-plugins-info-api-request.php (modified) (1 diff)
• standalone/class-plugins-info-api.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-plugin.php

@@ -245 +245 @@
                 ];
             }, $result['language_packs'] );
+        }
+
+        // Include json translation data directly for block plugins, so it's immediately available on insert
+        if ( !empty( $result['language_pack'] ) && !empty( $result['block_assets'] ) ) {
+            foreach ( $result['block_assets'] as $file ) {
+                if ( 'js' === pathinfo( $file, PATHINFO_EXTENSION ) ) {
+                    // Look inside language pack zip files for a correctly names .json
+                    foreach ( $result['language_pack'] as $lp ) {
+                        $version_path = 'trunk' === $result['stable_tag'] ? 'trunk' : 'tags/' . $result['stable_tag'];
+                        $_file = str_replace( "/{$version_path}/", '', $file );
+                        $zip_path = '/nfs/rosetta/builds/plugins/' . $result['slug'] . '/' . $result['version'] . '/' . $lp['language'] . '.zip';
+                        $json_file = $result['slug'] . '-' . $lp['language'] . '-' . md5( $_file ) . '.json';
+                        if ( $fp = fopen( "zip://$zip_path#$json_file", 'r' ) ) {
+                            $json = fread( $fp, 100 * KB_IN_BYTES ); // max 100kb
+                            if ( $json && feof($fp) ) {
+                                // Note that we're intentionally not decoding the json here, as we don't want to alter it by converting to and from PHP data types.
+                                $result['block_translations'][ $lp['language'] ][ $file ] = $json;
+                            }
+                            fclose( $fp );
+                        }
+                    }
+                }
+            }
         }
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/standalone/class-plugins-info-api-request.php

@@ -106 +106 @@
         'blocks'               => true,
         'block_assets'         => true,
+        'block_translations'   => true,
         'author_block_count'   => true,
         'author_block_rating'  => true,

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/standalone/class-plugins-info-api.php

@@ -178 +178 @@
                 }
             }
+        }
+
+        // Only include block translation data for the selected locale.
+        if ( ! empty( $response['block_translations'] ) ) {
+            $response['block_translations'] = !empty( $response['block_translations'][ $request->locale ] ) ? $response['block_translations'][ $request->locale ] : [];
         }