Use proper URL for resized images in the ImageObject schema

composer.json

@@ -91,7 +91,7 @@
 			"Yoast\\WP\\SEO\\Composer\\Actions::check_coding_standards"
 		],
 		"check-cs-thresholds": [
-			"@putenv YOASTCS_THRESHOLD_ERRORS=2481",
+			"@putenv YOASTCS_THRESHOLD_ERRORS=2477",
 			"@putenv YOASTCS_THRESHOLD_WARNINGS=252",
 			"Yoast\\WP\\SEO\\Composer\\Actions::check_cs_thresholds"
 		],

src/generators/schema/main-image.php

@@ -38,7 +38,7 @@ public function generate() {
 
 		// The first image in the content.
 		if ( $this->context->main_image_url ) {
-			return $this->helpers->schema->image->generate_from_url( $image_id, $this->context->main_image_url );
+			return $this->helpers->schema->image->generate_from_url( $image_id, $this->context->main_image_url, '', false, true, true );
 		}
 
 		return false;

src/helpers/schema/image-helper.php

@@ -48,23 +48,58 @@ public function __construct( HTML_Helper $html, Language_Helper $language, Main_
 	/**
 	 * Find an image based on its URL and generate a Schema object for it.
 	 *
-	 * @param string $schema_id      The `@id` to use for the returned image.
-	 * @param string $url            The image URL to base our object on.
-	 * @param string $caption        An optional caption.
-	 * @param bool   $add_hash       Whether a hash will be added as a suffix in the @id.
-	 * @param bool   $use_link_table Whether the SEO Links table will be used to retrieve the id.
-	 *
-	 * @return array Schema ImageObject array.
+	 * @param string $schema_id         The `@id` to use for the returned image.
+	 * @param string $url               The image URL to base our object on.
+	 * @param string $caption           An optional caption.
+	 * @param bool   $add_hash          Whether a hash will be added as a suffix in the @id.
+	 * @param bool   $use_link_table    Whether the SEO Links table will be used to retrieve the id.
+	 * @param bool   $check_for_resized Whether to account for resized images.
+	 *
+	 * @return string[] Schema ImageObject array.
 	 */
-	public function generate_from_url( $schema_id, $url, $caption = '', $add_hash = false, $use_link_table = true ) {
+	public function generate_from_url( $schema_id, $url, $caption = '', $add_hash = false, $use_link_table = true, $check_for_resized = false ) {
 		$attachment_id = $this->image->get_attachment_by_url( $url, $use_link_table );
 		if ( $attachment_id > 0 ) {
+			// First check if we should use the resized image and whether it is a resized image indeed.
+			if ( $check_for_resized && \preg_match( '/-(\d+)x(\d+)\.(jpg|jpeg|png|gif|webp)$/', $url ) ) {
+				return $this->generate_from_resized_url( $schema_id, $attachment_id, $caption, $add_hash, $url );
+			}
+
 			return $this->generate_from_attachment_id( $schema_id, $attachment_id, $caption, $add_hash );
 		}
 
 		return $this->simple_image_object( $schema_id, $url, $caption, $add_hash );
 	}
 
+	/**
+	 * Retrieve data about an image from the database and use it to generate a Schema object.
+	 *
+	 * @param string $schema_id     The `@id` to use for the returned image.
+	 * @param int    $attachment_id The attachment to retrieve data from.
+	 * @param string $caption       The caption string, if there is one.
+	 * @param bool   $add_hash      Whether a hash will be added as a suffix in the @id.
+	 * @param string $resized_url   The url of the resized image.
+	 *
+	 * @return string[] Schema ImageObject array.
+	 */
+	public function generate_from_resized_url( $schema_id, $attachment_id, $caption = '', $add_hash = false, $resized_url = '' ) {
+		$data = $this->generate_object();
+
+		$id_suffix = ( $add_hash ) ? \md5( $resized_url ) : '';
+
+		$data['@id']        = $schema_id . $id_suffix;
+		$data['url']        = $resized_url;
+		$data['contentUrl'] = $resized_url;
+
+		\preg_match( '/-(\d+)x(\d+)\.(jpg|jpeg|png|gif|webp)$/', $resized_url, $matches );
+		$data['width']  = $matches[1];
+		$data['height'] = $matches[2];
+
+		$data = $this->add_caption( $data, $attachment_id, $caption );
+
+		return $data;
+	}
+
 	/**
 	 * Retrieve data about an image from the database and use it to generate a Schema object.
 	 *
@@ -73,7 +108,7 @@ public function generate_from_url( $schema_id, $url, $caption = '', $add_hash =
 	 * @param string $caption       The caption string, if there is one.
 	 * @param bool   $add_hash      Whether a hash will be added as a suffix in the @id.
 	 *
-	 * @return array Schema ImageObject array.
+	 * @return string[] Schema ImageObject array.
 	 */
 	public function generate_from_attachment_id( $schema_id, $attachment_id, $caption = '', $add_hash = false ) {
 		$data = $this->generate_object();
@@ -98,7 +133,7 @@ public function generate_from_attachment_id( $schema_id, $attachment_id, $captio
 	 * @param string $caption         The caption string, if there is one.
 	 * @param bool   $add_hash        Whether a hash will be added as a suffix in the @id.
 	 *
-	 * @return array Schema ImageObject array.
+	 * @return string[] Schema ImageObject array.
 	 */
 	public function generate_from_attachment_meta( $schema_id, $attachment_meta, $caption = '', $add_hash = false ) {
 		$data = $this->generate_object();
@@ -125,7 +160,7 @@ public function generate_from_attachment_meta( $schema_id, $attachment_meta, $ca
 	 * @param string $caption   A caption, if set.
 	 * @param bool   $add_hash  Whether a hash will be added as a suffix in the @id.
 	 *
-	 * @return array Schema ImageObject array.
+	 * @return string[] Schema ImageObject array.
 	 */
 	public function simple_image_object( $schema_id, $url, $caption = '', $add_hash = false ) {
 		$data = $this->generate_object();

tests/Unit/Generators/Schema/Main_Image_Test.php

@@ -137,7 +137,7 @@ public function test_generate_from_url() {
 
 		$this->schema_image->expects( 'generate_from_url' )
 			->once()
-			->with( $image_id, $image_url )
+			->with( $image_id, $image_url, '', false, true, true )
 			->andReturn( $image_schema );
 
 		self::assertEquals( $image_schema, $this->instance->generate() );

tests/Unit/Helpers/Schema/Image_Helper_Test.php

@@ -88,6 +88,32 @@ public function test_generate_from_url_with_found_attachment_id() {
 		);
 	}
 
+	/**
+	 * Tests generating the schema from url with a found attachment id.
+	 *
+	 * @covers ::generate_from_url
+	 *
+	 * @return void
+	 */
+	public function test_generate_from_resized_url_with_found_attachment_id() {
+		$this->image
+			->expects( 'get_attachment_by_url' )
+			->once()
+			->with( 'https://example.org/image-300x300.jpg', true )
+			->andReturn( 1337 );
+
+		$this->instance
+			->expects( 'generate_from_resized_url' )
+			->once()
+			->with( '#schema-image-ABC', 1337, '', false, 'https://example.org/image-300x300.jpg' )
+			->andReturn( [] );
+
+		$this->assertEquals(
+			[],
+			$this->instance->generate_from_url( '#schema-image-ABC', 'https://example.org/image-300x300.jpg', '', false, true, true )
+		);
+	}
+
 	/**
 	 * Tests generating the schema from url no found attachment id.
 	 *
@@ -168,6 +194,53 @@ public function test_generate_from_attachment_id_with_caption_and_image_dimensio
 		);
 	}
 
+	/**
+	 * Tests the generate_from_resized_url method.
+	 *
+	 * @covers ::generate_from_resized_url
+	 * @covers ::generate_object
+	 * @covers ::add_image_size
+	 * @covers ::add_caption
+	 *
+	 * @return void
+	 */
+	public function test_generate_from_resized_url() {
+		$this->image
+			->expects( 'get_attachment_image_url' )
+			->never();
+
+		$this->image
+			->expects( 'get_metadata' )
+			->never();
+
+		$this->language
+			->expects( 'add_piece_language' )
+			->once()
+			->andReturnUsing( [ $this, 'set_language' ] );
+
+		$expected = [
+			'@type'      => 'ImageObject',
+			'@id'        => 'https://example.com/#/schema/logo/image/',
+			'url'        => 'https://example.com/logo-300x300.jpg',
+			'contentUrl' => 'https://example.com/logo-300x300.jpg',
+			'width'      => 300,
+			'height'     => 300,
+			'caption'    => 'Caption',
+			'inLanguage' => 'language',
+		];
+
+		$this->assertEquals(
+			$expected,
+			$this->instance->generate_from_resized_url(
+				'https://example.com/#/schema/logo/image/',
+				1337,
+				'Caption',
+				false,
+				'https://example.com/logo-300x300.jpg'
+			)
+		);
+	}
+
 	/**
 	 * Tests the generate_from_attachment_id method.
 	 *