insideout10 / wordlift-plugin

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

Indentation +498 added lines, -498 removed lines

@@ -19,522 +19,522 @@
  */
 class Wordlift_Post_To_Jsonld_Converter extends Wordlift_Abstract_Post_To_Jsonld_Converter {
 
-	/**
-	 * @var Wordlift_Post_To_Jsonld_Converter
-	 */
-	private static $instance;
-
-	/**
-	 * A {@link Wordlift_Log_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
-	 */
-	private $log;
-
-	/**
-	 * @var false
-	 */
-	private $disable_convert_filters;
-	/**
-	 * @var Object_Relation_Service
-	 */
-	private $object_relation_service;
-
-	/**
-	 * Wordlift_Post_To_Jsonld_Converter constructor.
-	 *
-	 * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_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, $user_service, $attachment_service, $disable_convert_filters = false ) {
-		parent::__construct( $entity_type_service, $user_service, $attachment_service, Wordlift_Property_Getter_Factory::create() );
-		$this->disable_convert_filters = $disable_convert_filters;
-		$this->object_relation_service = Object_Relation_Service::get_instance();
-		// Set a reference to the logger.
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Post_To_Jsonld_Converter' );
-
-		self::$instance = $this;
-
-	}
-
-	public static function get_instance() {
-
-		return self::$instance;
-	}
-
-	public function new_instance_with_filters_disabled() {
-		return new static( $this->entity_type_service, $this->user_service, $this->attachment_service, true );
-	}
-
-	/**
-	 * 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<Reference> $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 base JSON-LD and the list of entities referenced by this entity.
-		$jsonld = parent::convert( $post_id, $references, $references_infos );
-
-		// Set WebPage by default.
-		if ( empty( $jsonld['@type'] ) ) {
-			$jsonld['@type'] = 'WebPage';
-		}
-
-		// Get the entity name.
-		$jsonld['headline'] = $post->post_title;
-
-		$custom_fields = $this->entity_type_service->get_custom_fields_for_post( $post_id );
-
-		if ( isset( $custom_fields ) ) {
-			$this->process_type_custom_fields( $jsonld, $custom_fields, $post, $references, $references_infos );
-		}
-
-		// Set the published and modified dates.
-		/*
+    /**
+     * @var Wordlift_Post_To_Jsonld_Converter
+     */
+    private static $instance;
+
+    /**
+     * A {@link Wordlift_Log_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var Wordlift_Log_Service $log A {@link Wordlift_Log_Service} instance.
+     */
+    private $log;
+
+    /**
+     * @var false
+     */
+    private $disable_convert_filters;
+    /**
+     * @var Object_Relation_Service
+     */
+    private $object_relation_service;
+
+    /**
+     * Wordlift_Post_To_Jsonld_Converter constructor.
+     *
+     * @param \Wordlift_Entity_Type_Service $entity_type_service A {@link Wordlift_Entity_Type_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, $user_service, $attachment_service, $disable_convert_filters = false ) {
+        parent::__construct( $entity_type_service, $user_service, $attachment_service, Wordlift_Property_Getter_Factory::create() );
+        $this->disable_convert_filters = $disable_convert_filters;
+        $this->object_relation_service = Object_Relation_Service::get_instance();
+        // Set a reference to the logger.
+        $this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Post_To_Jsonld_Converter' );
+
+        self::$instance = $this;
+
+    }
+
+    public static function get_instance() {
+
+        return self::$instance;
+    }
+
+    public function new_instance_with_filters_disabled() {
+        return new static( $this->entity_type_service, $this->user_service, $this->attachment_service, true );
+    }
+
+    /**
+     * 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<Reference> $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 base JSON-LD and the list of entities referenced by this entity.
+        $jsonld = parent::convert( $post_id, $references, $references_infos );
+
+        // Set WebPage by default.
+        if ( empty( $jsonld['@type'] ) ) {
+            $jsonld['@type'] = 'WebPage';
+        }
+
+        // Get the entity name.
+        $jsonld['headline'] = $post->post_title;
+
+        $custom_fields = $this->entity_type_service->get_custom_fields_for_post( $post_id );
+
+        if ( isset( $custom_fields ) ) {
+            $this->process_type_custom_fields( $jsonld, $custom_fields, $post, $references, $references_infos );
+        }
+
+        // Set the published and modified dates.
+        /*
 		 * Set the `datePublished` and `dateModified` using the local timezone.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/887
 		 *
 		 * @since 3.20.0
 		 */
-		try {
-			$default_timezone = date_default_timezone_get();
-			$timezone         = get_option( 'timezone_string' );
-			if ( ! empty( $timezone ) ) {
-				date_default_timezone_set( $timezone ); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.timezone_change_date_default_timezone_set
-				$jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i:sP', false, $post );
-				$jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i:sP', false, $post );
-				date_default_timezone_set( $default_timezone ); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.timezone_change_date_default_timezone_set
-			} else {
-				$jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i', true, $post, false );
-				$jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i', true, $post, false );
-			}
-		} catch ( Exception $e ) {
-			$jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i', true, $post, false );
-			$jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i', true, $post, false );
-		}
-
-		// Get the word count for the post.
-		/*
+        try {
+            $default_timezone = date_default_timezone_get();
+            $timezone         = get_option( 'timezone_string' );
+            if ( ! empty( $timezone ) ) {
+                date_default_timezone_set( $timezone ); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.timezone_change_date_default_timezone_set
+                $jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i:sP', false, $post );
+                $jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i:sP', false, $post );
+                date_default_timezone_set( $default_timezone ); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.timezone_change_date_default_timezone_set
+            } else {
+                $jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i', true, $post, false );
+                $jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i', true, $post, false );
+            }
+        } catch ( Exception $e ) {
+            $jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i', true, $post, false );
+            $jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i', true, $post, false );
+        }
+
+        // Get the word count for the post.
+        /*
 		 * Do not display the `wordCount` on a `WebPage`.
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/888
 		 *
 		 * @since 3.20.0
 		 */
-		if ( ! empty( $jsonld['@type'] ) && 'WebPage' !== $jsonld['@type'] ) {
-			$post_adapter        = new Wordlift_Post_Adapter( $post_id );
-			$jsonld['wordCount'] = $post_adapter->word_count();
-		}
+        if ( ! empty( $jsonld['@type'] ) && 'WebPage' !== $jsonld['@type'] ) {
+            $post_adapter        = new Wordlift_Post_Adapter( $post_id );
+            $jsonld['wordCount'] = $post_adapter->word_count();
+        }
 
-		/*
+        /*
 		 * Add keywords, articleSection, commentCount and inLanguage properties to `Article` JSON-LD
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/1140
 		 *
 		 * @since 3.27.2
 		 */
-		if ( ! empty( $jsonld['@type'] ) && 'WebPage' !== $jsonld['@type'] ) {
-			$post_adapter    = new Wordlift_Post_Adapter( $post_id );
-			$keywords        = $post_adapter->keywords();
-			$article_section = $post_adapter->article_section();
-			$comment_count   = $post_adapter->comment_count();
-			$locale          = $post_adapter->locale();
-
-			if ( isset( $keywords ) ) {
-				$jsonld['keywords'] = $keywords;
-			}
-			if ( ! empty( $article_section ) ) {
-				$jsonld['articleSection'] = $article_section;
-			}
-			$jsonld['commentCount'] = $comment_count;
-			$jsonld['inLanguage']   = $locale;
-			$post_adapter->add_references( $post_id, $references );
-		}
-
-		// Set the publisher.
-		$this->set_publisher( $jsonld );
-
-		$references = $this->convert_references( $references );
-
-		// Process the references if any.
-		$this->set_mentions_and_about( $references, $post, $jsonld );
-
-		// Finally set the author.
-		$jsonld['author'] = $this->get_author( $post->post_author, $references );
-
-		// Return the JSON-LD if filters are disabled by the client.
-		if ( $this->disable_convert_filters ) {
-			return $jsonld;
-		}
-
-		/**
-		 * Call the `wl_post_jsonld_array` filter. This filter allows 3rd parties to also modify the references.
-		 *
-		 * @param array $value {
-		 *
-		 * @type array $jsonld The JSON-LD structure.
-		 * @type int[] $references An array of post IDs.
-		 * }
-		 * @since 3.25.0
-		 *
-		 * @see https://www.geeklab.info/2010/04/wordpress-pass-variables-by-reference-with-apply_filter/
-		 *
-		 * @api
-		 */
-		$ret_val = apply_filters(
-			'wl_post_jsonld_array',
-			array(
-				'jsonld'     => $jsonld,
-				'references' => $references,
-			),
-			$post_id
-		);
-
-		$jsonld     = $ret_val['jsonld'];
-		$references = $ret_val['references'];
-
-		/**
-		 * Call the `wl_post_jsonld` filter.
-		 *
-		 * @param array $jsonld The JSON-LD structure.
-		 * @param int $post_id The {@link WP_Post} `id`.
-		 * @param array $references The array of referenced entities.
-		 *
-		 * @since 3.14.0
-		 *
-		 * @api
-		 */
-		return apply_filters( 'wl_post_jsonld', $jsonld, $post_id, $references );
-	}
-
-	/**
-	 * Get the author's JSON-LD fragment.
-	 *
-	 * The JSON-LD fragment is generated using the {@link WP_User}'s data or
-	 * the referenced entity if configured for the {@link WP_User}.
-	 *
-	 * @param int   $author_id The author {@link WP_User}'s `id`.
-	 * @param array $references An array of referenced entities.
-	 *
-	 * @return string|array A JSON-LD structure.
-	 * @since 3.14.0
-	 */
-	public function get_author( $author_id, &$references ) {
-
-		// Get the entity bound to this user.
-		$entity_id = $this->user_service->get_entity( $author_id );
-
-		// If there's no entity bound return a simple author structure.
-		if ( empty( $entity_id ) || 'publish' !== get_post_status( $entity_id ) ) {
-
-			$author            = get_the_author_meta( 'display_name', $author_id );
-			$author_first_name = get_the_author_meta( 'first_name', $author_id );
-			$author_last_name  = get_the_author_meta( 'last_name', $author_id );
-			$author_uri        = $this->user_service->get_uri( $author_id );
-
-			return array(
-				'@type'      => 'Person',
-				'@id'        => $author_uri,
-				'name'       => $author,
-				'givenName'  => $author_first_name,
-				'familyName' => $author_last_name,
-				'url'        => get_author_posts_url( $author_id ),
-			);
-		}
-
-		// Add the author to the references.
-		$author_uri   = Wordlift_Entity_Service::get_instance()->get_uri( $entity_id );
-		$references[] = $entity_id;
-
-		// Return the JSON-LD for the referenced entity.
-		return array(
-			'@id' => $author_uri,
-		);
-	}
-
-	/**
-	 * Enrich the provided params array with publisher data, if available.
-	 *
-	 * @param array $params The parameters array.
-	 *
-	 * @since 3.10.0
-	 */
-	protected function set_publisher( &$params ) {
-
-		// If the publisher id isn't set don't do anything.
-		$publisher_id = Wordlift_Configuration_Service::get_instance()->get_publisher_id();
-		if ( null === $publisher_id ) {
-			return;
-		}
-
-		// Get the post instance.
-		$post = get_post( $publisher_id );
-		if ( null === $post ) {
-			// Publisher not found.
-			return;
-		}
-
-		// Get the item id.
-		$id = Wordlift_Entity_Service::get_instance()->get_uri( $publisher_id );
-
-		// Get the type.
-		$type = $this->entity_type_service->get( $publisher_id );
-
-		// Get the name.
-		$name = $post->post_title;
-
-		// Set the publisher data.
-		$params['publisher'] = array(
-			'@type' => $this->relative_to_context( $type['uri'] ),
-			'@id'   => $id,
-			'name'  => $name,
-		);
-
-		// Add the sameAs values associated with the publisher.
-		$storage_factory = Wordlift_Storage_Factory::get_instance();
-		$sameas          = $storage_factory->post_meta( Wordlift_Schema_Service::FIELD_SAME_AS )->get( $publisher_id );
-		if ( ! empty( $sameas ) ) {
-			$params['publisher']['sameAs'] = $sameas;
-		}
-
-		// Set the logo, only for http://schema.org/Organization as Person doesn't
-		// support the logo property.
-		//
-		// See http://schema.org/logo.
-		if ( 1 !== preg_match( '~Organization$~', $type['uri'] ) ) {
-			return;
-		}
-
-		// Get the publisher logo.
-		$publisher_logo = $this->get_publisher_logo( $post->ID );
-
-		// Bail out if the publisher logo isn't set.
-		if ( false === $publisher_logo ) {
-			return;
-		}
-
-		// Copy over some useful properties.
-		//
-		// See https://developers.google.com/search/docs/data-types/articles.
-		$params['publisher']['logo']['@type'] = 'ImageObject';
-		$params['publisher']['logo']['url']   = $publisher_logo['url'];
-
-		// 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.
-		$params['publisher']['logo']['width']  = $publisher_logo['width'];
-		$params['publisher']['logo']['height'] = $publisher_logo['height'];
-
-	}
-
-	/**
-	 * Get the publisher logo structure.
-	 *
-	 * The function returns false when the publisher logo cannot be determined, i.e.:
-	 *  - the post has no featured image.
-	 *  - the featured image has no file.
-	 *  - a wp_image_editor instance cannot be instantiated on the original file or on the publisher logo file.
-	 *
-	 * @param int $post_id The post id.
-	 *
-	 * @return array|false Returns an array with the `url`, `width` and `height` for the publisher logo or false in case
-	 *  of errors.
-	 * @since 3.19.2
-	 * @see https://github.com/insideout10/wordlift-plugin/issues/823 related issue.
-	 */
-	private function get_publisher_logo( $post_id ) {
-
-		// Get the featured image for the post.
-		$thumbnail_id = get_post_thumbnail_id( $post_id );
-
-		// Bail out if thumbnail not available.
-		if ( empty( $thumbnail_id ) || 0 === $thumbnail_id ) {
-			$this->log->info( "Featured image not set for post $post_id." );
-
-			return false;
-		}
-
-		// Get the uploads base URL.
-		$uploads_dir = wp_upload_dir();
-
-		// Get the attachment metadata.
-		$metadata = wp_get_attachment_metadata( $thumbnail_id );
-
-		// Bail out if the file isn't set.
-		if ( ! isset( $metadata['file'] ) ) {
-			$this->log->warn( "Featured image file not found for post $post_id." );
-
-			return false;
-		}
-
-		// Retrieve the relative filename, e.g. "2018/05/logo_publisher.png"
-		$path = $uploads_dir['basedir'] . DIRECTORY_SEPARATOR . $metadata['file'];
-
-		// Use image src, if local file does not exist. @see https://github.com/insideout10/wordlift-plugin/issues/1149
-		if ( ! file_exists( $path ) ) {
-			$this->log->warn( "Featured image file $path doesn't exist for post $post_id." );
-
-			$attachment_image_src = wp_get_attachment_image_src( $thumbnail_id, '' );
-			if ( $attachment_image_src ) {
-				return array(
-					'url'    => $attachment_image_src[0],
-					'width'  => $attachment_image_src[1],
-					'height' => $attachment_image_src[2],
-				);
-			}
-
-			// Bail out if we cant fetch wp_get_attachment_image_src
-			return false;
-
-		}
-
-		// Try to get the image editor and bail out if the editor cannot be instantiated.
-		$original_file_editor = wp_get_image_editor( $path );
-		if ( is_wp_error( $original_file_editor ) ) {
-			$this->log->warn( "Cannot instantiate WP Image Editor on file $path for post $post_id." );
-
-			return false;
-		}
-
-		// Generate the publisher logo filename, we cannot use the `width` and `height` because we're scaling
-		// and we don't actually know the end values.
-		$publisher_logo_path = $original_file_editor->generate_filename( '-publisher-logo' );
-
-		// If the file doesn't exist yet, create it.
-		if ( ! file_exists( $publisher_logo_path ) ) {
-			$original_file_editor->resize( 600, 60 );
-			$original_file_editor->save( $publisher_logo_path );
-		}
-
-		// Try to get the image editor and bail out if the editor cannot be instantiated.
-		$publisher_logo_editor = wp_get_image_editor( $publisher_logo_path );
-		if ( is_wp_error( $publisher_logo_editor ) ) {
-			$this->log->warn( "Cannot instantiate WP Image Editor on file $publisher_logo_path for post $post_id." );
-
-			return false;
-		}
-
-		// Get the actual size.
-		$size = $publisher_logo_editor->get_size();
-
-		// Finally return the array with data.
-		return array(
-			'url'    => $uploads_dir['baseurl'] . substr( $publisher_logo_path, strlen( $uploads_dir['basedir'] ) ),
-			'width'  => $size['width'],
-			'height' => $size['height'],
-		);
-	}
-
-	/**
-	 * @param $references
-	 * @param $post
-	 * @param $jsonld
-	 *
-	 * @return void
-	 */
-	private function set_mentions_and_about( $references, $post, &$jsonld ) {
-
-		if ( count( $references ) === 0 ) {
-			return;
-		}
-
-		// Prepare the `about` and `mentions` array.
-		$mentions = array();
-		$about    = array();
-
-		// If the entity is in the title, then it should be an `about`.
-		foreach ( $references as $reference ) {
-
-			if ( ! $reference instanceof Reference ) {
-				// This condition should never be reached.
-				continue;
-			}
-
-			// Get the entity labels.
-			$labels = Wordlift_Entity_Service::get_instance()->get_labels( $reference->get_id(), $reference->get_type() );
-			// Get the entity URI.
-			$item = array(
-				'@id' => Wordlift_Entity_Service::get_instance()->get_uri( $reference->get_id(), $reference->get_type() ),
-			);
-
-			$escaped_labels = array_map(
-				function ( $value ) {
-					return preg_quote( $value, '/' );
-				},
-				$labels
-			);
-
-			$matches = false;
-
-			// When the title is empty, then we shouldn't yield a match to about section.
-			if ( array_filter( $escaped_labels ) ) {
-				// Check if the labels match any part of the title.
-				$matches = 1 === preg_match( '/' . implode( '|', $escaped_labels ) . '/', $post->post_title );
-			}
-
-			// If the title matches, assign the entity to the about, otherwise to the mentions.
-			if ( $matches ) {
-				$about[] = $item;
-			} else {
-				$mentions[] = $item;
-			}
-		}
-
-		// If we have abouts, assign them to the JSON-LD.
-		if ( 0 < count( $about ) ) {
-			$jsonld['about'] = $about;
-		}
-
-		// If we have mentions, assign them to the JSON-LD.
-		if ( 0 < count( $mentions ) ) {
-			$jsonld['mentions'] = $mentions;
-		}
-
-		return $jsonld;
-	}
-
-	/**
-	 * Convert references to abstract data type if we find any.
-	 *
-	 * @param $references array<int|Reference>
-	 *
-	 * @return Reference[]
-	 */
-	private function convert_references( $references ) {
-		return array_map(
-			function ( $reference ) {
-				// Legacy code may still push numerical references to this
-				// $references variable, so convert it to post references.
-				if ( is_numeric( $reference ) ) {
-					  return new Post_Reference( $reference );
-				}
-
-				return $reference;
-
-			},
-			$references
-		);
-	}
+        if ( ! empty( $jsonld['@type'] ) && 'WebPage' !== $jsonld['@type'] ) {
+            $post_adapter    = new Wordlift_Post_Adapter( $post_id );
+            $keywords        = $post_adapter->keywords();
+            $article_section = $post_adapter->article_section();
+            $comment_count   = $post_adapter->comment_count();
+            $locale          = $post_adapter->locale();
+
+            if ( isset( $keywords ) ) {
+                $jsonld['keywords'] = $keywords;
+            }
+            if ( ! empty( $article_section ) ) {
+                $jsonld['articleSection'] = $article_section;
+            }
+            $jsonld['commentCount'] = $comment_count;
+            $jsonld['inLanguage']   = $locale;
+            $post_adapter->add_references( $post_id, $references );
+        }
+
+        // Set the publisher.
+        $this->set_publisher( $jsonld );
+
+        $references = $this->convert_references( $references );
+
+        // Process the references if any.
+        $this->set_mentions_and_about( $references, $post, $jsonld );
+
+        // Finally set the author.
+        $jsonld['author'] = $this->get_author( $post->post_author, $references );
+
+        // Return the JSON-LD if filters are disabled by the client.
+        if ( $this->disable_convert_filters ) {
+            return $jsonld;
+        }
+
+        /**
+         * Call the `wl_post_jsonld_array` filter. This filter allows 3rd parties to also modify the references.
+         *
+         * @param array $value {
+         *
+         * @type array $jsonld The JSON-LD structure.
+         * @type int[] $references An array of post IDs.
+         * }
+         * @since 3.25.0
+         *
+         * @see https://www.geeklab.info/2010/04/wordpress-pass-variables-by-reference-with-apply_filter/
+         *
+         * @api
+         */
+        $ret_val = apply_filters(
+            'wl_post_jsonld_array',
+            array(
+                'jsonld'     => $jsonld,
+                'references' => $references,
+            ),
+            $post_id
+        );
+
+        $jsonld     = $ret_val['jsonld'];
+        $references = $ret_val['references'];
+
+        /**
+         * Call the `wl_post_jsonld` filter.
+         *
+         * @param array $jsonld The JSON-LD structure.
+         * @param int $post_id The {@link WP_Post} `id`.
+         * @param array $references The array of referenced entities.
+         *
+         * @since 3.14.0
+         *
+         * @api
+         */
+        return apply_filters( 'wl_post_jsonld', $jsonld, $post_id, $references );
+    }
+
+    /**
+     * Get the author's JSON-LD fragment.
+     *
+     * The JSON-LD fragment is generated using the {@link WP_User}'s data or
+     * the referenced entity if configured for the {@link WP_User}.
+     *
+     * @param int   $author_id The author {@link WP_User}'s `id`.
+     * @param array $references An array of referenced entities.
+     *
+     * @return string|array A JSON-LD structure.
+     * @since 3.14.0
+     */
+    public function get_author( $author_id, &$references ) {
+
+        // Get the entity bound to this user.
+        $entity_id = $this->user_service->get_entity( $author_id );
+
+        // If there's no entity bound return a simple author structure.
+        if ( empty( $entity_id ) || 'publish' !== get_post_status( $entity_id ) ) {
+
+            $author            = get_the_author_meta( 'display_name', $author_id );
+            $author_first_name = get_the_author_meta( 'first_name', $author_id );
+            $author_last_name  = get_the_author_meta( 'last_name', $author_id );
+            $author_uri        = $this->user_service->get_uri( $author_id );
+
+            return array(
+                '@type'      => 'Person',
+                '@id'        => $author_uri,
+                'name'       => $author,
+                'givenName'  => $author_first_name,
+                'familyName' => $author_last_name,
+                'url'        => get_author_posts_url( $author_id ),
+            );
+        }
+
+        // Add the author to the references.
+        $author_uri   = Wordlift_Entity_Service::get_instance()->get_uri( $entity_id );
+        $references[] = $entity_id;
+
+        // Return the JSON-LD for the referenced entity.
+        return array(
+            '@id' => $author_uri,
+        );
+    }
+
+    /**
+     * Enrich the provided params array with publisher data, if available.
+     *
+     * @param array $params The parameters array.
+     *
+     * @since 3.10.0
+     */
+    protected function set_publisher( &$params ) {
+
+        // If the publisher id isn't set don't do anything.
+        $publisher_id = Wordlift_Configuration_Service::get_instance()->get_publisher_id();
+        if ( null === $publisher_id ) {
+            return;
+        }
+
+        // Get the post instance.
+        $post = get_post( $publisher_id );
+        if ( null === $post ) {
+            // Publisher not found.
+            return;
+        }
+
+        // Get the item id.
+        $id = Wordlift_Entity_Service::get_instance()->get_uri( $publisher_id );
+
+        // Get the type.
+        $type = $this->entity_type_service->get( $publisher_id );
+
+        // Get the name.
+        $name = $post->post_title;
+
+        // Set the publisher data.
+        $params['publisher'] = array(
+            '@type' => $this->relative_to_context( $type['uri'] ),
+            '@id'   => $id,
+            'name'  => $name,
+        );
+
+        // Add the sameAs values associated with the publisher.
+        $storage_factory = Wordlift_Storage_Factory::get_instance();
+        $sameas          = $storage_factory->post_meta( Wordlift_Schema_Service::FIELD_SAME_AS )->get( $publisher_id );
+        if ( ! empty( $sameas ) ) {
+            $params['publisher']['sameAs'] = $sameas;
+        }
+
+        // Set the logo, only for http://schema.org/Organization as Person doesn't
+        // support the logo property.
+        //
+        // See http://schema.org/logo.
+        if ( 1 !== preg_match( '~Organization$~', $type['uri'] ) ) {
+            return;
+        }
+
+        // Get the publisher logo.
+        $publisher_logo = $this->get_publisher_logo( $post->ID );
+
+        // Bail out if the publisher logo isn't set.
+        if ( false === $publisher_logo ) {
+            return;
+        }
+
+        // Copy over some useful properties.
+        //
+        // See https://developers.google.com/search/docs/data-types/articles.
+        $params['publisher']['logo']['@type'] = 'ImageObject';
+        $params['publisher']['logo']['url']   = $publisher_logo['url'];
+
+        // 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.
+        $params['publisher']['logo']['width']  = $publisher_logo['width'];
+        $params['publisher']['logo']['height'] = $publisher_logo['height'];
+
+    }
+
+    /**
+     * Get the publisher logo structure.
+     *
+     * The function returns false when the publisher logo cannot be determined, i.e.:
+     *  - the post has no featured image.
+     *  - the featured image has no file.
+     *  - a wp_image_editor instance cannot be instantiated on the original file or on the publisher logo file.
+     *
+     * @param int $post_id The post id.
+     *
+     * @return array|false Returns an array with the `url`, `width` and `height` for the publisher logo or false in case
+     *  of errors.
+     * @since 3.19.2
+     * @see https://github.com/insideout10/wordlift-plugin/issues/823 related issue.
+     */
+    private function get_publisher_logo( $post_id ) {
+
+        // Get the featured image for the post.
+        $thumbnail_id = get_post_thumbnail_id( $post_id );
+
+        // Bail out if thumbnail not available.
+        if ( empty( $thumbnail_id ) || 0 === $thumbnail_id ) {
+            $this->log->info( "Featured image not set for post $post_id." );
+
+            return false;
+        }
+
+        // Get the uploads base URL.
+        $uploads_dir = wp_upload_dir();
+
+        // Get the attachment metadata.
+        $metadata = wp_get_attachment_metadata( $thumbnail_id );
+
+        // Bail out if the file isn't set.
+        if ( ! isset( $metadata['file'] ) ) {
+            $this->log->warn( "Featured image file not found for post $post_id." );
+
+            return false;
+        }
+
+        // Retrieve the relative filename, e.g. "2018/05/logo_publisher.png"
+        $path = $uploads_dir['basedir'] . DIRECTORY_SEPARATOR . $metadata['file'];
+
+        // Use image src, if local file does not exist. @see https://github.com/insideout10/wordlift-plugin/issues/1149
+        if ( ! file_exists( $path ) ) {
+            $this->log->warn( "Featured image file $path doesn't exist for post $post_id." );
+
+            $attachment_image_src = wp_get_attachment_image_src( $thumbnail_id, '' );
+            if ( $attachment_image_src ) {
+                return array(
+                    'url'    => $attachment_image_src[0],
+                    'width'  => $attachment_image_src[1],
+                    'height' => $attachment_image_src[2],
+                );
+            }
+
+            // Bail out if we cant fetch wp_get_attachment_image_src
+            return false;
+
+        }
+
+        // Try to get the image editor and bail out if the editor cannot be instantiated.
+        $original_file_editor = wp_get_image_editor( $path );
+        if ( is_wp_error( $original_file_editor ) ) {
+            $this->log->warn( "Cannot instantiate WP Image Editor on file $path for post $post_id." );
+
+            return false;
+        }
+
+        // Generate the publisher logo filename, we cannot use the `width` and `height` because we're scaling
+        // and we don't actually know the end values.
+        $publisher_logo_path = $original_file_editor->generate_filename( '-publisher-logo' );
+
+        // If the file doesn't exist yet, create it.
+        if ( ! file_exists( $publisher_logo_path ) ) {
+            $original_file_editor->resize( 600, 60 );
+            $original_file_editor->save( $publisher_logo_path );
+        }
+
+        // Try to get the image editor and bail out if the editor cannot be instantiated.
+        $publisher_logo_editor = wp_get_image_editor( $publisher_logo_path );
+        if ( is_wp_error( $publisher_logo_editor ) ) {
+            $this->log->warn( "Cannot instantiate WP Image Editor on file $publisher_logo_path for post $post_id." );
+
+            return false;
+        }
+
+        // Get the actual size.
+        $size = $publisher_logo_editor->get_size();
+
+        // Finally return the array with data.
+        return array(
+            'url'    => $uploads_dir['baseurl'] . substr( $publisher_logo_path, strlen( $uploads_dir['basedir'] ) ),
+            'width'  => $size['width'],
+            'height' => $size['height'],
+        );
+    }
+
+    /**
+     * @param $references
+     * @param $post
+     * @param $jsonld
+     *
+     * @return void
+     */
+    private function set_mentions_and_about( $references, $post, &$jsonld ) {
+
+        if ( count( $references ) === 0 ) {
+            return;
+        }
+
+        // Prepare the `about` and `mentions` array.
+        $mentions = array();
+        $about    = array();
+
+        // If the entity is in the title, then it should be an `about`.
+        foreach ( $references as $reference ) {
+
+            if ( ! $reference instanceof Reference ) {
+                // This condition should never be reached.
+                continue;
+            }
+
+            // Get the entity labels.
+            $labels = Wordlift_Entity_Service::get_instance()->get_labels( $reference->get_id(), $reference->get_type() );
+            // Get the entity URI.
+            $item = array(
+                '@id' => Wordlift_Entity_Service::get_instance()->get_uri( $reference->get_id(), $reference->get_type() ),
+            );
+
+            $escaped_labels = array_map(
+                function ( $value ) {
+                    return preg_quote( $value, '/' );
+                },
+                $labels
+            );
+
+            $matches = false;
+
+            // When the title is empty, then we shouldn't yield a match to about section.
+            if ( array_filter( $escaped_labels ) ) {
+                // Check if the labels match any part of the title.
+                $matches = 1 === preg_match( '/' . implode( '|', $escaped_labels ) . '/', $post->post_title );
+            }
+
+            // If the title matches, assign the entity to the about, otherwise to the mentions.
+            if ( $matches ) {
+                $about[] = $item;
+            } else {
+                $mentions[] = $item;
+            }
+        }
+
+        // If we have abouts, assign them to the JSON-LD.
+        if ( 0 < count( $about ) ) {
+            $jsonld['about'] = $about;
+        }
+
+        // If we have mentions, assign them to the JSON-LD.
+        if ( 0 < count( $mentions ) ) {
+            $jsonld['mentions'] = $mentions;
+        }
+
+        return $jsonld;
+    }
+
+    /**
+     * Convert references to abstract data type if we find any.
+     *
+     * @param $references array<int|Reference>
+     *
+     * @return Reference[]
+     */
+    private function convert_references( $references ) {
+        return array_map(
+            function ( $reference ) {
+                // Legacy code may still push numerical references to this
+                // $references variable, so convert it to post references.
+                if ( is_numeric( $reference ) ) {
+                        return new Post_Reference( $reference );
+                }
+
+                return $reference;
+
+            },
+            $references
+        );
+    }
 
 }

Spacing +97 added lines, -97 removed lines

@@ -51,12 +51,12 @@ 
 	 *
 	 * @since 3.10.0
 	 */
-	public function __construct( $entity_type_service, $user_service, $attachment_service, $disable_convert_filters = false ) {
-		parent::__construct( $entity_type_service, $user_service, $attachment_service, Wordlift_Property_Getter_Factory::create() );
+	public function __construct($entity_type_service, $user_service, $attachment_service, $disable_convert_filters = false) {
+		parent::__construct($entity_type_service, $user_service, $attachment_service, Wordlift_Property_Getter_Factory::create());
 		$this->disable_convert_filters = $disable_convert_filters;
 		$this->object_relation_service = Object_Relation_Service::get_instance();
 		// Set a reference to the logger.
-		$this->log = Wordlift_Log_Service::get_logger( 'Wordlift_Post_To_Jsonld_Converter' );
+		$this->log = Wordlift_Log_Service::get_logger('Wordlift_Post_To_Jsonld_Converter');
 
 		self::$instance = $this;
 
@@ -68,7 +68,7 @@ 
 	}
 
 	public function new_instance_with_filters_disabled() {
-		return new static( $this->entity_type_service, $this->user_service, $this->attachment_service, true );
+		return new static($this->entity_type_service, $this->user_service, $this->attachment_service, true);
 	}
 
 	/**
@@ -82,30 +82,30 @@ 
 	 * @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 base JSON-LD and the list of entities referenced by this entity.
-		$jsonld = parent::convert( $post_id, $references, $references_infos );
+		$jsonld = parent::convert($post_id, $references, $references_infos);
 
 		// Set WebPage by default.
-		if ( empty( $jsonld['@type'] ) ) {
+		if (empty($jsonld['@type'])) {
 			$jsonld['@type'] = 'WebPage';
 		}
 
 		// Get the entity name.
 		$jsonld['headline'] = $post->post_title;
 
-		$custom_fields = $this->entity_type_service->get_custom_fields_for_post( $post_id );
+		$custom_fields = $this->entity_type_service->get_custom_fields_for_post($post_id);
 
-		if ( isset( $custom_fields ) ) {
-			$this->process_type_custom_fields( $jsonld, $custom_fields, $post, $references, $references_infos );
+		if (isset($custom_fields)) {
+			$this->process_type_custom_fields($jsonld, $custom_fields, $post, $references, $references_infos);
 		}
 
 		// Set the published and modified dates.
@@ -118,19 +118,19 @@ 
 		 */
 		try {
 			$default_timezone = date_default_timezone_get();
-			$timezone         = get_option( 'timezone_string' );
-			if ( ! empty( $timezone ) ) {
-				date_default_timezone_set( $timezone ); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.timezone_change_date_default_timezone_set
-				$jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i:sP', false, $post );
-				$jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i:sP', false, $post );
-				date_default_timezone_set( $default_timezone ); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.timezone_change_date_default_timezone_set
+			$timezone         = get_option('timezone_string');
+			if ( ! empty($timezone)) {
+				date_default_timezone_set($timezone); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.timezone_change_date_default_timezone_set
+				$jsonld['datePublished'] = get_post_time('Y-m-d\TH:i:sP', false, $post);
+				$jsonld['dateModified']  = get_post_modified_time('Y-m-d\TH:i:sP', false, $post);
+				date_default_timezone_set($default_timezone); //phpcs:ignore WordPress.DateTime.RestrictedFunctions.timezone_change_date_default_timezone_set
 			} else {
-				$jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i', true, $post, false );
-				$jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i', true, $post, false );
+				$jsonld['datePublished'] = get_post_time('Y-m-d\TH:i', true, $post, false);
+				$jsonld['dateModified']  = get_post_modified_time('Y-m-d\TH:i', true, $post, false);
 			}
-		} catch ( Exception $e ) {
-			$jsonld['datePublished'] = get_post_time( 'Y-m-d\TH:i', true, $post, false );
-			$jsonld['dateModified']  = get_post_modified_time( 'Y-m-d\TH:i', true, $post, false );
+		} catch (Exception $e) {
+			$jsonld['datePublished'] = get_post_time('Y-m-d\TH:i', true, $post, false);
+			$jsonld['dateModified']  = get_post_modified_time('Y-m-d\TH:i', true, $post, false);
 		}
 
 		// Get the word count for the post.
@@ -141,8 +141,8 @@ 
 		 *
 		 * @since 3.20.0
 		 */
-		if ( ! empty( $jsonld['@type'] ) && 'WebPage' !== $jsonld['@type'] ) {
-			$post_adapter        = new Wordlift_Post_Adapter( $post_id );
+		if ( ! empty($jsonld['@type']) && 'WebPage' !== $jsonld['@type']) {
+			$post_adapter        = new Wordlift_Post_Adapter($post_id);
 			$jsonld['wordCount'] = $post_adapter->word_count();
 		}
 
@@ -153,37 +153,37 @@ 
 		 *
 		 * @since 3.27.2
 		 */
-		if ( ! empty( $jsonld['@type'] ) && 'WebPage' !== $jsonld['@type'] ) {
-			$post_adapter    = new Wordlift_Post_Adapter( $post_id );
+		if ( ! empty($jsonld['@type']) && 'WebPage' !== $jsonld['@type']) {
+			$post_adapter    = new Wordlift_Post_Adapter($post_id);
 			$keywords        = $post_adapter->keywords();
 			$article_section = $post_adapter->article_section();
 			$comment_count   = $post_adapter->comment_count();
 			$locale          = $post_adapter->locale();
 
-			if ( isset( $keywords ) ) {
+			if (isset($keywords)) {
 				$jsonld['keywords'] = $keywords;
 			}
-			if ( ! empty( $article_section ) ) {
+			if ( ! empty($article_section)) {
 				$jsonld['articleSection'] = $article_section;
 			}
 			$jsonld['commentCount'] = $comment_count;
 			$jsonld['inLanguage']   = $locale;
-			$post_adapter->add_references( $post_id, $references );
+			$post_adapter->add_references($post_id, $references);
 		}
 
 		// Set the publisher.
-		$this->set_publisher( $jsonld );
+		$this->set_publisher($jsonld);
 
-		$references = $this->convert_references( $references );
+		$references = $this->convert_references($references);
 
 		// Process the references if any.
-		$this->set_mentions_and_about( $references, $post, $jsonld );
+		$this->set_mentions_and_about($references, $post, $jsonld);
 
 		// Finally set the author.
-		$jsonld['author'] = $this->get_author( $post->post_author, $references );
+		$jsonld['author'] = $this->get_author($post->post_author, $references);
 
 		// Return the JSON-LD if filters are disabled by the client.
-		if ( $this->disable_convert_filters ) {
+		if ($this->disable_convert_filters) {
 			return $jsonld;
 		}
 
@@ -224,7 +224,7 @@ 
 		 *
 		 * @api
 		 */
-		return apply_filters( 'wl_post_jsonld', $jsonld, $post_id, $references );
+		return apply_filters('wl_post_jsonld', $jsonld, $post_id, $references);
 	}
 
 	/**
@@ -239,18 +239,18 @@ 
 	 * @return string|array A JSON-LD structure.
 	 * @since 3.14.0
 	 */
-	public function get_author( $author_id, &$references ) {
+	public function get_author($author_id, &$references) {
 
 		// Get the entity bound to this user.
-		$entity_id = $this->user_service->get_entity( $author_id );
+		$entity_id = $this->user_service->get_entity($author_id);
 
 		// If there's no entity bound return a simple author structure.
-		if ( empty( $entity_id ) || 'publish' !== get_post_status( $entity_id ) ) {
+		if (empty($entity_id) || 'publish' !== get_post_status($entity_id)) {
 
-			$author            = get_the_author_meta( 'display_name', $author_id );
-			$author_first_name = get_the_author_meta( 'first_name', $author_id );
-			$author_last_name  = get_the_author_meta( 'last_name', $author_id );
-			$author_uri        = $this->user_service->get_uri( $author_id );
+			$author            = get_the_author_meta('display_name', $author_id);
+			$author_first_name = get_the_author_meta('first_name', $author_id);
+			$author_last_name  = get_the_author_meta('last_name', $author_id);
+			$author_uri        = $this->user_service->get_uri($author_id);
 
 			return array(
 				'@type'      => 'Person',
@@ -258,12 +258,12 @@ 
 				'name'       => $author,
 				'givenName'  => $author_first_name,
 				'familyName' => $author_last_name,
-				'url'        => get_author_posts_url( $author_id ),
+				'url'        => get_author_posts_url($author_id),
 			);
 		}
 
 		// Add the author to the references.
-		$author_uri   = Wordlift_Entity_Service::get_instance()->get_uri( $entity_id );
+		$author_uri   = Wordlift_Entity_Service::get_instance()->get_uri($entity_id);
 		$references[] = $entity_id;
 
 		// Return the JSON-LD for the referenced entity.
@@ -279,41 +279,41 @@ 
 	 *
 	 * @since 3.10.0
 	 */
-	protected function set_publisher( &$params ) {
+	protected function set_publisher(&$params) {
 
 		// If the publisher id isn't set don't do anything.
 		$publisher_id = Wordlift_Configuration_Service::get_instance()->get_publisher_id();
-		if ( null === $publisher_id ) {
+		if (null === $publisher_id) {
 			return;
 		}
 
 		// Get the post instance.
-		$post = get_post( $publisher_id );
-		if ( null === $post ) {
+		$post = get_post($publisher_id);
+		if (null === $post) {
 			// Publisher not found.
 			return;
 		}
 
 		// Get the item id.
-		$id = Wordlift_Entity_Service::get_instance()->get_uri( $publisher_id );
+		$id = Wordlift_Entity_Service::get_instance()->get_uri($publisher_id);
 
 		// Get the type.
-		$type = $this->entity_type_service->get( $publisher_id );
+		$type = $this->entity_type_service->get($publisher_id);
 
 		// Get the name.
 		$name = $post->post_title;
 
 		// Set the publisher data.
 		$params['publisher'] = array(
-			'@type' => $this->relative_to_context( $type['uri'] ),
+			'@type' => $this->relative_to_context($type['uri']),
 			'@id'   => $id,
 			'name'  => $name,
 		);
 
 		// Add the sameAs values associated with the publisher.
 		$storage_factory = Wordlift_Storage_Factory::get_instance();
-		$sameas          = $storage_factory->post_meta( Wordlift_Schema_Service::FIELD_SAME_AS )->get( $publisher_id );
-		if ( ! empty( $sameas ) ) {
+		$sameas          = $storage_factory->post_meta(Wordlift_Schema_Service::FIELD_SAME_AS)->get($publisher_id);
+		if ( ! empty($sameas)) {
 			$params['publisher']['sameAs'] = $sameas;
 		}
 
@@ -321,15 +321,15 @@ 
 		// support the logo property.
 		//
 		// See http://schema.org/logo.
-		if ( 1 !== preg_match( '~Organization$~', $type['uri'] ) ) {
+		if (1 !== preg_match('~Organization$~', $type['uri'])) {
 			return;
 		}
 
 		// Get the publisher logo.
-		$publisher_logo = $this->get_publisher_logo( $post->ID );
+		$publisher_logo = $this->get_publisher_logo($post->ID);
 
 		// Bail out if the publisher logo isn't set.
-		if ( false === $publisher_logo ) {
+		if (false === $publisher_logo) {
 			return;
 		}
 
@@ -363,14 +363,14 @@ 
 	 * @since 3.19.2
 	 * @see https://github.com/insideout10/wordlift-plugin/issues/823 related issue.
 	 */
-	private function get_publisher_logo( $post_id ) {
+	private function get_publisher_logo($post_id) {
 
 		// Get the featured image for the post.
-		$thumbnail_id = get_post_thumbnail_id( $post_id );
+		$thumbnail_id = get_post_thumbnail_id($post_id);
 
 		// Bail out if thumbnail not available.
-		if ( empty( $thumbnail_id ) || 0 === $thumbnail_id ) {
-			$this->log->info( "Featured image not set for post $post_id." );
+		if (empty($thumbnail_id) || 0 === $thumbnail_id) {
+			$this->log->info("Featured image not set for post $post_id.");
 
 			return false;
 		}
@@ -379,24 +379,24 @@ 
 		$uploads_dir = wp_upload_dir();
 
 		// Get the attachment metadata.
-		$metadata = wp_get_attachment_metadata( $thumbnail_id );
+		$metadata = wp_get_attachment_metadata($thumbnail_id);
 
 		// Bail out if the file isn't set.
-		if ( ! isset( $metadata['file'] ) ) {
-			$this->log->warn( "Featured image file not found for post $post_id." );
+		if ( ! isset($metadata['file'])) {
+			$this->log->warn("Featured image file not found for post $post_id.");
 
 			return false;
 		}
 
 		// Retrieve the relative filename, e.g. "2018/05/logo_publisher.png"
-		$path = $uploads_dir['basedir'] . DIRECTORY_SEPARATOR . $metadata['file'];
+		$path = $uploads_dir['basedir'].DIRECTORY_SEPARATOR.$metadata['file'];
 
 		// Use image src, if local file does not exist. @see https://github.com/insideout10/wordlift-plugin/issues/1149
-		if ( ! file_exists( $path ) ) {
-			$this->log->warn( "Featured image file $path doesn't exist for post $post_id." );
+		if ( ! file_exists($path)) {
+			$this->log->warn("Featured image file $path doesn't exist for post $post_id.");
 
-			$attachment_image_src = wp_get_attachment_image_src( $thumbnail_id, '' );
-			if ( $attachment_image_src ) {
+			$attachment_image_src = wp_get_attachment_image_src($thumbnail_id, '');
+			if ($attachment_image_src) {
 				return array(
 					'url'    => $attachment_image_src[0],
 					'width'  => $attachment_image_src[1],
@@ -410,27 +410,27 @@ 
 		}
 
 		// Try to get the image editor and bail out if the editor cannot be instantiated.
-		$original_file_editor = wp_get_image_editor( $path );
-		if ( is_wp_error( $original_file_editor ) ) {
-			$this->log->warn( "Cannot instantiate WP Image Editor on file $path for post $post_id." );
+		$original_file_editor = wp_get_image_editor($path);
+		if (is_wp_error($original_file_editor)) {
+			$this->log->warn("Cannot instantiate WP Image Editor on file $path for post $post_id.");
 
 			return false;
 		}
 
 		// Generate the publisher logo filename, we cannot use the `width` and `height` because we're scaling
 		// and we don't actually know the end values.
-		$publisher_logo_path = $original_file_editor->generate_filename( '-publisher-logo' );
+		$publisher_logo_path = $original_file_editor->generate_filename('-publisher-logo');
 
 		// If the file doesn't exist yet, create it.
-		if ( ! file_exists( $publisher_logo_path ) ) {
-			$original_file_editor->resize( 600, 60 );
-			$original_file_editor->save( $publisher_logo_path );
+		if ( ! file_exists($publisher_logo_path)) {
+			$original_file_editor->resize(600, 60);
+			$original_file_editor->save($publisher_logo_path);
 		}
 
 		// Try to get the image editor and bail out if the editor cannot be instantiated.
-		$publisher_logo_editor = wp_get_image_editor( $publisher_logo_path );
-		if ( is_wp_error( $publisher_logo_editor ) ) {
-			$this->log->warn( "Cannot instantiate WP Image Editor on file $publisher_logo_path for post $post_id." );
+		$publisher_logo_editor = wp_get_image_editor($publisher_logo_path);
+		if (is_wp_error($publisher_logo_editor)) {
+			$this->log->warn("Cannot instantiate WP Image Editor on file $publisher_logo_path for post $post_id.");
 
 			return false;
 		}
@@ -440,7 +440,7 @@ 
 
 		// Finally return the array with data.
 		return array(
-			'url'    => $uploads_dir['baseurl'] . substr( $publisher_logo_path, strlen( $uploads_dir['basedir'] ) ),
+			'url'    => $uploads_dir['baseurl'].substr($publisher_logo_path, strlen($uploads_dir['basedir'])),
 			'width'  => $size['width'],
 			'height' => $size['height'],
 		);
@@ -453,9 +453,9 @@ 
 	 *
 	 * @return void
 	 */
-	private function set_mentions_and_about( $references, $post, &$jsonld ) {
+	private function set_mentions_and_about($references, $post, &$jsonld) {
 
-		if ( count( $references ) === 0 ) {
+		if (count($references) === 0) {
 			return;
 		}
 
@@ -464,23 +464,23 @@ 
 		$about    = array();
 
 		// If the entity is in the title, then it should be an `about`.
-		foreach ( $references as $reference ) {
+		foreach ($references as $reference) {
 
-			if ( ! $reference instanceof Reference ) {
+			if ( ! $reference instanceof Reference) {
 				// This condition should never be reached.
 				continue;
 			}
 
 			// Get the entity labels.
-			$labels = Wordlift_Entity_Service::get_instance()->get_labels( $reference->get_id(), $reference->get_type() );
+			$labels = Wordlift_Entity_Service::get_instance()->get_labels($reference->get_id(), $reference->get_type());
 			// Get the entity URI.
 			$item = array(
-				'@id' => Wordlift_Entity_Service::get_instance()->get_uri( $reference->get_id(), $reference->get_type() ),
+				'@id' => Wordlift_Entity_Service::get_instance()->get_uri($reference->get_id(), $reference->get_type()),
 			);
 
 			$escaped_labels = array_map(
-				function ( $value ) {
-					return preg_quote( $value, '/' );
+				function($value) {
+					return preg_quote($value, '/');
 				},
 				$labels
 			);
@@ -488,13 +488,13 @@ 
 			$matches = false;
 
 			// When the title is empty, then we shouldn't yield a match to about section.
-			if ( array_filter( $escaped_labels ) ) {
+			if (array_filter($escaped_labels)) {
 				// Check if the labels match any part of the title.
-				$matches = 1 === preg_match( '/' . implode( '|', $escaped_labels ) . '/', $post->post_title );
+				$matches = 1 === preg_match('/'.implode('|', $escaped_labels).'/', $post->post_title);
 			}
 
 			// If the title matches, assign the entity to the about, otherwise to the mentions.
-			if ( $matches ) {
+			if ($matches) {
 				$about[] = $item;
 			} else {
 				$mentions[] = $item;
@@ -502,12 +502,12 @@ 
 		}
 
 		// If we have abouts, assign them to the JSON-LD.
-		if ( 0 < count( $about ) ) {
+		if (0 < count($about)) {
 			$jsonld['about'] = $about;
 		}
 
 		// If we have mentions, assign them to the JSON-LD.
-		if ( 0 < count( $mentions ) ) {
+		if (0 < count($mentions)) {
 			$jsonld['mentions'] = $mentions;
 		}
 
@@ -521,13 +521,13 @@ 
 	 *
 	 * @return Reference[]
 	 */
-	private function convert_references( $references ) {
+	private function convert_references($references) {
 		return array_map(
-			function ( $reference ) {
+			function($reference) {
 				// Legacy code may still push numerical references to this
 				// $references variable, so convert it to post references.
-				if ( is_numeric( $reference ) ) {
-					  return new Post_Reference( $reference );
+				if (is_numeric($reference)) {
+					  return new Post_Reference($reference);
 				}
 
 				return $reference;

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

Indentation +47 added lines, -47 removed lines

@@ -13,64 +13,64 @@
  */
 class Wordlift_Debug_Service {
 
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.7.2
-	 * @access private
-	 * @var Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.7.2
+     * @access private
+     * @var Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
 
-	/**
-	 * A {@link Wordlift_Uri_Service} instance.
-	 *
-	 * @since  3.10.0
-	 * @access private
-	 * @var \Wordlift_Uri_Service $uri_service A {@link Wordlift_Uri_Service} instance.
-	 */
-	private $uri_service;
+    /**
+     * A {@link Wordlift_Uri_Service} instance.
+     *
+     * @since  3.10.0
+     * @access private
+     * @var \Wordlift_Uri_Service $uri_service A {@link Wordlift_Uri_Service} instance.
+     */
+    private $uri_service;
 
-	/**
-	 * Wordlift_Debug_Service constructor.
-	 *
-	 * @since 3.7.2
-	 *
-	 * @param Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 * @param \Wordlift_Uri_Service   $uri_service    A {@link Wordlift_Uri_Service} instance.
-	 */
-	public function __construct( $entity_service, $uri_service ) {
+    /**
+     * Wordlift_Debug_Service constructor.
+     *
+     * @since 3.7.2
+     *
+     * @param Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     * @param \Wordlift_Uri_Service   $uri_service    A {@link Wordlift_Uri_Service} instance.
+     */
+    public function __construct( $entity_service, $uri_service ) {
 
-		$this->entity_service = $entity_service;
-		$this->uri_service    = $uri_service;
+        $this->entity_service = $entity_service;
+        $this->uri_service    = $uri_service;
 
-		add_action( 'wp_ajax_wl_dump_uri', array( $this, 'dump_uri' ) );
+        add_action( 'wp_ajax_wl_dump_uri', array( $this, 'dump_uri' ) );
 
-	}
+    }
 
-	public function dump_uri() {
+    public function dump_uri() {
 
-		if ( ! isset( $_GET['id'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			wp_send_json_error( 'id not set' );
-		}
+        if ( ! isset( $_GET['id'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+            wp_send_json_error( 'id not set' );
+        }
 
-		$post_id = (int) $_GET['id']; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+        $post_id = (int) $_GET['id']; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
-		$post = get_post( $post_id );
+        $post = get_post( $post_id );
 
-		$uri       = $this->entity_service->get_uri( $post_id );
-		$build_uri = $this->uri_service->build_uri( $post->post_title, $post->post_type );
+        $uri       = $this->entity_service->get_uri( $post_id );
+        $build_uri = $this->uri_service->build_uri( $post->post_title, $post->post_type );
 
-		wp_send_json_success(
-			array(
-				'uri'               => $uri,
-				'post_title'        => sprintf( '%s (%s)', $post->post_title, mb_detect_encoding( $post->post_title ) ),
-				'post_title_ascii'  => mb_convert_encoding( $post->post_title, 'ASCII' ),
-				'build_uri'         => $build_uri,
-				'build_uri_convert' => mb_convert_encoding( $build_uri, 'ASCII' ),
-			)
-		);
+        wp_send_json_success(
+            array(
+                'uri'               => $uri,
+                'post_title'        => sprintf( '%s (%s)', $post->post_title, mb_detect_encoding( $post->post_title ) ),
+                'post_title_ascii'  => mb_convert_encoding( $post->post_title, 'ASCII' ),
+                'build_uri'         => $build_uri,
+                'build_uri_convert' => mb_convert_encoding( $build_uri, 'ASCII' ),
+            )
+        );
 
-	}
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -39,35 +39,35 @@
 	 * @param Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
 	 * @param \Wordlift_Uri_Service   $uri_service    A {@link Wordlift_Uri_Service} instance.
 	 */
-	public function __construct( $entity_service, $uri_service ) {
+	public function __construct($entity_service, $uri_service) {
 
 		$this->entity_service = $entity_service;
 		$this->uri_service    = $uri_service;
 
-		add_action( 'wp_ajax_wl_dump_uri', array( $this, 'dump_uri' ) );
+		add_action('wp_ajax_wl_dump_uri', array($this, 'dump_uri'));
 
 	}
 
 	public function dump_uri() {
 
-		if ( ! isset( $_GET['id'] ) ) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
-			wp_send_json_error( 'id not set' );
+		if ( ! isset($_GET['id'])) { //phpcs:ignore WordPress.Security.NonceVerification.Recommended
+			wp_send_json_error('id not set');
 		}
 
 		$post_id = (int) $_GET['id']; //phpcs:ignore WordPress.Security.NonceVerification.Recommended
 
-		$post = get_post( $post_id );
+		$post = get_post($post_id);
 
-		$uri       = $this->entity_service->get_uri( $post_id );
-		$build_uri = $this->uri_service->build_uri( $post->post_title, $post->post_type );
+		$uri       = $this->entity_service->get_uri($post_id);
+		$build_uri = $this->uri_service->build_uri($post->post_title, $post->post_type);
 
 		wp_send_json_success(
 			array(
 				'uri'               => $uri,
-				'post_title'        => sprintf( '%s (%s)', $post->post_title, mb_detect_encoding( $post->post_title ) ),
-				'post_title_ascii'  => mb_convert_encoding( $post->post_title, 'ASCII' ),
+				'post_title'        => sprintf('%s (%s)', $post->post_title, mb_detect_encoding($post->post_title)),
+				'post_title_ascii'  => mb_convert_encoding($post->post_title, 'ASCII'),
 				'build_uri'         => $build_uri,
-				'build_uri_convert' => mb_convert_encoding( $build_uri, 'ASCII' ),
+				'build_uri_convert' => mb_convert_encoding($build_uri, 'ASCII'),
 			)
 		);
 

src/includes/class-wordlift-property-factory.php

Indentation +43 added lines, -43 removed lines

@@ -11,58 +11,58 @@
  */
 class Wordlift_Property_Factory {
 
-	/**
-	 * The default {@link Wordlift_Property_Service}.
-	 *
-	 * @since 3.7.0
-	 * @access private
-	 * @var \Wordlift_Property_Service $default_property_service The default {@link Wordlift_Property_Service}.
-	 */
-	private $default_property_service;
+    /**
+     * The default {@link Wordlift_Property_Service}.
+     *
+     * @since 3.7.0
+     * @access private
+     * @var \Wordlift_Property_Service $default_property_service The default {@link Wordlift_Property_Service}.
+     */
+    private $default_property_service;
 
-	private $property_services = array();
+    private $property_services = array();
 
-	/**
-	 * Wordlift_Property_Factory constructor.
-	 *
-	 * @since 3.7.0
-	 *
-	 * @param \Wordlift_Property_Service $default_property_service
-	 */
-	public function __construct( $default_property_service ) {
+    /**
+     * Wordlift_Property_Factory constructor.
+     *
+     * @since 3.7.0
+     *
+     * @param \Wordlift_Property_Service $default_property_service
+     */
+    public function __construct( $default_property_service ) {
 
-		$this->default_property_service = $default_property_service;
+        $this->default_property_service = $default_property_service;
 
-	}
+    }
 
-	/**
-	 * Set the {@link Wordlift_Property_Service} which handles that meta key.
-	 *
-	 * @since 3.7.0
-	 *
-	 * @param string                     $meta_key WordPress' meta key.
-	 * @param \Wordlift_Property_Service $property_service A {@link Wordlift_Property_Service} instance.
-	 */
-	public function register( $meta_key, $property_service ) {
+    /**
+     * Set the {@link Wordlift_Property_Service} which handles that meta key.
+     *
+     * @since 3.7.0
+     *
+     * @param string                     $meta_key WordPress' meta key.
+     * @param \Wordlift_Property_Service $property_service A {@link Wordlift_Property_Service} instance.
+     */
+    public function register( $meta_key, $property_service ) {
 
-		$this->property_services[ $meta_key ] = $property_service;
+        $this->property_services[ $meta_key ] = $property_service;
 
-	}
+    }
 
-	/**
-	 * Get the {@link Wordlift_Property_Service} which handles the specified meta key.
-	 *
-	 * @since 3.7.0
-	 *
-	 * @param $meta_key
-	 *
-	 * @return \Wordlift_Property_Service The {@link Wordlift_Property_Service} which handles the specified meta key.
-	 */
-	public function get( $meta_key ) {
+    /**
+     * Get the {@link Wordlift_Property_Service} which handles the specified meta key.
+     *
+     * @since 3.7.0
+     *
+     * @param $meta_key
+     *
+     * @return \Wordlift_Property_Service The {@link Wordlift_Property_Service} which handles the specified meta key.
+     */
+    public function get( $meta_key ) {
 
-		$service = $this->property_services[ $meta_key ];
+        $service = $this->property_services[ $meta_key ];
 
-		return $service ? $service : $this->default_property_service;
-	}
+        return $service ? $service : $this->default_property_service;
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -29,7 +29,7 @@ 
 	 *
 	 * @param \Wordlift_Property_Service $default_property_service
 	 */
-	public function __construct( $default_property_service ) {
+	public function __construct($default_property_service) {
 
 		$this->default_property_service = $default_property_service;
 
@@ -43,9 +43,9 @@ 
 	 * @param string                     $meta_key WordPress' meta key.
 	 * @param \Wordlift_Property_Service $property_service A {@link Wordlift_Property_Service} instance.
 	 */
-	public function register( $meta_key, $property_service ) {
+	public function register($meta_key, $property_service) {
 
-		$this->property_services[ $meta_key ] = $property_service;
+		$this->property_services[$meta_key] = $property_service;
 
 	}
 
@@ -58,9 +58,9 @@ 
 	 *
 	 * @return \Wordlift_Property_Service The {@link Wordlift_Property_Service} which handles the specified meta key.
 	 */
-	public function get( $meta_key ) {
+	public function get($meta_key) {
 
-		$service = $this->property_services[ $meta_key ];
+		$service = $this->property_services[$meta_key];
 
 		return $service ? $service : $this->default_property_service;
 	}

src/includes/linked-data/storage/class-wordlift-post-image-storage.php

Indentation +41 added lines, -41 removed lines

@@ -18,46 +18,46 @@
  */
 class Wordlift_Post_Image_Storage extends Wordlift_Storage {
 
-	/**
-	 * Get the property value.
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array|string|null A single string, or an array of values or null
-	 *                           if the property isn't recognized.
-	 * @since 3.15.0
-	 */
-	public function get( $post_id ) {
-
-		// Prepare the return array.
-		$image_urls = array();
-
-		// If there is a featured image it has the priority.
-		$featured_image_id = get_post_thumbnail_id( $post_id );
-		if ( is_numeric( $featured_image_id ) && 0 < $featured_image_id ) {
-			$image_url = wp_get_attachment_url( $featured_image_id );
-
-			$image_urls[] = $image_url;
-		}
-
-		$images = get_children(
-			array(
-				'post_parent'    => $post_id,
-				'post_type'      => 'attachment',
-				'post_mime_type' => 'image',
-			)
-		);
-
-		// Collect the URLs.
-		foreach ( $images as $attachment_id => $attachment ) {
-			$image_url = wp_get_attachment_url( $attachment_id );
-			// Ensure the URL isn't collected already.
-			if ( ! in_array( $image_url, $image_urls, true ) ) {
-				array_push( $image_urls, $image_url );
-			}
-		}
-
-		return $image_urls;
-	}
+    /**
+     * Get the property value.
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array|string|null A single string, or an array of values or null
+     *                           if the property isn't recognized.
+     * @since 3.15.0
+     */
+    public function get( $post_id ) {
+
+        // Prepare the return array.
+        $image_urls = array();
+
+        // If there is a featured image it has the priority.
+        $featured_image_id = get_post_thumbnail_id( $post_id );
+        if ( is_numeric( $featured_image_id ) && 0 < $featured_image_id ) {
+            $image_url = wp_get_attachment_url( $featured_image_id );
+
+            $image_urls[] = $image_url;
+        }
+
+        $images = get_children(
+            array(
+                'post_parent'    => $post_id,
+                'post_type'      => 'attachment',
+                'post_mime_type' => 'image',
+            )
+        );
+
+        // Collect the URLs.
+        foreach ( $images as $attachment_id => $attachment ) {
+            $image_url = wp_get_attachment_url( $attachment_id );
+            // Ensure the URL isn't collected already.
+            if ( ! in_array( $image_url, $image_urls, true ) ) {
+                array_push( $image_urls, $image_url );
+            }
+        }
+
+        return $image_urls;
+    }
 
 }

Spacing +8 added lines, -8 removed lines

@@ -27,15 +27,15 @@ 
 	 *                           if the property isn't recognized.
 	 * @since 3.15.0
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
 		// Prepare the return array.
 		$image_urls = array();
 
 		// If there is a featured image it has the priority.
-		$featured_image_id = get_post_thumbnail_id( $post_id );
-		if ( is_numeric( $featured_image_id ) && 0 < $featured_image_id ) {
-			$image_url = wp_get_attachment_url( $featured_image_id );
+		$featured_image_id = get_post_thumbnail_id($post_id);
+		if (is_numeric($featured_image_id) && 0 < $featured_image_id) {
+			$image_url = wp_get_attachment_url($featured_image_id);
 
 			$image_urls[] = $image_url;
 		}
@@ -49,11 +49,11 @@ 
 		);
 
 		// Collect the URLs.
-		foreach ( $images as $attachment_id => $attachment ) {
-			$image_url = wp_get_attachment_url( $attachment_id );
+		foreach ($images as $attachment_id => $attachment) {
+			$image_url = wp_get_attachment_url($attachment_id);
 			// Ensure the URL isn't collected already.
-			if ( ! in_array( $image_url, $image_urls, true ) ) {
-				array_push( $image_urls, $image_url );
+			if ( ! in_array($image_url, $image_urls, true)) {
+				array_push($image_urls, $image_url);
 			}
 		}
 

src/includes/linked-data/storage/class-wordlift-post-meta-uri-storage.php

Indentation +42 added lines, -42 removed lines

@@ -18,53 +18,53 @@
  */
 class Wordlift_Post_Meta_Uri_Storage extends Wordlift_Post_Meta_Storage {
 
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
 
-	/**
-	 * Create a {@link Wordlift_Post_Meta_Uri_Storage} instance.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param string                   $meta_key       The meta key to read data from.
-	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	public function __construct( $meta_key, $entity_service ) {
-		parent::__construct( $meta_key );
+    /**
+     * Create a {@link Wordlift_Post_Meta_Uri_Storage} instance.
+     *
+     * @since 3.15.0
+     *
+     * @param string                   $meta_key       The meta key to read data from.
+     * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    public function __construct( $meta_key, $entity_service ) {
+        parent::__construct( $meta_key );
 
-		$this->entity_service = $entity_service;
+        $this->entity_service = $entity_service;
 
-	}
+    }
 
-	/**
-	 * Get the value for the specified meta key.
-	 *
-	 * The value is expected to be an entity post, for which the URI is loaded
-	 * and returned.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array An array of URIs (or an empty array if nothing is set).
-	 */
-	public function get( $post_id ) {
-		$values = parent::get( $post_id );
+    /**
+     * Get the value for the specified meta key.
+     *
+     * The value is expected to be an entity post, for which the URI is loaded
+     * and returned.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array An array of URIs (or an empty array if nothing is set).
+     */
+    public function get( $post_id ) {
+        $values = parent::get( $post_id );
 
-		$entity_service = $this->entity_service;
+        $entity_service = $this->entity_service;
 
-		return array_map(
-			function ( $item ) use ( $entity_service ) {
-				return $entity_service->get_uri( $item );
-			},
-			$values
-		);
-	}
+        return array_map(
+            function ( $item ) use ( $entity_service ) {
+                return $entity_service->get_uri( $item );
+            },
+            $values
+        );
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -35,8 +35,8 @@ 
 	 * @param string                   $meta_key       The meta key to read data from.
 	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
 	 */
-	public function __construct( $meta_key, $entity_service ) {
-		parent::__construct( $meta_key );
+	public function __construct($meta_key, $entity_service) {
+		parent::__construct($meta_key);
 
 		$this->entity_service = $entity_service;
 
@@ -54,14 +54,14 @@ 
 	 *
 	 * @return array An array of URIs (or an empty array if nothing is set).
 	 */
-	public function get( $post_id ) {
-		$values = parent::get( $post_id );
+	public function get($post_id) {
+		$values = parent::get($post_id);
 
 		$entity_service = $this->entity_service;
 
 		return array_map(
-			function ( $item ) use ( $entity_service ) {
-				return $entity_service->get_uri( $item );
+			function($item) use ($entity_service) {
+				return $entity_service->get_uri($item);
 			},
 			$values
 		);

src/includes/linked-data/storage/class-wordlift-post-taxonomy-storage.php

Indentation +43 added lines, -43 removed lines

@@ -18,52 +18,52 @@
  */
 class Wordlift_Post_Taxonomy_Storage extends Wordlift_Storage {
 
-	/**
-	 * The taxonomy name.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var string $taxonomy The taxonomy name.
-	 */
-	private $taxonomy;
+    /**
+     * The taxonomy name.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var string $taxonomy The taxonomy name.
+     */
+    private $taxonomy;
 
-	/**
-	 * Create a {@link Wordlift_Post_Taxonomy_Storage} with the specified
-	 * taxonomy name.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param string $taxonomy The taxonomy name.
-	 */
-	public function __construct( $taxonomy ) {
+    /**
+     * Create a {@link Wordlift_Post_Taxonomy_Storage} with the specified
+     * taxonomy name.
+     *
+     * @since 3.15.0
+     *
+     * @param string $taxonomy The taxonomy name.
+     */
+    public function __construct( $taxonomy ) {
 
-		$this->taxonomy = $taxonomy;
-	}
+        $this->taxonomy = $taxonomy;
+    }
 
-	/**
-	 * Get the taxonomy's terms associated with the specified {@link WP_Post}.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array|WP_Error An array of terms or {@link WP_Error} in case of error.
-	 */
-	public function get( $post_id ) {
+    /**
+     * Get the taxonomy's terms associated with the specified {@link WP_Post}.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array|WP_Error An array of terms or {@link WP_Error} in case of error.
+     */
+    public function get( $post_id ) {
 
-		return wp_get_post_terms(
-			$post_id,
-			$this->taxonomy,
-			array(
-				'hide_empty' => false,
-				// Because of #334 (and the AAM plugin) we changed fields from 'id=>slug' to 'all'.
-				// An issue has been opened with the AAM plugin author as well.
-				//
-				// see https://github.com/insideout10/wordlift-plugin/issues/334
-				// see https://wordpress.org/support/topic/idslug-not-working-anymore?replies=1#post-8806863
-				'fields'     => 'all',
-			)
-		);
-	}
+        return wp_get_post_terms(
+            $post_id,
+            $this->taxonomy,
+            array(
+                'hide_empty' => false,
+                // Because of #334 (and the AAM plugin) we changed fields from 'id=>slug' to 'all'.
+                // An issue has been opened with the AAM plugin author as well.
+                //
+                // see https://github.com/insideout10/wordlift-plugin/issues/334
+                // see https://wordpress.org/support/topic/idslug-not-working-anymore?replies=1#post-8806863
+                'fields'     => 'all',
+            )
+        );
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -35,7 +35,7 @@ 
 	 *
 	 * @param string $taxonomy The taxonomy name.
 	 */
-	public function __construct( $taxonomy ) {
+	public function __construct($taxonomy) {
 
 		$this->taxonomy = $taxonomy;
 	}
@@ -49,7 +49,7 @@ 
 	 *
 	 * @return array|WP_Error An array of terms or {@link WP_Error} in case of error.
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
 		return wp_get_post_terms(
 			$post_id,

src/includes/linked-data/storage/class-wordlift-post-schema-class-storage.php

Indentation +32 added lines, -32 removed lines

@@ -18,37 +18,37 @@
  */
 class Wordlift_Post_Schema_Class_Storage extends Wordlift_Storage {
 
-	/**
-	 * Get the schema class for the specified {@link WP_Post}.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return string|array An array of schema classes.
-	 */
-	public function get( $post_id ) {
-
-		// Get the type names (CamelCase).
-		$names = Wordlift_Entity_Type_Service::get_instance()->get_names( $post_id );
-
-		// If we don't find any type use the legacy function to get the URI.
-		if ( empty( $names ) ) {
-			$type = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
-
-			return $type['uri'];
-		}
-
-		// Prepend the `schema.org` base URI.
-		$uris = array_map(
-			function ( $item ) {
-				return "http://schema.org/$item";
-			},
-			$names
-		);
-
-		// Finally return the schema uri.
-		return 1 === count( $uris ) ? $uris[0] : $uris;
-	}
+    /**
+     * Get the schema class for the specified {@link WP_Post}.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return string|array An array of schema classes.
+     */
+    public function get( $post_id ) {
+
+        // Get the type names (CamelCase).
+        $names = Wordlift_Entity_Type_Service::get_instance()->get_names( $post_id );
+
+        // If we don't find any type use the legacy function to get the URI.
+        if ( empty( $names ) ) {
+            $type = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
+
+            return $type['uri'];
+        }
+
+        // Prepend the `schema.org` base URI.
+        $uris = array_map(
+            function ( $item ) {
+                return "http://schema.org/$item";
+            },
+            $names
+        );
+
+        // Finally return the schema uri.
+        return 1 === count( $uris ) ? $uris[0] : $uris;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -27,28 +27,28 @@
 	 *
 	 * @return string|array An array of schema classes.
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
 		// Get the type names (CamelCase).
-		$names = Wordlift_Entity_Type_Service::get_instance()->get_names( $post_id );
+		$names = Wordlift_Entity_Type_Service::get_instance()->get_names($post_id);
 
 		// If we don't find any type use the legacy function to get the URI.
-		if ( empty( $names ) ) {
-			$type = Wordlift_Entity_Type_Service::get_instance()->get( $post_id );
+		if (empty($names)) {
+			$type = Wordlift_Entity_Type_Service::get_instance()->get($post_id);
 
 			return $type['uri'];
 		}
 
 		// Prepend the `schema.org` base URI.
 		$uris = array_map(
-			function ( $item ) {
+			function($item) {
 				return "http://schema.org/$item";
 			},
 			$names
 		);
 
 		// Finally return the schema uri.
-		return 1 === count( $uris ) ? $uris[0] : $uris;
+		return 1 === count($uris) ? $uris[0] : $uris;
 	}
 
 }

src/includes/linked-data/storage/class-wordlift-post-related-storage.php

Indentation +44 added lines, -44 removed lines

@@ -21,56 +21,56 @@
  */
 class Wordlift_Post_Related_Storage extends Wordlift_Storage {
 
-	/**
-	 * The {@link Wordlift_Entity_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	private $entity_service;
+    /**
+     * The {@link Wordlift_Entity_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    private $entity_service;
 
-	/**
-	 * Create a {@link Wordlift_Post_Related_Storage} instance.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
-	 */
-	public function __construct( $entity_service ) {
+    /**
+     * Create a {@link Wordlift_Post_Related_Storage} instance.
+     *
+     * @since 3.15.0
+     *
+     * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
+     */
+    public function __construct( $entity_service ) {
 
-		$this->entity_service = $entity_service;
+        $this->entity_service = $entity_service;
 
-	}
+    }
 
-	/**
-	 * Get the property value.
-	 *
-	 * There is no filter for entities or posts, the returned data here can
-	 * be used for `relations` and `references` according to the client.
-	 *
-	 * @since 3.15.0
-	 *
-	 * @param int $post_id The {@link WP_Post}'s id.
-	 *
-	 * @return array|string|null A single string, or an array of values or null
-	 *                           if the property isn't recognized.
-	 */
-	public function get( $post_id ) {
+    /**
+     * Get the property value.
+     *
+     * There is no filter for entities or posts, the returned data here can
+     * be used for `relations` and `references` according to the client.
+     *
+     * @since 3.15.0
+     *
+     * @param int $post_id The {@link WP_Post}'s id.
+     *
+     * @return array|string|null A single string, or an array of values or null
+     *                           if the property isn't recognized.
+     */
+    public function get( $post_id ) {
 
-		// get related entities.
-		$related = wl_core_get_related_entity_ids( $post_id );
+        // get related entities.
+        $related = wl_core_get_related_entity_ids( $post_id );
 
-		// A reference to the entity service.
-		$entity_service = $this->entity_service;
+        // A reference to the entity service.
+        $entity_service = $this->entity_service;
 
-		// Map the related posts' ids to URIs.
-		return array_map(
-			function ( $item ) use ( $entity_service ) {
-				return $entity_service->get_uri( $item );
-			},
-			$related
-		);
-	}
+        // Map the related posts' ids to URIs.
+        return array_map(
+            function ( $item ) use ( $entity_service ) {
+                return $entity_service->get_uri( $item );
+            },
+            $related
+        );
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -37,7 +37,7 @@ 
 	 *
 	 * @param \Wordlift_Entity_Service $entity_service The {@link Wordlift_Entity_Service} instance.
 	 */
-	public function __construct( $entity_service ) {
+	public function __construct($entity_service) {
 
 		$this->entity_service = $entity_service;
 
@@ -56,18 +56,18 @@ 
 	 * @return array|string|null A single string, or an array of values or null
 	 *                           if the property isn't recognized.
 	 */
-	public function get( $post_id ) {
+	public function get($post_id) {
 
 		// get related entities.
-		$related = wl_core_get_related_entity_ids( $post_id );
+		$related = wl_core_get_related_entity_ids($post_id);
 
 		// A reference to the entity service.
 		$entity_service = $this->entity_service;
 
 		// Map the related posts' ids to URIs.
 		return array_map(
-			function ( $item ) use ( $entity_service ) {
-				return $entity_service->get_uri( $item );
+			function($item) use ($entity_service) {
+				return $entity_service->get_uri($item);
 			},
 			$related
 		);

src/includes/class-wordlift-tinymce-adapter.php

Indentation +55 added lines, -55 removed lines

@@ -14,77 +14,77 @@
  */
 class Wordlift_Tinymce_Adapter {
 
-	/**
-	 * The {@link Wordlift} plugin instance.
-	 *
-	 * @since  3.12.0
-	 * @access private
-	 * @var \Wordlift $plugin The {@link Wordlift} plugin instance.
-	 */
-	private $plugin;
+    /**
+     * The {@link Wordlift} plugin instance.
+     *
+     * @since  3.12.0
+     * @access private
+     * @var \Wordlift $plugin The {@link Wordlift} plugin instance.
+     */
+    private $plugin;
 
-	/**
-	 * Wordlift_Tinymce_Adapter constructor.
-	 *
-	 * @param \Wordlift $plugin The {@link Wordlift} plugin instance.
-	 */
-	public function __construct( $plugin ) {
+    /**
+     * Wordlift_Tinymce_Adapter constructor.
+     *
+     * @param \Wordlift $plugin The {@link Wordlift} plugin instance.
+     */
+    public function __construct( $plugin ) {
 
-		$this->plugin = $plugin;
+        $this->plugin = $plugin;
 
-	}
+    }
 
-	/**
-	 * Load the TinyMCE plugin. This method is called by the WP mce_external_plugins hook.
-	 *
-	 * @param array $plugins The existing plugins array.
-	 *
-	 * @return array The modified plugins array.
-	 * @since 3.12.0
-	 */
-	public function mce_external_plugins( $plugins ) {
+    /**
+     * Load the TinyMCE plugin. This method is called by the WP mce_external_plugins hook.
+     *
+     * @param array $plugins The existing plugins array.
+     *
+     * @return array The modified plugins array.
+     * @since 3.12.0
+     */
+    public function mce_external_plugins( $plugins ) {
 
-		/**
-		 * Bail out if you are on Media Library
-		 *
-		 * @since 3.27.1
-		 *
-		 * @see https://github.com/insideout10/wordlift-plugin/issues/1122
-		 */
-		if ( isset( get_current_screen()->base ) && get_current_screen()->base === 'upload' ) {
-			return $plugins;
-		}
+        /**
+         * Bail out if you are on Media Library
+         *
+         * @since 3.27.1
+         *
+         * @see https://github.com/insideout10/wordlift-plugin/issues/1122
+         */
+        if ( isset( get_current_screen()->base ) && get_current_screen()->base === 'upload' ) {
+            return $plugins;
+        }
 
-		/*
+        /*
 		 * Call the `wl_can_see_classification_box` filter to determine whether we can display the classification box.
 		 *
 		 * @since 3.20.3
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/914
 		 */
-		if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
-			return $plugins;
-		}
+        if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
+            return $plugins;
+        }
 
-		// Get WordLift's version as a cache killer.
-		$version = $this->plugin->get_version();
+        // Get WordLift's version as a cache killer.
+        $version = $this->plugin->get_version();
 
-		// User can edit?
-		$can_edit = current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' );
+        // User can edit?
+        $can_edit = current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' );
 
-		// If user can't edit or rich editing isn't enabled, bail out.
-		if ( ! $can_edit || ! get_user_option( 'rich_editing' ) ) {
-			return $plugins;
-		}
+        // If user can't edit or rich editing isn't enabled, bail out.
+        if ( ! $can_edit || ! get_user_option( 'rich_editing' ) ) {
+            return $plugins;
+        }
 
-		// Add our own JavaScript file to TinyMCE's extensions.
-		// DO NOT use the minified version, it'll yield errors with AngularJS.
-		$plugins['wordlift']      = plugin_dir_url( __DIR__ ) . 'js/wordlift-reloaded.js?ver=' . $version;
-		$plugins['wl_shortcodes'] = plugin_dir_url( __DIR__ ) . 'admin/js/wordlift_shortcode_tinymce_plugin.js?ver=' . $version;
-		$plugins['wl_tinymce']    = plugin_dir_url( __DIR__ ) . 'admin/js/1/tinymce.js?ver=' . $version;
-		$plugins['wl_tinymce_2']  = plugin_dir_url( __DIR__ ) . 'js/dist/tiny-mce.js?ver=' . $version;
+        // Add our own JavaScript file to TinyMCE's extensions.
+        // DO NOT use the minified version, it'll yield errors with AngularJS.
+        $plugins['wordlift']      = plugin_dir_url( __DIR__ ) . 'js/wordlift-reloaded.js?ver=' . $version;
+        $plugins['wl_shortcodes'] = plugin_dir_url( __DIR__ ) . 'admin/js/wordlift_shortcode_tinymce_plugin.js?ver=' . $version;
+        $plugins['wl_tinymce']    = plugin_dir_url( __DIR__ ) . 'admin/js/1/tinymce.js?ver=' . $version;
+        $plugins['wl_tinymce_2']  = plugin_dir_url( __DIR__ ) . 'js/dist/tiny-mce.js?ver=' . $version;
 
-		return $plugins;
-	}
+        return $plugins;
+    }
 
 }

Spacing +10 added lines, -10 removed lines

@@ -28,7 +28,7 @@ 
 	 *
 	 * @param \Wordlift $plugin The {@link Wordlift} plugin instance.
 	 */
-	public function __construct( $plugin ) {
+	public function __construct($plugin) {
 
 		$this->plugin = $plugin;
 
@@ -42,7 +42,7 @@ 
 	 * @return array The modified plugins array.
 	 * @since 3.12.0
 	 */
-	public function mce_external_plugins( $plugins ) {
+	public function mce_external_plugins($plugins) {
 
 		/**
 		 * Bail out if you are on Media Library
@@ -51,7 +51,7 @@ 
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/1122
 		 */
-		if ( isset( get_current_screen()->base ) && get_current_screen()->base === 'upload' ) {
+		if (isset(get_current_screen()->base) && get_current_screen()->base === 'upload') {
 			return $plugins;
 		}
 
@@ -62,7 +62,7 @@ 
 		 *
 		 * @see https://github.com/insideout10/wordlift-plugin/issues/914
 		 */
-		if ( ! apply_filters( 'wl_can_see_classification_box', true ) ) {
+		if ( ! apply_filters('wl_can_see_classification_box', true)) {
 			return $plugins;
 		}
 
@@ -70,19 +70,19 @@ 
 		$version = $this->plugin->get_version();
 
 		// User can edit?
-		$can_edit = current_user_can( 'edit_posts' ) || current_user_can( 'edit_pages' );
+		$can_edit = current_user_can('edit_posts') || current_user_can('edit_pages');
 
 		// If user can't edit or rich editing isn't enabled, bail out.
-		if ( ! $can_edit || ! get_user_option( 'rich_editing' ) ) {
+		if ( ! $can_edit || ! get_user_option('rich_editing')) {
 			return $plugins;
 		}
 
 		// Add our own JavaScript file to TinyMCE's extensions.
 		// DO NOT use the minified version, it'll yield errors with AngularJS.
-		$plugins['wordlift']      = plugin_dir_url( __DIR__ ) . 'js/wordlift-reloaded.js?ver=' . $version;
-		$plugins['wl_shortcodes'] = plugin_dir_url( __DIR__ ) . 'admin/js/wordlift_shortcode_tinymce_plugin.js?ver=' . $version;
-		$plugins['wl_tinymce']    = plugin_dir_url( __DIR__ ) . 'admin/js/1/tinymce.js?ver=' . $version;
-		$plugins['wl_tinymce_2']  = plugin_dir_url( __DIR__ ) . 'js/dist/tiny-mce.js?ver=' . $version;
+		$plugins['wordlift']      = plugin_dir_url(__DIR__).'js/wordlift-reloaded.js?ver='.$version;
+		$plugins['wl_shortcodes'] = plugin_dir_url(__DIR__).'admin/js/wordlift_shortcode_tinymce_plugin.js?ver='.$version;
+		$plugins['wl_tinymce']    = plugin_dir_url(__DIR__).'admin/js/1/tinymce.js?ver='.$version;
+		$plugins['wl_tinymce_2']  = plugin_dir_url(__DIR__).'js/dist/tiny-mce.js?ver='.$version;
 
 		return $plugins;
 	}