Changeset 2611 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-screenshots.php

Timestamp:

02/25/2016 05:03:19 AM (10 years ago)

Author:

dd32

Message:

Plugin Directory: Switch to using namespaces instead of implemeting them through classnames.
This change introduces an autoloader and relies upon it for most file inclusions, this should encourage us to keep a standard naming schema and writing more component classes going forward.
See #1584

Location:

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

Files:

• . (copied) (copied from sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes )
• class-screenshots.php (moved) (moved from sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/screenshots.php ) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/shortcodes/class-screenshots.php

@@ -1 +1 @@
 <?php
-add_shortcode( 'wporg-plugins-screenshots', 'wporg_plugins_screenshots' );
+namespace WordPressdotorg\Plugin_Directory\Shortcodes;
+use WordPressdotorg\Plugin_Directory\Template;
 
-function wporg_plugins_screenshots() {
-        $plugin = get_post();
+/**
+ * The [wporg-plugins-screenshots] shortcode handler to display a plugins screenshots.
+ *
+ * @package WordPressdotorg_Plugin_Directory
+ */
+class Screenshots {
+        static function display() {
+                $plugin = get_post();
 
-        // All indexed from 1
-        $screenshot_descriptions = get_post_meta( $plugin->ID, 'screenshots', true );
-        $assets_screenshots = get_post_meta( $plugin->ID, 'assets_screenshots', true );
+                // All indexed from 1
+                $screenshot_descriptions = get_post_meta( $plugin->ID, 'screenshots', true );
+                $assets_screenshots = get_post_meta( $plugin->ID, 'assets_screenshots', true );
 
-        $output = '';
-        foreach ( $screenshot_descriptions as $index => $description ) {
-                // Find the image that corresponds with the text.
-                // The image numbers are stored within the 'resolution' key.
-                $found = false;
-                foreach ( $assets_screenshots as $image ) {
-                        if ( $index == $image['resolution'] ) {
-                                $found = true;
-                                break;
+                $output = '';
+                foreach ( $screenshot_descriptions as $index => $description ) {
+                        // Find the image that corresponds with the text.
+                        // The image numbers are stored within the 'resolution' key.
+                        $found = false;
+                        foreach ( $assets_screenshots as $image ) {
+                                if ( $index == $image['resolution'] ) {
+                                        $found = true;
+                                        break;
+                                }
                         }
-                }
-                if ( ! $found ) {
-                        continue;
-                }
+                        if ( ! $found ) {
+                                continue;
+                        }
 
-                if ( ! empty( $image['location'] ) && 'plugin' == $image['location'] ) {
-                        // Screenshot is within the plugin folder
-                        $url = sprintf(
-                                'https://s.w.org/plugins/%s/%s?rev=%s',
-                                $plugin->post_name,
-                                $image['filename'],
-                                $image['revision']
-                        );
-                } else {
-                        // In the /assets/ folder
-                        $url = sprintf(
-                                'https://ps.w.org/%s/assets/%s?rev=%s',
-                                $plugin->post_name,
-                                $image['filename'],
-                                $image['revision']
+                        $url = Template::get_asset_url( $plugin->post_name, $image );
+
+                        $output .= sprintf(
+                                '<li>
+                                        <a href="%1$s" rel="nofollow">
+                                                <img class="screenshot" src="%1$s">
+                                        </a>
+                                        <p>%2$s</p>
+                                </li>',
+                                $url,
+                                $description
                         );
                 }
 
-                $output .= sprintf(
-                        '<li>
-                                <a href="%1$s" rel="nofollow">
-                                        <img class="screenshot" src="%1$s">
-                                </a>
-                                <p>%2$s</p>
-                        </li>',
-                        $url,
-                        $description
-                );
+                return '<ol class="screenshots">' . $output . '</ol>';
+
         }
-
-        return '<ol class="screenshots">' . $output . '</ol>';
-
 }