insideout10 / wordlift-plugin

src/classes/jsonld/class-jsonld-article-wrapper.php

Spacing +33 added lines, -33 removed lines

@@ -48,7 +48,7 @@ 
 	 * @param Wordlift_Post_To_Jsonld_Converter $post_to_jsonld_converter
 	 * @param $cached_postid_to_jsonld_converter
 	 */
-	public function __construct( $post_to_jsonld_converter, $cached_postid_to_jsonld_converter ) {
+	public function __construct($post_to_jsonld_converter, $cached_postid_to_jsonld_converter) {
 
 		$this->post_to_jsonld_converter = $post_to_jsonld_converter->new_instance_with_filters_disabled();
 
@@ -67,16 +67,16 @@ 
 		$this->entity_uri_service = \Wordlift_Entity_Uri_Service::get_instance();
 	}
 
-	public function after_get_jsonld( $jsonld, $post_id, $context ) {
+	public function after_get_jsonld($jsonld, $post_id, $context) {
 
 		// Invalid data structure
-		if ( ! is_array( $jsonld ) || ! isset( $jsonld[0] ) || ! is_array( $jsonld[0] ) ) {
+		if ( ! is_array($jsonld) || ! isset($jsonld[0]) || ! is_array($jsonld[0])) {
 			return $jsonld;
 		}
 
-		if ( Jsonld_Context_Enum::PAGE !== $context
+		if (Jsonld_Context_Enum::PAGE !== $context
 			 // Returns true for "1", "true", "on" and "yes". Returns false otherwise.
-			 && ! filter_input( INPUT_GET, 'article_wrapper', FILTER_VALIDATE_BOOLEAN ) ) {
+			 && ! filter_input(INPUT_GET, 'article_wrapper', FILTER_VALIDATE_BOOLEAN)) {
 			return $jsonld;
 		}
 
@@ -85,7 +85,7 @@ 
 
 		// Don't wrap in article if the json-ld is already about an Article (or its descendants). `@type` must be
 		// in the schema.org context, i.e. `Article`, not `http://schema.org/Article`.
-		if ( ! isset( $post_jsonld['@id'] ) || ! isset( $post_jsonld['@type'] ) || $this->is_article( $post_jsonld['@type'] ) ) {
+		if ( ! isset($post_jsonld['@id']) || ! isset($post_jsonld['@type']) || $this->is_article($post_jsonld['@type'])) {
 			return $jsonld;
 		}
 
@@ -93,23 +93,23 @@ 
 		$reference_infos = array();
 
 		// Convert the post as Article.
-		$article_jsonld = $this->post_to_jsonld_converter->convert( $post_id, $references, $reference_infos, new Relations() );
+		$article_jsonld = $this->post_to_jsonld_converter->convert($post_id, $references, $reference_infos, new Relations());
 
-		$article_jsonld['@id'] = $post_jsonld['@id'] . '#article';
+		$article_jsonld['@id'] = $post_jsonld['@id'].'#article';
 		// Reset the type, since by default the type assigned via the Entity Type taxonomy is used.
 		$article_jsonld['@type'] = 'Article';
-		$article_jsonld['about'] = array( '@id' => $post_jsonld['@id'] );
+		$article_jsonld['about'] = array('@id' => $post_jsonld['@id']);
 
 		// Copy over the URLs.
-		if ( isset( $post_jsonld['url'] ) ) {
+		if (isset($post_jsonld['url'])) {
 			$article_jsonld['url'] = $post_jsonld['url'];
 		}
 
-		array_unshift( $jsonld, $article_jsonld );
+		array_unshift($jsonld, $article_jsonld);
 
 		// Only add authors if the key is present.
-		if ( isset( $article_jsonld['author'] ) ) {
-			$this->add_author_data_if_missing( $jsonld, $article_jsonld['author'] );
+		if (isset($article_jsonld['author'])) {
+			$this->add_author_data_if_missing($jsonld, $article_jsonld['author']);
 		}
 
 		return $jsonld;
@@ -121,16 +121,16 @@ 
 	 *
 	 * @return void
 	 */
-	private function add_author_data_if_missing( &$jsonld, $author ) {
+	private function add_author_data_if_missing(&$jsonld, $author) {
 
 		// For each author when if the `@id` is present and if not, add the author data.
-		$entities_ids = $this->get_list_of_entities_ids( $author );
-		foreach ( $entities_ids as $entity_id ) {
-			if ( $this->is_author_entity_present_in_graph( $jsonld, $entity_id ) ) {
+		$entities_ids = $this->get_list_of_entities_ids($author);
+		foreach ($entities_ids as $entity_id) {
+			if ($this->is_author_entity_present_in_graph($jsonld, $entity_id)) {
 				continue;
 			}
 
-			$jsonld[] = $this->get_author_data_by_entity_id( $entity_id );
+			$jsonld[] = $this->get_author_data_by_entity_id($entity_id);
 		}
 
 	}
@@ -143,38 +143,38 @@ 
 	 *
 	 * @return array An array of `@id`s. Empty if not found.
 	 */
-	private function get_list_of_entities_ids( $items ) {
+	private function get_list_of_entities_ids($items) {
 		// If this is not an array, then what are we doing here? we return an empty array.
-		if ( ! is_array( $items ) || empty( $items ) ) {
+		if ( ! is_array($items) || empty($items)) {
 			return array();
 		}
 
 		// If this is a keyed array and the `@id` is set, then we return that single `@id`.
-		if ( isset( $items['@id'] ) ) {
-			return array( $items['@id'] );
+		if (isset($items['@id'])) {
+			return array($items['@id']);
 		}
 
 		// Poor way to check whether this is an indexed array.
-		if ( ! isset( $items[0] ) ) {
+		if ( ! isset($items[0])) {
 			return array();
 		}
 
 		// If this is an indexed array then we return the `@id` for all the items in the array.
-		return array_filter( array_column( $items, '@id' ) );
+		return array_filter(array_column($items, '@id'));
 	}
 
-	private function is_article( $schema_types ) {
+	private function is_article($schema_types) {
 
-		$array_intersect = array_intersect( self::$article_types, (array) $schema_types );
+		$array_intersect = array_intersect(self::$article_types, (array) $schema_types);
 
-		return ! empty( $array_intersect );
+		return ! empty($array_intersect);
 	}
 
-	private function get_author_data_by_entity_id( $author_linked_entity_id ) {
+	private function get_author_data_by_entity_id($author_linked_entity_id) {
 
-		$author_entity_post = $this->entity_uri_service->get_entity( $author_linked_entity_id );
+		$author_entity_post = $this->entity_uri_service->get_entity($author_linked_entity_id);
 
-		if ( ! $author_entity_post instanceof \WP_Post ) {
+		if ( ! $author_entity_post instanceof \WP_Post) {
 			return false;
 		}
 
@@ -190,10 +190,10 @@ 
 
 	}
 
-	private function is_author_entity_present_in_graph( $jsonld, $author_entity_id ) {
+	private function is_author_entity_present_in_graph($jsonld, $author_entity_id) {
 
-		foreach ( $jsonld as $item ) {
-			if ( $item && array_key_exists( '@id', $item ) && $item['@id'] === $author_entity_id ) {
+		foreach ($jsonld as $item) {
+			if ($item && array_key_exists('@id', $item) && $item['@id'] === $author_entity_id) {
 				return true;
 			}
 		}

src/modules/jsonld-author-filter/load.php

Spacing +7 added lines, -7 removed lines

@@ -20,21 +20,21 @@ 
  *
  * @see https://www.geeklab.info/2010/04/wordpress-pass-variables-by-reference-with-apply_filter/
  */
-function _wl_jsonld_author__author_filter( $args_arr, $post_id ) {
+function _wl_jsonld_author__author_filter($args_arr, $post_id) {
 
 	$references = $args_arr['references'];
 
 	$coauthor_plugin_path = 'co-authors-plus/co-authors-plus.php';
 
 	// If the co-authors plugin is active.
-	if ( ! is_plugin_active( $coauthor_plugin_path ) || ! function_exists( 'get_coauthors' ) ) {
+	if ( ! is_plugin_active($coauthor_plugin_path) || ! function_exists('get_coauthors')) {
 		return $args_arr;
 	}
 
-	$coauthors = get_coauthors( $post_id );
+	$coauthors = get_coauthors($post_id);
 
 	// And we have multiple authors on a post.
-	if ( empty( $coauthors ) ) {
+	if (empty($coauthors)) {
 		return $args_arr;
 	}
 
@@ -43,8 +43,8 @@ 
 
 	// Build array of authors.
 	$wordlift_post_to_jsonld_converter = Wordlift_Post_To_Jsonld_Converter::get_instance();
-	foreach ( $coauthors as $coauthor ) {
-		$author[] = $wordlift_post_to_jsonld_converter->get_author( $coauthor->ID, $references );
+	foreach ($coauthors as $coauthor) {
+		$author[] = $wordlift_post_to_jsonld_converter->get_author($coauthor->ID, $references);
 	}
 
 	return array(
@@ -54,4 +54,4 @@ 
 }
 
 // Add the filter
-add_filter( 'wl_jsonld_author', '_wl_jsonld_author__author_filter', 10, 2 );
+add_filter('wl_jsonld_author', '_wl_jsonld_author__author_filter', 10, 2);