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

Timestamp:

12/19/2016 07:17:41 AM (10 years ago)

Author:

dd32

Message:

Plugin Directory: Expose Screenshots in the sections section of the API response so WordPress can display them.

See #2112

File:

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

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

@@ -111 +111 @@
                         $result['sections'][ $_pages[ $i ] ] = apply_filters( 'the_content', $_pages[ $i + 1 ], $_pages[ $i ] );
                 }
+                $result['sections']['screenshots'] = ''; // placeholder to put screenshots prior to reviews at the end.
                 $result['sections']['reviews'] = $this->get_plugin_reviews_markup( $post->post_name );
+
                 if ( !empty( $result['sections']['faq'] ) ) {
-                        $result['sections']['enhanced_faq'] = $result['sections']['faq'];
-                        $result['sections']['faq'] = $this->get_simplified_faq_markup( $result['sections']['enhanced_faq'] );
+                        $result['sections']['faq'] = $this->get_simplified_faq_markup( $result['sections']['faq'] );
                 } 
                 
@@ -135 +136 @@
                                 'caption' => array_key_exists( $image['resolution'], $descriptions ) ? $descriptions[ $image['resolution'] ] : ''
                         );
+                }
+
+                if ( $result['screenshots'] ) {
+                        $result['sections']['screenshots'] = $this->get_screenshot_markup( $result['screenshots'] );
+                } else {
+                        unset( $result['sections']['screenshots'] );
                 }
 
@@ -294 +301 @@
         }
 
+        protected function get_screenshot_markup( $screenshots ) {
+                $markup = '<ol>';
+
+                foreach ( $screenshots as $shot ) {
+                        if ( $shot['caption'] ) {
+                                $markup .= sprintf(
+                                        '<li><a href="%1$s"><img src="%1$s" alt="%2$s"></a><p>%3$s</p></li>',
+                                        esc_attr( $shot['src'] ),
+                                        esc_attr( $shot['caption'] ),
+                                        $shot['caption']
+                                );
+                        } else {
+                                $markup .= sprintf(
+                                        '<li><a href="%1$s"><img src="%1$s" alt=""></a></li>',
+                                        esc_attr( $shot['src'] )
+                                );
+                        }
+                }
+
+                $markup .= '</ol>';
+                return $markup;
+        }
+
 }