Changeset 7114

Timestamp:

04/16/2018 04:07:08 PM (7 years ago)

Author:

Otto42

Message:

Showcase: Force images to use https URLs (some old entries have http ones), adjust screenshot image builder to not duplicate code.

File:

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

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

@@ -38 +38 @@
     $screenshot = apply_filters( 'wporg_showcase_screenshot_src', $screenshot, $post, $width );
 
+    // force screenshot URLs to be https
+    $screenshot = str_replace( 'http://', 'https://', $screenshot );
+
     if ( $echo ) {
         echo $screenshot;
@@ -49 +52 @@
     global $post;
 
-    $screenshot = get_post_meta($post->ID, 'screenshot', true);
-    if ( empty( $screenshot ) ) {
-        $screenshot = 'https://wordpress.com/mshots/v1/http%3A%2F%2F' . get_site_domain( true, false );
-    }
+    $screenshot = site_screenshot_src( $width, false );
     $srcset = $screenshot;