Add dynamic social share images for plugin pages (Meta #5926)

environments/plugin-directory/.wp-env.json

@@ -15,7 +15,8 @@
 		"wp-content/mu-plugins": "./mocks/mu-plugins",
 		"wp-content/mu-plugins/pub": "../wordpress.org/public_html/wp-content/mu-plugins/pub",
 		"wp-content/mu-plugins/wporg-mu-plugins": "WordPress/wporg-mu-plugins#build",
-		"wp-content/env-bin": "./plugin-directory/bin"
+		"wp-content/env-bin": "./plugin-directory/bin",
+		"style": "../wordpress.org/public_html/style"
 	},
 	"lifecycleScripts": {
 		"afterStart": "bash plugin-directory/bin/after-start.sh"

environments/plugin-directory/.wp-env.test.json

@@ -6,6 +6,7 @@
 		"../wordpress.org/public_html/wp-content/plugins/plugin-directory"
 	],
 	"mappings": {
+		"wp-content/mu-plugins/wporg-mu-plugins": "WordPress/wporg-mu-plugins#build",
 		"wp-content/env-bin": "./plugin-directory/bin"
 	},
 	"lifecycleScripts": {

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

@@ -116,27 +116,7 @@ public function plugin_info_data( $request, $post ) {
 		$result['author_profile'] = $profile_url;
 		$result['contributors']   = array();
 
-		$contributors = get_terms( array(
-			'taxonomy'   => 'plugin_contributors',
-			'object_ids' => array( $post->ID ),
-			'orderby'    => 'term_order',
-			'fields'     => 'names',
-		) );
-
-		if ( is_wp_error( $contributors ) ) {
-			$contributors = array();
-		}
-
-		if ( ! $contributors ) {
-			$contributors = [ $author->user_nicename ];
-		}
-
-		foreach ( $contributors as $contributor ) {
-			$user = get_user_by( 'slug', $contributor );
-			if ( ! $user ) {
-				continue;
-			}
-
+		foreach ( Template::get_plugin_contributors( $post ) as $user ) {
 			$result['contributors'][ $user->user_nicename ] = array(
 				'profile'      => $this->get_user_profile_link( $user ),
 				'avatar'       => get_avatar_url( $user, array(

wordpress.org/public_html/wp-content/plugins/plugin-directory/class-plugin-directory.php

@@ -42,9 +42,12 @@ private function __construct() {
 		add_filter( 'pre_update_option_jetpack_options', array( $this, 'filter_jetpack_options' ) );
 		add_filter( 'jetpack_sitemap_post_types', array( $this, 'jetpack_sitemap_post_types' ) );
 		add_filter( 'jetpack_sitemap_skip_post', array( $this, 'jetpack_sitemap_skip_post' ), 10, 2 );
+		add_filter( 'jetpack_enable_open_graph', array( $this, 'disable_jetpack_open_graph_for_plugins' ), 99 );
+		add_filter( 'jetpack_disable_twitter_cards', array( $this, 'disable_jetpack_twitter_cards_for_plugins' ), 99 );
 		add_action( 'template_redirect', array( $this, 'prevent_canonical_for_plugins' ), 9 );
 		add_action( 'template_redirect', array( $this, 'custom_redirects' ), 1 );
 		add_action( 'template_redirect', array( $this, 'geopattern_icon_route' ), 0 );
+		add_action( 'template_redirect', array( $this, 'share_image_route' ), 0 );
 		add_filter( 'query_vars', array( $this, 'filter_query_vars' ), 1 );
 		add_filter( 'single_term_title', array( $this, 'filter_single_term_title' ) );
 		add_filter( 'get_the_archive_title_prefix', array( $this, 'filter_get_the_archive_title_prefix' ) );
@@ -428,6 +431,9 @@ public function init() {
 		// Add a rule for generated plugin icons. geopattern-icon/demo.svg | geopattern-icon/demo_abc123.svg
 		add_rewrite_rule( '^geopattern-icon/([^/_]+)(_([a-f0-9]{6}))?\.svg$', 'index.php?name=$matches[1]&geopattern_icon=$matches[3]', 'top' );
 
+		// Share images. share-image/demo.jpg | share-image/demo_a1b2c3d4.jpg (token is a cache buster).
+		add_rewrite_rule( '^share-image/([^/_]+)(_([a-f0-9]{8}))?\.jpg$', 'index.php?plugin_share_image=$matches[1]', 'top' );
+
 		// Handle plugin admin requests
 		add_rewrite_rule( '^([^/]+)/advanced/?$', 'index.php?name=$matches[1]&plugin_advanced=1', 'top' );
 
@@ -1207,6 +1213,7 @@ public function filter_query_vars( $vars ) {
 		$vars[] = 'redirect_plugin_tab';
 		$vars[] = 'plugin_advanced';
 		$vars[] = 'geopattern_icon';
+		$vars[] = 'plugin_share_image';
 		$vars[] = 'block_search';
 
 		// Remove support for any query vars the Plugin Directory doesn't support/need on the front-end.
@@ -1509,6 +1516,30 @@ function geopattern_icon_route() {
 		die();
 	}
 
+	/**
+	 * Output a JPEG share image for a given plugin.
+	 */
+	public function share_image_route() {
+		$slug = get_query_var( 'plugin_share_image' );
+
+		if ( ! $slug ) {
+			return;
+		}
+
+		$plugin = self::get_plugin_post( $slug );
+		if ( ! $plugin || 'publish' !== $plugin->post_status ) {
+			status_header( 404 );
+			die();
+		}
+
+		if ( ! Plugin_Share_Image::output( $plugin ) ) {
+			status_header( 500 );
+			die();
+		}
+
+		die();
+	}
+
 	/**
 	 * The array of post types to be included in the sitemap.
 	 *
@@ -1543,6 +1574,44 @@ public function jetpack_sitemap_skip_post( $skip, $plugin_db_row ) {
 		return $skip;
 	}
 
+	/**
+	 * Disable Jetpack Open Graph on plugin pages.
+	 *
+	 * The theme outputs custom Open Graph tags, including the dynamic share image.
+	 *
+	 * @param bool $enabled Whether Jetpack Open Graph is enabled.
+	 * @return bool
+	 */
+	public function disable_jetpack_open_graph_for_plugins( $enabled ) {
+		if ( ! did_action( 'wp' ) ) {
+			return $enabled;
+		}
+
+		if ( is_singular( 'plugin' ) ) {
+			return false;
+		}
+
+		return $enabled;
+	}
+
+	/**
+	 * Disable Jetpack Twitter Cards on plugin pages.
+	 *
+	 * @param bool $disabled Whether Jetpack Twitter Cards are disabled.
+	 * @return bool
+	 */
+	public function disable_jetpack_twitter_cards_for_plugins( $disabled ) {
+		if ( ! did_action( 'wp' ) ) {
+			return $disabled;
+		}
+
+		if ( is_singular( 'plugin' ) ) {
+			return true;
+		}
+
+		return $disabled;
+	}
+
 	/**
 	 * Whitelists the oembed providers whitelist.
 	 *