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 (6 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'] ) {