Changeset 1238 for sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-themes/functions.php

Timestamp:

02/10/2015 12:18:47 AM (10 years ago)

Author:

obenland

Message:

WP.org Themes: Use Photon for theme screenshots.

The Theme Directory feeds off of images to illustrate themes, so we will always
have a sizable footprint. In listings, screenshots are retina, so four times
the size they could be, on single they are over 700px wide. No reason not to
try our best to keep it managable though!

Using Photon will cut page weight to about half of what it is now.
We're adjusting the size to only what we need and remove all meta data.

See ​https://wordpress.slack.com/archives/meta/p1423518323000926

Fixes #845.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-themes/functions.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-themes/functions.php

@@ -241 +241 @@
 
     foreach ( $api->themes as &$theme ) {
-        $theme->name        = wp_kses( $theme->name,        $themes_allowedtags );
-        $theme->author      = wp_kses( $theme->author,      $themes_allowedtags );
-        $theme->version     = wp_kses( $theme->version,     $themes_allowedtags );
-        $theme->description = wp_kses( $theme->description, $themes_allowedtags );
-        $theme->num_ratings = number_format_i18n( $theme->num_ratings );
-        $theme->preview_url = set_url_scheme( $theme->preview_url );
+        $theme->name           = wp_kses( $theme->name,        $themes_allowedtags );
+        $theme->author         = wp_kses( $theme->author,      $themes_allowedtags );
+        $theme->version        = wp_kses( $theme->version,     $themes_allowedtags );
+        $theme->description    = wp_kses( $theme->description, $themes_allowedtags );
+        $theme->num_ratings    = number_format_i18n( $theme->num_ratings );
+        $theme->preview_url    = set_url_scheme( $theme->preview_url );
+        wporg_themes_photon_screen_shot( $theme );
     }
 
@@ -264 +265 @@
     }
 
-    $theme->name        = wp_kses( $theme->name,        $themes_allowedtags );
-    $theme->author      = wp_kses( $theme->author,      $themes_allowedtags );
-    $theme->version     = wp_kses( $theme->version,     $themes_allowedtags );
-    $theme->description = wp_kses( $theme->description, $themes_allowedtags );
-    $theme->num_ratings = number_format_i18n( $theme->num_ratings );
-    $theme->preview_url = set_url_scheme( $theme->preview_url );
+    $theme->name           = wp_kses( $theme->name,        $themes_allowedtags );
+    $theme->author         = wp_kses( $theme->author,      $themes_allowedtags );
+    $theme->version        = wp_kses( $theme->version,     $themes_allowedtags );
+    $theme->description    = wp_kses( $theme->description, $themes_allowedtags );
+    $theme->num_ratings    = number_format_i18n( $theme->num_ratings );
+    $theme->preview_url    = set_url_scheme( $theme->preview_url );
+    wporg_themes_photon_screen_shot( $theme );
 
     wp_send_json_success( $theme );
@@ -275 +277 @@
 add_action( 'wp_ajax_theme-info',        'wporg_themes_theme_info' );
 add_action( 'wp_ajax_nopriv_theme-info', 'wporg_themes_theme_info' );
+
+/**
+ * Photon-ifies the screen shot URL.
+ *
+ * @param object $theme
+ * @return object
+ */
+function wporg_themes_photon_screen_shot( $theme ) {
+    if ( preg_match( '/screenshot.(jpg|jpeg|png|gif)/', $theme->screenshot_url, $match ) ) {
+        $theme->screenshot_url = sprintf( 'https://i0.wp.com/themes.svn.wordpress.org/%1$s/%2$s/%3$s',
+            $theme->slug,
+            $theme->version,
+            $match[0]
+        );
+    }
+    return $theme;
+}
 
 /**