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

Timestamp:

05/03/2019 07:03:20 AM (7 years ago)

Author:

dd32

Message:

Plugin Directory: Introduce localised screenshots, allowing plugins to have translated screenshots on the website and within the wp-admin plugin installer.

This works by allowing screenshots located in the /assets/ folder to define the locale of the image, eg. screenshot-1-de_DE.png or screenshot-1-ary.png.

Props ck3lee, dd32.
See #3935.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/api/routes/class-plugin.php (modified) (1 diff)

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

@@ -155 +155 @@
                 $result['download_link']     = Template::download_link( $post );
 
-                $result['screenshots'] = array();
-                $descriptions          = get_post_meta( $post->ID, 'screenshots', true ) ?: array();
-                $screen_shots          = get_post_meta( $post->ID, 'assets_screenshots', true ) ?: array();
-
-                /*
-                 * Find the image that corresponds with the text.
-                 * The image numbers are stored within the 'resolution' key.
-                 */
-                foreach ( $screen_shots as $image ) {
-                        $src     = Template::get_asset_url( $post, $image );
-                        $caption = '';
-                        if ( $descriptions && ! empty( $descriptions[ (int) $image['resolution'] ] ) ) {
-                                $caption = $descriptions[ (int) $image['resolution'] ];
-                                $caption = Plugin_I18n::instance()->translate( 'screenshot-' . $image['resolution'], $caption, [ 'post_id' => $post->ID ] );
-                        }
-
-                        $result['screenshots'][ $image['resolution'] ] = compact( 'src', 'caption' );
-                }
+                // Reduce images to caption + src
+                $result['screenshots'] = array_map(
+                        function( $image ) {
+                                return [
+                                        'src'     => $image['src'],
+                                        'caption' => $image['caption'],
+                                ];
+                        },
+                        Template::get_screenshots( $post )
+                );
 
                 if ( $result['screenshots'] ) {