insideout10 / wordlift-plugin

src/includes/class-wordlift-attachment-service.php

Indentation +169 added lines, -169 removed lines

@@ -16,85 +16,85 @@ 
  */
 class Wordlift_Attachment_Service {
 
-	/**
-	 * The singleton instance.
-	 *
-	 * @since 3.20.0
-	 * @access private
-	 * @var \Wordlift_Attachment_Service The singleton instance.
-	 */
-	private static $instance;
-
-	/**
-	 * Create a {@link Wordlift_Attachment_Service} instance.
-	 *
-	 * @since 3.20.0
-	 */
-	public function __construct() {
-
-		self::$instance = $this;
-
-	}
-
-	/**
-	 * Get the singleton instance.
-	 *
-	 * @return \Wordlift_Attachment_Service The singleton instance.
-	 * @since 3.20.0
-	 *
-	 */
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	/**
-	 * Get an attachment ID given a URL.
-	 *
-	 * Inspired from https://wpscholar.com/blog/get-attachment-id-from-wp-image-url/
-	 *
-	 * @param string $url The attachment URL.
-	 *
-	 * @return int|false Attachment ID on success, false on failure
-	 * @since 3.10.0
-	 *
-	 */
-	public function get_attachment_id( $url ) {
-
-		// Get the upload directory data, we need the base URL to check whether
-		// the URL we received is within WP.
-		$dir = wp_upload_dir();
-
-		// Get the filename, the extension is kept.
-		if ( 1 !== preg_match( "@^{$dir['baseurl']}/(.+?)(?:-\d+x\d+)?(\.\w+)$@", $url, $matches ) ) {
-			return false;
-		}
-
-		$filename = $matches[1] . $matches[2];
-
-		// The following query is CPU-intensive, we need to review it.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/689.
-		//
-		// Query for attachments with the specified filename.
-		$query = new WP_Query( array(
-			'post_type'           => 'attachment',
-			'post_status'         => 'inherit',
-			'fields'              => 'ids',
-			'meta_query'          => array(
-				array(
-					'value'   => $filename,
-					'compare' => '=',
-					'key'     => '_wp_attached_file',
-				),
-			),
-			'posts_per_page'      => 1,
-			'ignore_sticky_posts' => true,
-		) );
-
-		// If there are no posts, return.
-		if ( $query->have_posts() ) {
-			return $query->posts[0];
+    /**
+     * The singleton instance.
+     *
+     * @since 3.20.0
+     * @access private
+     * @var \Wordlift_Attachment_Service The singleton instance.
+     */
+    private static $instance;
+
+    /**
+     * Create a {@link Wordlift_Attachment_Service} instance.
+     *
+     * @since 3.20.0
+     */
+    public function __construct() {
+
+        self::$instance = $this;
+
+    }
+
+    /**
+     * Get the singleton instance.
+     *
+     * @return \Wordlift_Attachment_Service The singleton instance.
+     * @since 3.20.0
+     *
+     */
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    /**
+     * Get an attachment ID given a URL.
+     *
+     * Inspired from https://wpscholar.com/blog/get-attachment-id-from-wp-image-url/
+     *
+     * @param string $url The attachment URL.
+     *
+     * @return int|false Attachment ID on success, false on failure
+     * @since 3.10.0
+     *
+     */
+    public function get_attachment_id( $url ) {
+
+        // Get the upload directory data, we need the base URL to check whether
+        // the URL we received is within WP.
+        $dir = wp_upload_dir();
+
+        // Get the filename, the extension is kept.
+        if ( 1 !== preg_match( "@^{$dir['baseurl']}/(.+?)(?:-\d+x\d+)?(\.\w+)$@", $url, $matches ) ) {
+            return false;
+        }
+
+        $filename = $matches[1] . $matches[2];
+
+        // The following query is CPU-intensive, we need to review it.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/689.
+        //
+        // Query for attachments with the specified filename.
+        $query = new WP_Query( array(
+            'post_type'           => 'attachment',
+            'post_status'         => 'inherit',
+            'fields'              => 'ids',
+            'meta_query'          => array(
+                array(
+                    'value'   => $filename,
+                    'compare' => '=',
+                    'key'     => '_wp_attached_file',
+                ),
+            ),
+            'posts_per_page'      => 1,
+            'ignore_sticky_posts' => true,
+        ) );
+
+        // If there are no posts, return.
+        if ( $query->have_posts() ) {
+            return $query->posts[0];
 //			foreach ( $query->posts as $attachment_id ) {
 //
 //				// Get the attachment metadata, we need the filename.
@@ -109,95 +109,95 @@ 
 //					return $attachment_id;
 //				}
 //			}
-		}
-
-		// If we got here, we couldn't find any attachment.
-		return false;
-	}
-
-	/**
-	 * Get images embedded in the post content.
-	 *
-	 * @param string $content The post content.
-	 *
-	 * @return array An array of attachment ids.
-	 * @since 3.10.0
-	 *
-	 */
-	public function get_image_embeds( $content ) {
-
-		// Go over all the images included in the post content, check if they are
-		// in the DB, and if so include them.
-		$images = array();
-		if ( false === preg_match_all( '#<img [^>]*src="([^\\">]*)"[^>]*>#', $content, $images ) ) {
-			return array();
-		}
-
-		// Map the image URLs to attachment ids.
-		$that = $this;
-		$ids  = array_map( function ( $url ) use ( $that ) {
-			return $that->get_attachment_id( $url );
-		}, $images[1] );
-
-		// Filter out not found ids (i.e. id is false).
-		return array_filter( $ids, function ( $item ) {
-			return false !== $item;
-		} );
-	}
-
-	/**
-	 * Get images linked via the `gallery` shortcode.
-	 *
-	 * @param \WP_Post $post A {@link WP_Post} instance.
-	 *
-	 * @return array An array of attachment ids.
-	 * @since 3.10.0
-	 *
-	 */
-	public function get_gallery( $post ) {
-
-		// @todo: the `gallery` shortcode has an `exclude` attribute which isn't
-		// checked at the moment.
-
-		// Prepare the return value.
-		$ids = array();
-
-		// As the above for images in galleries.
-		// Code inspired by http://wordpress.stackexchange.com/questions/80408/how-to-get-page-post-gallery-attachment-images-in-order-they-are-set-in-backend
-		$pattern = get_shortcode_regex();
-
-		if ( preg_match_all( '/' . $pattern . '/s', $post->post_content, $matches )
-		     && array_key_exists( 2, $matches )
-		     && in_array( 'gallery', $matches[2] )
-		) {
-
-			$keys = array_keys( $matches[2], 'gallery' );
-
-			foreach ( $keys as $key ) {
-				$atts = shortcode_parse_atts( $matches[3][ $key ] );
-
-				if ( is_array( $atts ) && array_key_exists( 'ids', $atts ) ) {
-					// gallery images insert explicitly by their ids.
-
-					foreach ( explode( ',', $atts['ids'] ) as $attachment_id ) {
-						// Since we do not check for actual image existence
-						// when generating the json content, check it now.
-						if ( wp_get_attachment_image_src( $attachment_id, 'full' ) ) {
-							$ids[ $attachment_id ] = true;
-						}
-					}
-				} else {
-					// gallery shortcode with no ids uses all the images
-					// attached to the post.
-					$images = get_attached_media( 'image', $post->ID );
-					foreach ( $images as $attachment ) {
-						$ids[ $attachment->ID ] = true;
-					}
-				}
-			}
-		}
-
-		return array_keys( $ids );
-	}
+        }
+
+        // If we got here, we couldn't find any attachment.
+        return false;
+    }
+
+    /**
+     * Get images embedded in the post content.
+     *
+     * @param string $content The post content.
+     *
+     * @return array An array of attachment ids.
+     * @since 3.10.0
+     *
+     */
+    public function get_image_embeds( $content ) {
+
+        // Go over all the images included in the post content, check if they are
+        // in the DB, and if so include them.
+        $images = array();
+        if ( false === preg_match_all( '#<img [^>]*src="([^\\">]*)"[^>]*>#', $content, $images ) ) {
+            return array();
+        }
+
+        // Map the image URLs to attachment ids.
+        $that = $this;
+        $ids  = array_map( function ( $url ) use ( $that ) {
+            return $that->get_attachment_id( $url );
+        }, $images[1] );
+
+        // Filter out not found ids (i.e. id is false).
+        return array_filter( $ids, function ( $item ) {
+            return false !== $item;
+        } );
+    }
+
+    /**
+     * Get images linked via the `gallery` shortcode.
+     *
+     * @param \WP_Post $post A {@link WP_Post} instance.
+     *
+     * @return array An array of attachment ids.
+     * @since 3.10.0
+     *
+     */
+    public function get_gallery( $post ) {
+
+        // @todo: the `gallery` shortcode has an `exclude` attribute which isn't
+        // checked at the moment.
+
+        // Prepare the return value.
+        $ids = array();
+
+        // As the above for images in galleries.
+        // Code inspired by http://wordpress.stackexchange.com/questions/80408/how-to-get-page-post-gallery-attachment-images-in-order-they-are-set-in-backend
+        $pattern = get_shortcode_regex();
+
+        if ( preg_match_all( '/' . $pattern . '/s', $post->post_content, $matches )
+             && array_key_exists( 2, $matches )
+             && in_array( 'gallery', $matches[2] )
+        ) {
+
+            $keys = array_keys( $matches[2], 'gallery' );
+
+            foreach ( $keys as $key ) {
+                $atts = shortcode_parse_atts( $matches[3][ $key ] );
+
+                if ( is_array( $atts ) && array_key_exists( 'ids', $atts ) ) {
+                    // gallery images insert explicitly by their ids.
+
+                    foreach ( explode( ',', $atts['ids'] ) as $attachment_id ) {
+                        // Since we do not check for actual image existence
+                        // when generating the json content, check it now.
+                        if ( wp_get_attachment_image_src( $attachment_id, 'full' ) ) {
+                            $ids[ $attachment_id ] = true;
+                        }
+                    }
+                } else {
+                    // gallery shortcode with no ids uses all the images
+                    // attached to the post.
+                    $images = get_attached_media( 'image', $post->ID );
+                    foreach ( $images as $attachment ) {
+                        $ids[ $attachment->ID ] = true;
+                    }
+                }
+            }
+        }
+
+        return array_keys( $ids );
+    }
 
 }

Spacing +27 added lines, -27 removed lines

@@ -59,25 +59,25 @@ 
 	 * @since 3.10.0
 	 *
 	 */
-	public function get_attachment_id( $url ) {
+	public function get_attachment_id($url) {
 
 		// Get the upload directory data, we need the base URL to check whether
 		// the URL we received is within WP.
 		$dir = wp_upload_dir();
 
 		// Get the filename, the extension is kept.
-		if ( 1 !== preg_match( "@^{$dir['baseurl']}/(.+?)(?:-\d+x\d+)?(\.\w+)$@", $url, $matches ) ) {
+		if (1 !== preg_match("@^{$dir['baseurl']}/(.+?)(?:-\d+x\d+)?(\.\w+)$@", $url, $matches)) {
 			return false;
 		}
 
-		$filename = $matches[1] . $matches[2];
+		$filename = $matches[1].$matches[2];
 
 		// The following query is CPU-intensive, we need to review it.
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/689.
 		//
 		// Query for attachments with the specified filename.
-		$query = new WP_Query( array(
+		$query = new WP_Query(array(
 			'post_type'           => 'attachment',
 			'post_status'         => 'inherit',
 			'fields'              => 'ids',
@@ -90,10 +90,10 @@ 
 			),
 			'posts_per_page'      => 1,
 			'ignore_sticky_posts' => true,
-		) );
+		));
 
 		// If there are no posts, return.
-		if ( $query->have_posts() ) {
+		if ($query->have_posts()) {
 			return $query->posts[0];
 //			foreach ( $query->posts as $attachment_id ) {
 //
@@ -124,23 +124,23 @@ 
 	 * @since 3.10.0
 	 *
 	 */
-	public function get_image_embeds( $content ) {
+	public function get_image_embeds($content) {
 
 		// Go over all the images included in the post content, check if they are
 		// in the DB, and if so include them.
 		$images = array();
-		if ( false === preg_match_all( '#<img [^>]*src="([^\\">]*)"[^>]*>#', $content, $images ) ) {
+		if (false === preg_match_all('#<img [^>]*src="([^\\">]*)"[^>]*>#', $content, $images)) {
 			return array();
 		}
 
 		// Map the image URLs to attachment ids.
 		$that = $this;
-		$ids  = array_map( function ( $url ) use ( $that ) {
-			return $that->get_attachment_id( $url );
-		}, $images[1] );
+		$ids  = array_map(function($url) use ($that) {
+			return $that->get_attachment_id($url);
+		}, $images[1]);
 
 		// Filter out not found ids (i.e. id is false).
-		return array_filter( $ids, function ( $item ) {
+		return array_filter($ids, function($item) {
 			return false !== $item;
 		} );
 	}
@@ -154,7 +154,7 @@ 
 	 * @since 3.10.0
 	 *
 	 */
-	public function get_gallery( $post ) {
+	public function get_gallery($post) {
 
 		// @todo: the `gallery` shortcode has an `exclude` attribute which isn't
 		// checked at the moment.
@@ -166,38 +166,38 @@ 
 		// Code inspired by http://wordpress.stackexchange.com/questions/80408/how-to-get-page-post-gallery-attachment-images-in-order-they-are-set-in-backend
 		$pattern = get_shortcode_regex();
 
-		if ( preg_match_all( '/' . $pattern . '/s', $post->post_content, $matches )
-		     && array_key_exists( 2, $matches )
-		     && in_array( 'gallery', $matches[2] )
+		if (preg_match_all('/'.$pattern.'/s', $post->post_content, $matches)
+		     && array_key_exists(2, $matches)
+		     && in_array('gallery', $matches[2])
 		) {
 
-			$keys = array_keys( $matches[2], 'gallery' );
+			$keys = array_keys($matches[2], 'gallery');
 
-			foreach ( $keys as $key ) {
-				$atts = shortcode_parse_atts( $matches[3][ $key ] );
+			foreach ($keys as $key) {
+				$atts = shortcode_parse_atts($matches[3][$key]);
 
-				if ( is_array( $atts ) && array_key_exists( 'ids', $atts ) ) {
+				if (is_array($atts) && array_key_exists('ids', $atts)) {
 					// gallery images insert explicitly by their ids.
 
-					foreach ( explode( ',', $atts['ids'] ) as $attachment_id ) {
+					foreach (explode(',', $atts['ids']) as $attachment_id) {
 						// Since we do not check for actual image existence
 						// when generating the json content, check it now.
-						if ( wp_get_attachment_image_src( $attachment_id, 'full' ) ) {
-							$ids[ $attachment_id ] = true;
+						if (wp_get_attachment_image_src($attachment_id, 'full')) {
+							$ids[$attachment_id] = true;
 						}
 					}
 				} else {
 					// gallery shortcode with no ids uses all the images
 					// attached to the post.
-					$images = get_attached_media( 'image', $post->ID );
-					foreach ( $images as $attachment ) {
-						$ids[ $attachment->ID ] = true;
+					$images = get_attached_media('image', $post->ID);
+					foreach ($images as $attachment) {
+						$ids[$attachment->ID] = true;
 					}
 				}
 			}
 		}
 
-		return array_keys( $ids );
+		return array_keys($ids);
 	}
 
 }

src/includes/class-wordlift-abstract-post-to-jsonld-converter.php

Indentation +320 added lines, -320 removed lines

@@ -25,345 +25,345 @@
  */
 abstract class Wordlift_Abstract_Post_To_Jsonld_Converter implements Wordlift_Post_Converter {
 
-	/**
-	 * The JSON-LD context.
-	 *
-	 * @since 3.10.0
-	 */
-	const CONTEXT = 'http://schema.org';
-
-	/**
-	 * A {@link Wordlift_Entity_Type_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access protected
-	 * @var \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
-	 */
-	protected $entity_type_service;
-
-	/**
-	 * A {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access protected
-	 * @var \Wordlift_Entity_Service $entity_type_service A {@link Wordlift_Entity_Service} instance.
-	 */
-	protected $entity_service;
-
-	/**
-	 * A {@link Wordlift_User_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var \Wordlift_User_Service $user_service A {@link Wordlift_User_Service} instance.
-	 */
-	protected $user_service;
-
-	/**
-	 * A {@link Wordlift_Attachment_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var \Wordlift_Attachment_Service $attachment_service A {@link Wordlift_Attachment_Service} instance.
-	 */
-	protected $attachment_service;
-
-	/**
-	 * Wordlift_Post_To_Jsonld_Converter constructor.
-	 *
-	 * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
-	 * @param \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_User_Service $user_service A {@link Wordlift_User_Service} instance.
-	 * @param \Wordlift_Attachment_Service $attachment_service A {@link Wordlift_Attachment_Service} instance.
-	 *
-	 * @since 3.10.0
-	 *
-	 */
-	public function __construct( $entity_type_service, $entity_service, $user_service, $attachment_service ) {
-
-		$this->entity_type_service = $entity_type_service;
-		$this->entity_service      = $entity_service;
-		$this->user_service        = $user_service;
-		$this->attachment_service  = $attachment_service;
-	}
-
-	/**
-	 * Convert the provided {@link WP_Post} to a JSON-LD array. Any entity reference
-	 * found while processing the post is set in the $references array.
-	 *
-	 * @param int $post_id The post id.
-	 * @param array $references An array of entity references.
-	 * @param array $references_infos
-	 *
-	 * @return array A JSON-LD array.
-	 * @since 3.10.0
-	 */
-	public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
-
-		// Get the post instance.
-		$post = get_post( $post_id );
-		if ( null === $post ) {
-			// Post not found.
-			return null;
-		}
-
-		// Get the post URI @id.
-		$id = $this->entity_service->get_uri( $post->ID );
-		if ( is_null( $id ) ) {
-			$id = 'get_uri returned null, dataset is ' . wl_configuration_get_redlink_dataset_uri();
-		}
-
-		/*
+    /**
+     * The JSON-LD context.
+     *
+     * @since 3.10.0
+     */
+    const CONTEXT = 'http://schema.org';
+
+    /**
+     * A {@link Wordlift_Entity_Type_Service} instance.
+     *
+     * @since  3.10.0
+     * @access protected
+     * @var \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
+     */
+    protected $entity_type_service;
+
+    /**
+     * A {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.10.0
+     * @access protected
+     * @var \Wordlift_Entity_Service $entity_type_service A {@link Wordlift_Entity_Service} instance.
+     */
+    protected $entity_service;
+
+    /**
+     * A {@link Wordlift_User_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var \Wordlift_User_Service $user_service A {@link Wordlift_User_Service} instance.
+     */
+    protected $user_service;
+
+    /**
+     * A {@link Wordlift_Attachment_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var \Wordlift_Attachment_Service $attachment_service A {@link Wordlift_Attachment_Service} instance.
+     */
+    protected $attachment_service;
+
+    /**
+     * Wordlift_Post_To_Jsonld_Converter constructor.
+     *
+     * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_Service} instance.
+     * @param \Wordlift_Entity_Service $entity_service A {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_User_Service $user_service A {@link Wordlift_User_Service} instance.
+     * @param \Wordlift_Attachment_Service $attachment_service A {@link Wordlift_Attachment_Service} instance.
+     *
+     * @since 3.10.0
+     *
+     */
+    public function __construct( $entity_type_service, $entity_service, $user_service, $attachment_service ) {
+
+        $this->entity_type_service = $entity_type_service;
+        $this->entity_service      = $entity_service;
+        $this->user_service        = $user_service;
+        $this->attachment_service  = $attachment_service;
+    }
+
+    /**
+     * Convert the provided {@link WP_Post} to a JSON-LD array. Any entity reference
+     * found while processing the post is set in the $references array.
+     *
+     * @param int $post_id The post id.
+     * @param array $references An array of entity references.
+     * @param array $references_infos
+     *
+     * @return array A JSON-LD array.
+     * @since 3.10.0
+     */
+    public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
+
+        // Get the post instance.
+        $post = get_post( $post_id );
+        if ( null === $post ) {
+            // Post not found.
+            return null;
+        }
+
+        // Get the post URI @id.
+        $id = $this->entity_service->get_uri( $post->ID );
+        if ( is_null( $id ) ) {
+            $id = 'get_uri returned null, dataset is ' . wl_configuration_get_redlink_dataset_uri();
+        }
+
+        /*
 		 * The `types` variable holds one or more entity types. The `type` variable isn't used anymore.
 		 *
 		 * @since 3.20.0 We support more than one entity type.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/835
 		 */
-		//		// Get the entity @type. We consider `post` BlogPostings.
-		//		$type = $this->entity_type_service->get( $post_id );
-		$types = $this->entity_type_service->get_names( $post_id );
-		$type  = self::make_one( $types );
-
-		// Prepare the response.
-		$jsonld = array(
-			'@context'    => self::CONTEXT,
-			'@id'         => $id,
-			'@type'       => $type,
-			'description' => Wordlift_Post_Excerpt_Helper::get_text_excerpt( $post ),
-		);
-
-		// Set the `mainEntityOfPage` property if the post has some contents.
-		/*
+        //		// Get the entity @type. We consider `post` BlogPostings.
+        //		$type = $this->entity_type_service->get( $post_id );
+        $types = $this->entity_type_service->get_names( $post_id );
+        $type  = self::make_one( $types );
+
+        // Prepare the response.
+        $jsonld = array(
+            '@context'    => self::CONTEXT,
+            '@id'         => $id,
+            '@type'       => $type,
+            'description' => Wordlift_Post_Excerpt_Helper::get_text_excerpt( $post ),
+        );
+
+        // Set the `mainEntityOfPage` property if the post has some contents.
+        /*
 		 * Apply the `wl_post_content` filter, in case 3rd parties want to change the post content, e.g.
 		 * because the content is written elsewhere.
 		 *
 		 * @since 3.20.0
 		 */
-		$post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
-		if ( ! empty( $post_content ) ) {
-			// We're setting the `mainEntityOfPage` to signal which one is the
-			// main entity for the specified URL. It might be however that the
-			// post/page is actually about another specific entity. How WL deals
-			// with that hasn't been defined yet (see https://github.com/insideout10/wordlift-plugin/issues/451).
-			//
-			// See http://schema.org/mainEntityOfPage
-			//
-			// No need to specify `'@type' => 'WebPage'.
-			//
-			// See https://github.com/insideout10/wordlift-plugin/issues/451.
-			$jsonld['mainEntityOfPage'] = get_the_permalink( $post->ID );
-
-			/**
-			 * @see https://github.com/insideout10/wordlift-plugin/issues/1207
-			 *
-			 * @since 3.27.7
-			 */
-			if ( in_array( $type, array( 'Occupation', 'OccupationAggregationByEmployer' ) ) ) {
-				$jsonld['mainEntityOfPage'] = array(
-					'@type'        => 'WebPage',
-					'lastReviewed' => get_post_time( 'Y-m-d\TH:i:sP', true, $post, false )
-				);
-			}
-		};
-
-		$this->set_images( $this->attachment_service, $post, $jsonld );
-
-		// Get the entities referenced by this post and set it to the `references`
-		// array so that the caller can do further processing, such as printing out
-		// more of those references.
-		$references_without_locations = Object_Relation_Service::get_instance()
-		                                                       ->get_references( $post_id, Object_Type_Enum::POST );
-
-		/*
+        $post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
+        if ( ! empty( $post_content ) ) {
+            // We're setting the `mainEntityOfPage` to signal which one is the
+            // main entity for the specified URL. It might be however that the
+            // post/page is actually about another specific entity. How WL deals
+            // with that hasn't been defined yet (see https://github.com/insideout10/wordlift-plugin/issues/451).
+            //
+            // See http://schema.org/mainEntityOfPage
+            //
+            // No need to specify `'@type' => 'WebPage'.
+            //
+            // See https://github.com/insideout10/wordlift-plugin/issues/451.
+            $jsonld['mainEntityOfPage'] = get_the_permalink( $post->ID );
+
+            /**
+             * @see https://github.com/insideout10/wordlift-plugin/issues/1207
+             *
+             * @since 3.27.7
+             */
+            if ( in_array( $type, array( 'Occupation', 'OccupationAggregationByEmployer' ) ) ) {
+                $jsonld['mainEntityOfPage'] = array(
+                    '@type'        => 'WebPage',
+                    'lastReviewed' => get_post_time( 'Y-m-d\TH:i:sP', true, $post, false )
+                );
+            }
+        };
+
+        $this->set_images( $this->attachment_service, $post, $jsonld );
+
+        // Get the entities referenced by this post and set it to the `references`
+        // array so that the caller can do further processing, such as printing out
+        // more of those references.
+        $references_without_locations = Object_Relation_Service::get_instance()
+                                                                ->get_references( $post_id, Object_Type_Enum::POST );
+
+        /*
 		 * Add the locations to the references.
 		 *
 		 * @since 3.19.5
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/858.
 		 */
-		// A reference to use in closure.
-		$entity_type_service = $this->entity_type_service;
-		$locations           = array_reduce( $references_without_locations, function ( $carry, $reference ) use ( $entity_type_service ) {
-			/**
-			 * @var $reference Reference
-			 */
-			// @see https://schema.org/location for the schema.org types using the `location` property.
-			if ( ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Action' )
-			     && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Event' )
-			     && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Organization' ) ) {
-
-				return $carry;
-			}
-			$post_location_ids        = get_post_meta( $reference->get_id(), Wordlift_Schema_Service::FIELD_LOCATION );
-			$post_location_references = array_map( function ( $post_id ) {
-				return new Post_Reference( $post_id );
-			}, $post_location_ids );
-
-			return array_merge( $carry,  $post_location_references);
-		}, array() );
-
-		// Merge the references with the referenced locations if any.
-		$references = array_merge( $references_without_locations, $locations );
-
-		return $jsonld;
-	}
-
-	/**
-	 * If the provided value starts with the schema.org context, we remove the schema.org
-	 * part since it is set with the '@context'.
-	 *
-	 * @param string $value The property value.
-	 *
-	 * @return string The property value without the context.
-	 * @since 3.10.0
-	 *
-	 */
-	public function relative_to_context( $value ) {
-
-		return 0 === strpos( $value, self::CONTEXT . '/' ) ? substr( $value, strlen( self::CONTEXT ) + 1 ) : $value;
-	}
-
-	/**
-	 * Set the images, by looking for embedded images, for images loaded via the
-	 * gallery and for the featured image.
-	 *
-	 * Uses the cache service to store the results of this function for a day.
-	 *
-	 * @param $attachment_service Wordlift_Attachment_Service
-	 * @param WP_Post $post The target {@link WP_Post}.
-	 * @param array $jsonld The JSON-LD array.
-	 *
-	 * @since 3.10.0
-	 */
-	public static function set_images( $attachment_service, $post, &$jsonld ) {
-
-		// Prepare the attachment ids array.
-		$ids = array();
-
-		// Set the thumbnail id as first attachment id, if any.
-		$thumbnail_id = get_post_thumbnail_id( $post->ID );
-		if ( '' !== $thumbnail_id && 0 !== $thumbnail_id ) {
-			$ids[] = $thumbnail_id;
-		}
-
-		// For the time being the following is being removed since the query
-		// initiated by `get_image_embeds` is consuming lots of CPU.
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/689.
-		//
-		// Get the embeds, removing existing ids.
-		if ( apply_filters( 'wl_feature__enable__image-embeds', true ) ) {
-			$embeds = array_diff( $attachment_service->get_image_embeds( $post->post_content ), $ids );
-		} else {
-			$embeds = array();
-		}
-
-		// Get the gallery, removing existing ids.
-		$gallery = array_diff( $attachment_service->get_gallery( $post ), $ids, $embeds );
-
-		// Map the attachment ids to images' data structured for schema.org use.
-		$images_with_sizes = array_filter(
-			array_reduce( array_merge( $ids, $embeds, $gallery ),
-				function ( $carry, $item ) {
-					/*
+        // A reference to use in closure.
+        $entity_type_service = $this->entity_type_service;
+        $locations           = array_reduce( $references_without_locations, function ( $carry, $reference ) use ( $entity_type_service ) {
+            /**
+             * @var $reference Reference
+             */
+            // @see https://schema.org/location for the schema.org types using the `location` property.
+            if ( ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Action' )
+                 && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Event' )
+                 && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Organization' ) ) {
+
+                return $carry;
+            }
+            $post_location_ids        = get_post_meta( $reference->get_id(), Wordlift_Schema_Service::FIELD_LOCATION );
+            $post_location_references = array_map( function ( $post_id ) {
+                return new Post_Reference( $post_id );
+            }, $post_location_ids );
+
+            return array_merge( $carry,  $post_location_references);
+        }, array() );
+
+        // Merge the references with the referenced locations if any.
+        $references = array_merge( $references_without_locations, $locations );
+
+        return $jsonld;
+    }
+
+    /**
+     * If the provided value starts with the schema.org context, we remove the schema.org
+     * part since it is set with the '@context'.
+     *
+     * @param string $value The property value.
+     *
+     * @return string The property value without the context.
+     * @since 3.10.0
+     *
+     */
+    public function relative_to_context( $value ) {
+
+        return 0 === strpos( $value, self::CONTEXT . '/' ) ? substr( $value, strlen( self::CONTEXT ) + 1 ) : $value;
+    }
+
+    /**
+     * Set the images, by looking for embedded images, for images loaded via the
+     * gallery and for the featured image.
+     *
+     * Uses the cache service to store the results of this function for a day.
+     *
+     * @param $attachment_service Wordlift_Attachment_Service
+     * @param WP_Post $post The target {@link WP_Post}.
+     * @param array $jsonld The JSON-LD array.
+     *
+     * @since 3.10.0
+     */
+    public static function set_images( $attachment_service, $post, &$jsonld ) {
+
+        // Prepare the attachment ids array.
+        $ids = array();
+
+        // Set the thumbnail id as first attachment id, if any.
+        $thumbnail_id = get_post_thumbnail_id( $post->ID );
+        if ( '' !== $thumbnail_id && 0 !== $thumbnail_id ) {
+            $ids[] = $thumbnail_id;
+        }
+
+        // For the time being the following is being removed since the query
+        // initiated by `get_image_embeds` is consuming lots of CPU.
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/689.
+        //
+        // Get the embeds, removing existing ids.
+        if ( apply_filters( 'wl_feature__enable__image-embeds', true ) ) {
+            $embeds = array_diff( $attachment_service->get_image_embeds( $post->post_content ), $ids );
+        } else {
+            $embeds = array();
+        }
+
+        // Get the gallery, removing existing ids.
+        $gallery = array_diff( $attachment_service->get_gallery( $post ), $ids, $embeds );
+
+        // Map the attachment ids to images' data structured for schema.org use.
+        $images_with_sizes = array_filter(
+            array_reduce( array_merge( $ids, $embeds, $gallery ),
+                function ( $carry, $item ) {
+                    /*
 					* @todo: we're not sure that we're getting attachment data here, we
 					* should filter `false`s.
 					*/
 
-					$sources = array_merge(
-						Wordlift_Image_Service::get_sources( $item ),
-						array( wp_get_attachment_image_src( $item, 'full' ) )
-					);
-
-					$sources_with_image = array_filter( $sources, function ( $source ) {
-						return ! empty( $source[0] );
-					} );
-
-					// Get the attachment data.
-					// $attachment = wp_get_attachment_image_src( $item, 'full' );
-
-					// var_dump( array( "sources-$item" => Wordlift_Image_Service::get_sources( $item ) ) );
-
-					// Bail if image is not found.
-					// In some cases, you can delete the image from the database
-					// or from uploads dir, but the image id still exists as featured image
-					// or in [gallery] shortcode.
-					//					if ( empty( $attachment[0] ) ) {
-					if ( empty( $sources_with_image ) ) {
-						return $carry;
-					}
-
-					// Merge the arrays.
-					return array_merge(
-						$carry,
-						$sources_with_image
-					);
-				}
-				// Initial array.
-				, array() )
-		);
-
-		// Refactor data as per schema.org specifications.
-		$images = array_map( function ( $attachment ) {
-			return Wordlift_Abstract_Post_To_Jsonld_Converter::set_image_size(
-				array(
-					'@type' => 'ImageObject',
-					'url'   => $attachment[0],
-				), $attachment
-			);
-		}, $images_with_sizes );
-
-		// Add images if present.
-		if ( 0 < count( $images ) ) {
-			$jsonld['image'] = $images;
-		}
-
-	}
-
-	/**
-	 * If the provided array of values contains only one value, then one single
-	 * value is returned, otherwise the original array is returned.
-	 *
-	 * @param array $value An array of values.
-	 *
-	 * @return mixed|array A single value or the original array.
-	 * @since 3.20.0 The function has been moved from {@link Wordlift_Entity_Post_To_Jsonld_Converter} to
-	 *  {@link Wordlift_Abstract_Post_To_Jsonld_Converter}.
-	 * @since  3.8.0
-	 * @access private
-	 *
-	 */
-	protected static function make_one( $value ) {
-
-		return 1 === count( $value ) ? $value[0] : $value;
-	}
-
-	/**
-	 * Process the provided array by adding the width / height if the values
-	 * are available and are greater than 0.
-	 *
-	 * @param array $image The `ImageObject` array.
-	 * @param array $attachment The attachment array.
-	 *
-	 * @return array The enriched `ImageObject` array.
-	 * @since 3.14.0
-	 *
-	 */
-	public static function set_image_size( $image, $attachment ) {
-
-		// If you specify a "width" or "height" value you should leave out
-		// 'px'. For example: "width":"4608px" should be "width":"4608".
-		//
-		// See https://github.com/insideout10/wordlift-plugin/issues/451.
-		if ( isset( $attachment[1] ) && is_numeric( $attachment[1] ) && 0 < $attachment[1] ) {
-			$image['width'] = $attachment[1];
-		}
-
-		if ( isset( $attachment[2] ) && is_numeric( $attachment[2] ) && 0 < $attachment[2] ) {
-			$image['height'] = $attachment[2];
-		}
-
-		return $image;
-	}
+                    $sources = array_merge(
+                        Wordlift_Image_Service::get_sources( $item ),
+                        array( wp_get_attachment_image_src( $item, 'full' ) )
+                    );
+
+                    $sources_with_image = array_filter( $sources, function ( $source ) {
+                        return ! empty( $source[0] );
+                    } );
+
+                    // Get the attachment data.
+                    // $attachment = wp_get_attachment_image_src( $item, 'full' );
+
+                    // var_dump( array( "sources-$item" => Wordlift_Image_Service::get_sources( $item ) ) );
+
+                    // Bail if image is not found.
+                    // In some cases, you can delete the image from the database
+                    // or from uploads dir, but the image id still exists as featured image
+                    // or in [gallery] shortcode.
+                    //					if ( empty( $attachment[0] ) ) {
+                    if ( empty( $sources_with_image ) ) {
+                        return $carry;
+                    }
+
+                    // Merge the arrays.
+                    return array_merge(
+                        $carry,
+                        $sources_with_image
+                    );
+                }
+                // Initial array.
+                , array() )
+        );
+
+        // Refactor data as per schema.org specifications.
+        $images = array_map( function ( $attachment ) {
+            return Wordlift_Abstract_Post_To_Jsonld_Converter::set_image_size(
+                array(
+                    '@type' => 'ImageObject',
+                    'url'   => $attachment[0],
+                ), $attachment
+            );
+        }, $images_with_sizes );
+
+        // Add images if present.
+        if ( 0 < count( $images ) ) {
+            $jsonld['image'] = $images;
+        }
+
+    }
+
+    /**
+     * If the provided array of values contains only one value, then one single
+     * value is returned, otherwise the original array is returned.
+     *
+     * @param array $value An array of values.
+     *
+     * @return mixed|array A single value or the original array.
+     * @since 3.20.0 The function has been moved from {@link Wordlift_Entity_Post_To_Jsonld_Converter} to
+     *  {@link Wordlift_Abstract_Post_To_Jsonld_Converter}.
+     * @since  3.8.0
+     * @access private
+     *
+     */
+    protected static function make_one( $value ) {
+
+        return 1 === count( $value ) ? $value[0] : $value;
+    }
+
+    /**
+     * Process the provided array by adding the width / height if the values
+     * are available and are greater than 0.
+     *
+     * @param array $image The `ImageObject` array.
+     * @param array $attachment The attachment array.
+     *
+     * @return array The enriched `ImageObject` array.
+     * @since 3.14.0
+     *
+     */
+    public static function set_image_size( $image, $attachment ) {
+
+        // If you specify a "width" or "height" value you should leave out
+        // 'px'. For example: "width":"4608px" should be "width":"4608".
+        //
+        // See https://github.com/insideout10/wordlift-plugin/issues/451.
+        if ( isset( $attachment[1] ) && is_numeric( $attachment[1] ) && 0 < $attachment[1] ) {
+            $image['width'] = $attachment[1];
+        }
+
+        if ( isset( $attachment[2] ) && is_numeric( $attachment[2] ) && 0 < $attachment[2] ) {
+            $image['height'] = $attachment[2];
+        }
+
+        return $image;
+    }
 }

Spacing +52 added lines, -52 removed lines

@@ -79,7 +79,7 @@ 
 	 * @since 3.10.0
 	 *
 	 */
-	public function __construct( $entity_type_service, $entity_service, $user_service, $attachment_service ) {
+	public function __construct($entity_type_service, $entity_service, $user_service, $attachment_service) {
 
 		$this->entity_type_service = $entity_type_service;
 		$this->entity_service      = $entity_service;
@@ -98,19 +98,19 @@ 
 	 * @return array A JSON-LD array.
 	 * @since 3.10.0
 	 */
-	public function convert( $post_id, &$references = array(), &$references_infos = array() ) {
+	public function convert($post_id, &$references = array(), &$references_infos = array()) {
 
 		// Get the post instance.
-		$post = get_post( $post_id );
-		if ( null === $post ) {
+		$post = get_post($post_id);
+		if (null === $post) {
 			// Post not found.
 			return null;
 		}
 
 		// Get the post URI @id.
-		$id = $this->entity_service->get_uri( $post->ID );
-		if ( is_null( $id ) ) {
-			$id = 'get_uri returned null, dataset is ' . wl_configuration_get_redlink_dataset_uri();
+		$id = $this->entity_service->get_uri($post->ID);
+		if (is_null($id)) {
+			$id = 'get_uri returned null, dataset is '.wl_configuration_get_redlink_dataset_uri();
 		}
 
 		/*
@@ -122,15 +122,15 @@ 
 		 */
 		//		// Get the entity @type. We consider `post` BlogPostings.
 		//		$type = $this->entity_type_service->get( $post_id );
-		$types = $this->entity_type_service->get_names( $post_id );
-		$type  = self::make_one( $types );
+		$types = $this->entity_type_service->get_names($post_id);
+		$type  = self::make_one($types);
 
 		// Prepare the response.
 		$jsonld = array(
 			'@context'    => self::CONTEXT,
 			'@id'         => $id,
 			'@type'       => $type,
-			'description' => Wordlift_Post_Excerpt_Helper::get_text_excerpt( $post ),
+			'description' => Wordlift_Post_Excerpt_Helper::get_text_excerpt($post),
 		);
 
 		// Set the `mainEntityOfPage` property if the post has some contents.
@@ -140,8 +140,8 @@ 
 		 *
 		 * @since 3.20.0
 		 */
-		$post_content = apply_filters( 'wl_post_content', $post->post_content, $post );
-		if ( ! empty( $post_content ) ) {
+		$post_content = apply_filters('wl_post_content', $post->post_content, $post);
+		if ( ! empty($post_content)) {
 			// We're setting the `mainEntityOfPage` to signal which one is the
 			// main entity for the specified URL. It might be however that the
 			// post/page is actually about another specific entity. How WL deals
@@ -152,28 +152,28 @@ 
 			// No need to specify `'@type' => 'WebPage'.
 			//
 			// See https://github.com/insideout10/wordlift-plugin/issues/451.
-			$jsonld['mainEntityOfPage'] = get_the_permalink( $post->ID );
+			$jsonld['mainEntityOfPage'] = get_the_permalink($post->ID);
 
 			/**
 			 * @see https://github.com/insideout10/wordlift-plugin/issues/1207
 			 *
 			 * @since 3.27.7
 			 */
-			if ( in_array( $type, array( 'Occupation', 'OccupationAggregationByEmployer' ) ) ) {
+			if (in_array($type, array('Occupation', 'OccupationAggregationByEmployer'))) {
 				$jsonld['mainEntityOfPage'] = array(
 					'@type'        => 'WebPage',
-					'lastReviewed' => get_post_time( 'Y-m-d\TH:i:sP', true, $post, false )
+					'lastReviewed' => get_post_time('Y-m-d\TH:i:sP', true, $post, false)
 				);
 			}
 		};
 
-		$this->set_images( $this->attachment_service, $post, $jsonld );
+		$this->set_images($this->attachment_service, $post, $jsonld);
 
 		// Get the entities referenced by this post and set it to the `references`
 		// array so that the caller can do further processing, such as printing out
 		// more of those references.
 		$references_without_locations = Object_Relation_Service::get_instance()
-		                                                       ->get_references( $post_id, Object_Type_Enum::POST );
+		                                                       ->get_references($post_id, Object_Type_Enum::POST);
 
 		/*
 		 * Add the locations to the references.
@@ -184,27 +184,27 @@ 
 		 */
 		// A reference to use in closure.
 		$entity_type_service = $this->entity_type_service;
-		$locations           = array_reduce( $references_without_locations, function ( $carry, $reference ) use ( $entity_type_service ) {
+		$locations           = array_reduce($references_without_locations, function($carry, $reference) use ($entity_type_service) {
 			/**
 			 * @var $reference Reference
 			 */
 			// @see https://schema.org/location for the schema.org types using the `location` property.
-			if ( ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Action' )
-			     && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Event' )
-			     && ! $entity_type_service->has_entity_type( $reference->get_id(), 'http://schema.org/Organization' ) ) {
+			if ( ! $entity_type_service->has_entity_type($reference->get_id(), 'http://schema.org/Action')
+			     && ! $entity_type_service->has_entity_type($reference->get_id(), 'http://schema.org/Event')
+			     && ! $entity_type_service->has_entity_type($reference->get_id(), 'http://schema.org/Organization')) {
 
 				return $carry;
 			}
-			$post_location_ids        = get_post_meta( $reference->get_id(), Wordlift_Schema_Service::FIELD_LOCATION );
-			$post_location_references = array_map( function ( $post_id ) {
-				return new Post_Reference( $post_id );
-			}, $post_location_ids );
+			$post_location_ids        = get_post_meta($reference->get_id(), Wordlift_Schema_Service::FIELD_LOCATION);
+			$post_location_references = array_map(function($post_id) {
+				return new Post_Reference($post_id);
+			}, $post_location_ids);
 
-			return array_merge( $carry,  $post_location_references);
-		}, array() );
+			return array_merge($carry, $post_location_references);
+		}, array());
 
 		// Merge the references with the referenced locations if any.
-		$references = array_merge( $references_without_locations, $locations );
+		$references = array_merge($references_without_locations, $locations);
 
 		return $jsonld;
 	}
@@ -219,9 +219,9 @@ 
 	 * @since 3.10.0
 	 *
 	 */
-	public function relative_to_context( $value ) {
+	public function relative_to_context($value) {
 
-		return 0 === strpos( $value, self::CONTEXT . '/' ) ? substr( $value, strlen( self::CONTEXT ) + 1 ) : $value;
+		return 0 === strpos($value, self::CONTEXT.'/') ? substr($value, strlen(self::CONTEXT) + 1) : $value;
 	}
 
 	/**
@@ -236,14 +236,14 @@ 
 	 *
 	 * @since 3.10.0
 	 */
-	public static function set_images( $attachment_service, $post, &$jsonld ) {
+	public static function set_images($attachment_service, $post, &$jsonld) {
 
 		// Prepare the attachment ids array.
 		$ids = array();
 
 		// Set the thumbnail id as first attachment id, if any.
-		$thumbnail_id = get_post_thumbnail_id( $post->ID );
-		if ( '' !== $thumbnail_id && 0 !== $thumbnail_id ) {
+		$thumbnail_id = get_post_thumbnail_id($post->ID);
+		if ('' !== $thumbnail_id && 0 !== $thumbnail_id) {
 			$ids[] = $thumbnail_id;
 		}
 
@@ -253,31 +253,31 @@ 
 		// See https://github.com/insideout10/wordlift-plugin/issues/689.
 		//
 		// Get the embeds, removing existing ids.
-		if ( apply_filters( 'wl_feature__enable__image-embeds', true ) ) {
-			$embeds = array_diff( $attachment_service->get_image_embeds( $post->post_content ), $ids );
+		if (apply_filters('wl_feature__enable__image-embeds', true)) {
+			$embeds = array_diff($attachment_service->get_image_embeds($post->post_content), $ids);
 		} else {
 			$embeds = array();
 		}
 
 		// Get the gallery, removing existing ids.
-		$gallery = array_diff( $attachment_service->get_gallery( $post ), $ids, $embeds );
+		$gallery = array_diff($attachment_service->get_gallery($post), $ids, $embeds);
 
 		// Map the attachment ids to images' data structured for schema.org use.
 		$images_with_sizes = array_filter(
-			array_reduce( array_merge( $ids, $embeds, $gallery ),
-				function ( $carry, $item ) {
+			array_reduce(array_merge($ids, $embeds, $gallery),
+				function($carry, $item) {
 					/*
 					* @todo: we're not sure that we're getting attachment data here, we
 					* should filter `false`s.
 					*/
 
 					$sources = array_merge(
-						Wordlift_Image_Service::get_sources( $item ),
-						array( wp_get_attachment_image_src( $item, 'full' ) )
+						Wordlift_Image_Service::get_sources($item),
+						array(wp_get_attachment_image_src($item, 'full'))
 					);
 
-					$sources_with_image = array_filter( $sources, function ( $source ) {
-						return ! empty( $source[0] );
+					$sources_with_image = array_filter($sources, function($source) {
+						return ! empty($source[0]);
 					} );
 
 					// Get the attachment data.
@@ -290,7 +290,7 @@ 
 					// or from uploads dir, but the image id still exists as featured image
 					// or in [gallery] shortcode.
 					//					if ( empty( $attachment[0] ) ) {
-					if ( empty( $sources_with_image ) ) {
+					if (empty($sources_with_image)) {
 						return $carry;
 					}
 
@@ -301,21 +301,21 @@ 
 					);
 				}
 				// Initial array.
-				, array() )
+				, array())
 		);
 
 		// Refactor data as per schema.org specifications.
-		$images = array_map( function ( $attachment ) {
+		$images = array_map(function($attachment) {
 			return Wordlift_Abstract_Post_To_Jsonld_Converter::set_image_size(
 				array(
 					'@type' => 'ImageObject',
 					'url'   => $attachment[0],
 				), $attachment
 			);
-		}, $images_with_sizes );
+		}, $images_with_sizes);
 
 		// Add images if present.
-		if ( 0 < count( $images ) ) {
+		if (0 < count($images)) {
 			$jsonld['image'] = $images;
 		}
 
@@ -334,9 +334,9 @@ 
 	 * @access private
 	 *
 	 */
-	protected static function make_one( $value ) {
+	protected static function make_one($value) {
 
-		return 1 === count( $value ) ? $value[0] : $value;
+		return 1 === count($value) ? $value[0] : $value;
 	}
 
 	/**
@@ -350,17 +350,17 @@ 
 	 * @since 3.14.0
 	 *
 	 */
-	public static function set_image_size( $image, $attachment ) {
+	public static function set_image_size($image, $attachment) {
 
 		// If you specify a "width" or "height" value you should leave out
 		// 'px'. For example: "width":"4608px" should be "width":"4608".
 		//
 		// See https://github.com/insideout10/wordlift-plugin/issues/451.
-		if ( isset( $attachment[1] ) && is_numeric( $attachment[1] ) && 0 < $attachment[1] ) {
+		if (isset($attachment[1]) && is_numeric($attachment[1]) && 0 < $attachment[1]) {
 			$image['width'] = $attachment[1];
 		}
 
-		if ( isset( $attachment[2] ) && is_numeric( $attachment[2] ) && 0 < $attachment[2] ) {
+		if (isset($attachment[2]) && is_numeric($attachment[2]) && 0 < $attachment[2]) {
 			$image['height'] = $attachment[2];
 		}