insideout10 / wordlift-plugin

src/wordlift/vocabulary/api/class-tag-rest-endpoint.php

Indentation +115 added lines, -115 removed lines

@@ -15,121 +15,121 @@
  */
 class Tag_Rest_Endpoint {
 
-	/**
-	 * @var Term_Data_Factory
-	 */
-	private $term_data_factory;
-
-	/**
-	 * Tag_Rest_Endpoint constructor.
-	 *
-	 * @param Term_Data_Factory $term_data_factory
-	 */
-	public function __construct( $term_data_factory ) {
-
-		$this->term_data_factory = $term_data_factory;
-
-	}
-
-
-	public function register_routes() {
-		$that = $this;
-		add_action( 'rest_api_init',
-			function () use ( $that ) {
-				register_rest_route(
-					Api_Config::REST_NAMESPACE,
-					'/tags',
-					array(
-						'methods'             => WP_REST_Server::CREATABLE,
-						'callback'            => array( $that, 'get_tags' ),
-						//@todo : review the permission level
-						'permission_callback' => function () {
-							return current_user_can( 'manage_options' );
-						},
-						'args'                => array(
-							'limit'  => array(
-								'validate_callback' => function ( $param, $request, $key ) {
-									return is_numeric( $param ) && $param;
-								},
-								'required'          => true,
-							),
-							'offset' => array(
-								'validate_callback' => function ( $param, $request, $key ) {
-									return is_numeric( $param );
-								},
-								'required'          => true,
-							),
-						),
-					)
-				);
-			} );
-
-
-	}
-
-
-	public function get_tags( $request ) {
-
-		$data   = $request->get_params();
-		$offset = (int) $data['offset'];
-		$limit  = (int) $data['limit'];
-		$tags = $this->get_terms_from_db( $limit, $offset );
-		$term_data_list = array();
-
-		foreach ( $tags as $tag ) {
-
-			if ( $this->is_tag_excluded_from_ui( $tag ) ) {
-				continue;
-			}
-
-			/**
-			 * @param $tag \WP_Term
-			 */
-			$term_data_instance = $this->term_data_factory->get_term_data($tag);
-			$term_data = $term_data_instance->get_data();
-			if ( $term_data['entities'] ) {
-				$term_data_list[] = $term_data;
-			}
-		}
-
-
-		return $term_data_list;
-	}
-
-	/**
-	 * @param \WP_Term $tag
-	 *
-	 * @return bool
-	 */
-	private function is_tag_excluded_from_ui( $tag ) {
-		return (int) get_term_meta( $tag->term_id, Entity_Rest_Endpoint::IGNORE_TAG_FROM_LISTING, true ) === 1;
-	}
-
-	/**
-	 * @param $limit
-	 * @param $offset
-	 *
-	 * @return int|\WP_Error|\WP_Term[]
-	 */
-	public function get_terms_from_db( $limit, $offset ) {
-
-
-		return Terms_Compat::get_terms(Terms_Compat::get_public_taxonomies(), array(
-			'hide_empty' => false,
-			'number'     => $limit,
-			'offset'     => $offset,
-			'meta_query' => array(
-				array(
-					'key'     => Analysis_Background_Service::ENTITIES_PRESENT_FOR_TERM,
-					'compare' => 'EXISTS'
-				)
-			),
-			'orderby'    => 'count',
-			'order'      => 'DESC',
-		));
-
-
-	}
+    /**
+     * @var Term_Data_Factory
+     */
+    private $term_data_factory;
+
+    /**
+     * Tag_Rest_Endpoint constructor.
+     *
+     * @param Term_Data_Factory $term_data_factory
+     */
+    public function __construct( $term_data_factory ) {
+
+        $this->term_data_factory = $term_data_factory;
+
+    }
+
+
+    public function register_routes() {
+        $that = $this;
+        add_action( 'rest_api_init',
+            function () use ( $that ) {
+                register_rest_route(
+                    Api_Config::REST_NAMESPACE,
+                    '/tags',
+                    array(
+                        'methods'             => WP_REST_Server::CREATABLE,
+                        'callback'            => array( $that, 'get_tags' ),
+                        //@todo : review the permission level
+                        'permission_callback' => function () {
+                            return current_user_can( 'manage_options' );
+                        },
+                        'args'                => array(
+                            'limit'  => array(
+                                'validate_callback' => function ( $param, $request, $key ) {
+                                    return is_numeric( $param ) && $param;
+                                },
+                                'required'          => true,
+                            ),
+                            'offset' => array(
+                                'validate_callback' => function ( $param, $request, $key ) {
+                                    return is_numeric( $param );
+                                },
+                                'required'          => true,
+                            ),
+                        ),
+                    )
+                );
+            } );
+
+
+    }
+
+
+    public function get_tags( $request ) {
+
+        $data   = $request->get_params();
+        $offset = (int) $data['offset'];
+        $limit  = (int) $data['limit'];
+        $tags = $this->get_terms_from_db( $limit, $offset );
+        $term_data_list = array();
+
+        foreach ( $tags as $tag ) {
+
+            if ( $this->is_tag_excluded_from_ui( $tag ) ) {
+                continue;
+            }
+
+            /**
+             * @param $tag \WP_Term
+             */
+            $term_data_instance = $this->term_data_factory->get_term_data($tag);
+            $term_data = $term_data_instance->get_data();
+            if ( $term_data['entities'] ) {
+                $term_data_list[] = $term_data;
+            }
+        }
+
+
+        return $term_data_list;
+    }
+
+    /**
+     * @param \WP_Term $tag
+     *
+     * @return bool
+     */
+    private function is_tag_excluded_from_ui( $tag ) {
+        return (int) get_term_meta( $tag->term_id, Entity_Rest_Endpoint::IGNORE_TAG_FROM_LISTING, true ) === 1;
+    }
+
+    /**
+     * @param $limit
+     * @param $offset
+     *
+     * @return int|\WP_Error|\WP_Term[]
+     */
+    public function get_terms_from_db( $limit, $offset ) {
+
+
+        return Terms_Compat::get_terms(Terms_Compat::get_public_taxonomies(), array(
+            'hide_empty' => false,
+            'number'     => $limit,
+            'offset'     => $offset,
+            'meta_query' => array(
+                array(
+                    'key'     => Analysis_Background_Service::ENTITIES_PRESENT_FOR_TERM,
+                    'compare' => 'EXISTS'
+                )
+            ),
+            'orderby'    => 'count',
+            'order'      => 'DESC',
+        ));
+
+
+    }
 
 
 }

Spacing +18 added lines, -18 removed lines

@@ -25,7 +25,7 @@ 
 	 *
 	 * @param Term_Data_Factory $term_data_factory
 	 */
-	public function __construct( $term_data_factory ) {
+	public function __construct($term_data_factory) {
 
 		$this->term_data_factory = $term_data_factory;
 
@@ -34,28 +34,28 @@ 
 
 	public function register_routes() {
 		$that = $this;
-		add_action( 'rest_api_init',
-			function () use ( $that ) {
+		add_action('rest_api_init',
+			function() use ($that) {
 				register_rest_route(
 					Api_Config::REST_NAMESPACE,
 					'/tags',
 					array(
 						'methods'             => WP_REST_Server::CREATABLE,
-						'callback'            => array( $that, 'get_tags' ),
+						'callback'            => array($that, 'get_tags'),
 						//@todo : review the permission level
-						'permission_callback' => function () {
-							return current_user_can( 'manage_options' );
+						'permission_callback' => function() {
+							return current_user_can('manage_options');
 						},
 						'args'                => array(
 							'limit'  => array(
-								'validate_callback' => function ( $param, $request, $key ) {
-									return is_numeric( $param ) && $param;
+								'validate_callback' => function($param, $request, $key) {
+									return is_numeric($param) && $param;
 								},
 								'required'          => true,
 							),
 							'offset' => array(
-								'validate_callback' => function ( $param, $request, $key ) {
-									return is_numeric( $param );
+								'validate_callback' => function($param, $request, $key) {
+									return is_numeric($param);
 								},
 								'required'          => true,
 							),
@@ -68,17 +68,17 @@ 
 	}
 
 
-	public function get_tags( $request ) {
+	public function get_tags($request) {
 
 		$data   = $request->get_params();
 		$offset = (int) $data['offset'];
 		$limit  = (int) $data['limit'];
-		$tags = $this->get_terms_from_db( $limit, $offset );
+		$tags = $this->get_terms_from_db($limit, $offset);
 		$term_data_list = array();
 
-		foreach ( $tags as $tag ) {
+		foreach ($tags as $tag) {
 
-			if ( $this->is_tag_excluded_from_ui( $tag ) ) {
+			if ($this->is_tag_excluded_from_ui($tag)) {
 				continue;
 			}
 
@@ -87,7 +87,7 @@ 
 			 */
 			$term_data_instance = $this->term_data_factory->get_term_data($tag);
 			$term_data = $term_data_instance->get_data();
-			if ( $term_data['entities'] ) {
+			if ($term_data['entities']) {
 				$term_data_list[] = $term_data;
 			}
 		}
@@ -101,8 +101,8 @@ 
 	 *
 	 * @return bool
 	 */
-	private function is_tag_excluded_from_ui( $tag ) {
-		return (int) get_term_meta( $tag->term_id, Entity_Rest_Endpoint::IGNORE_TAG_FROM_LISTING, true ) === 1;
+	private function is_tag_excluded_from_ui($tag) {
+		return (int) get_term_meta($tag->term_id, Entity_Rest_Endpoint::IGNORE_TAG_FROM_LISTING, true) === 1;
 	}
 
 	/**
@@ -111,7 +111,7 @@ 
 	 *
 	 * @return int|\WP_Error|\WP_Term[]
 	 */
-	public function get_terms_from_db( $limit, $offset ) {
+	public function get_terms_from_db($limit, $offset) {
 
 
 		return Terms_Compat::get_terms(Terms_Compat::get_public_taxonomies(), array(

src/wordlift/vocabulary/jsonld/class-jsonld-utils.php

Indentation +9 added lines, -9 removed lines

@@ -12,16 +12,16 @@
 
 class Jsonld_Utils {
 
-	/**
-	 * @param $term_id
-	 *
-	 * @return array|array[]|mixed
-	 */
-	public static function get_matched_entities_for_term( $term_id ) {
+    /**
+     * @param $term_id
+     *
+     * @return array|array[]|mixed
+     */
+    public static function get_matched_entities_for_term( $term_id ) {
 
-		$entity = Entity_List_Factory::get_instance( $term_id );
+        $entity = Entity_List_Factory::get_instance( $term_id );
 
-		return $entity->get_jsonld_data();
-	}
+        return $entity->get_jsonld_data();
+    }
 
 }
\ No newline at end of file

Spacing +2 added lines, -2 removed lines

@@ -17,9 +17,9 @@
 	 *
 	 * @return array|array[]|mixed
 	 */
-	public static function get_matched_entities_for_term( $term_id ) {
+	public static function get_matched_entities_for_term($term_id) {
 
-		$entity = Entity_List_Factory::get_instance( $term_id );
+		$entity = Entity_List_Factory::get_instance($term_id);
 
 		return $entity->get_jsonld_data();
 	}

src/wordlift/vocabulary/jsonld/class-post-jsonld.php

Indentation +73 added lines, -73 removed lines

@@ -14,106 +14,106 @@
 
 
 
-	public function enhance_post_jsonld() {
-		add_filter( 'wl_post_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 2 );
-		add_filter( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 11, 2 );
-	}
+    public function enhance_post_jsonld() {
+        add_filter( 'wl_post_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 2 );
+        add_filter( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 11, 2 );
+    }
 
-	public function wl_post_jsonld_array( $arr, $post_id ) {
+    public function wl_post_jsonld_array( $arr, $post_id ) {
 
-		$jsonld     = $arr['jsonld'];
-		$references = $arr['references'];
+        $jsonld     = $arr['jsonld'];
+        $references = $arr['references'];
 
-		$this->add_mentions( $post_id, $jsonld, $references );
+        $this->add_mentions( $post_id, $jsonld, $references );
 
-		return array(
-			'jsonld'     => $jsonld,
-			'references' => $references
-		);
+        return array(
+            'jsonld'     => $jsonld,
+            'references' => $references
+        );
 
-	}
+    }
 
-	public function add_mentions( $post_id, &$jsonld, &$references ) {
+    public function add_mentions( $post_id, &$jsonld, &$references ) {
 
-		$taxonomies = Terms_Compat::get_public_taxonomies();
-		$terms      = array();
+        $taxonomies = Terms_Compat::get_public_taxonomies();
+        $terms      = array();
 
-		foreach ( $taxonomies as $taxonomy ) {
-			$taxonomy_terms = get_the_terms( $post_id, $taxonomy );
-			if ( ! $taxonomy_terms ) {
-				continue;
-			}
-			$terms = array_merge( $taxonomy_terms, $terms );
-		}
+        foreach ( $taxonomies as $taxonomy ) {
+            $taxonomy_terms = get_the_terms( $post_id, $taxonomy );
+            if ( ! $taxonomy_terms ) {
+                continue;
+            }
+            $terms = array_merge( $taxonomy_terms, $terms );
+        }
 
-		if ( ! $terms ) {
-			return;
-		}
+        if ( ! $terms ) {
+            return;
+        }
 
-		if ( ! array_key_exists( 'mentions', $jsonld ) ) {
-			$jsonld['mentions'] = array();
-		}
+        if ( ! array_key_exists( 'mentions', $jsonld ) ) {
+            $jsonld['mentions'] = array();
+        }
 
-		foreach ( $terms as $term ) {
+        foreach ( $terms as $term ) {
 
-			$is_matched = intval( get_term_meta( $term->term_id, Entity_Rest_Endpoint::IGNORE_TAG_FROM_LISTING, true ) ) === 1;
+            $is_matched = intval( get_term_meta( $term->term_id, Entity_Rest_Endpoint::IGNORE_TAG_FROM_LISTING, true ) ) === 1;
 
-			if ( ! $is_matched ) {
-				continue;
-			}
+            if ( ! $is_matched ) {
+                continue;
+            }
 
-			$entities = Jsonld_Utils::get_matched_entities_for_term( $term->term_id );
+            $entities = Jsonld_Utils::get_matched_entities_for_term( $term->term_id );
 
-			if ( count( $entities ) === 0 ) {
-				continue;
-			}
+            if ( count( $entities ) === 0 ) {
+                continue;
+            }
 
-			$jsonld['mentions'] = array_merge( $jsonld['mentions'], self::add_additional_attrs( $term, $entities ) );
-		}
+            $jsonld['mentions'] = array_merge( $jsonld['mentions'], self::add_additional_attrs( $term, $entities ) );
+        }
 
-	}
+    }
 
-	/**
-	 * @param $term \WP_Term
-	 * @param $entities
-	 *
-	 * @return array
-	 */
-	public static function add_additional_attrs( $term, $entities ) {
+    /**
+     * @param $term \WP_Term
+     * @param $entities
+     *
+     * @return array
+     */
+    public static function add_additional_attrs( $term, $entities ) {
 
-		return array_map( function ( $entity ) use ( $term ) {
-			$entity['@id'] = get_term_link( $term->term_id ) . '#id';
-			if ( ! empty( $term->description ) ) {
-				$entity['description'] = $term->description;
-			}
+        return array_map( function ( $entity ) use ( $term ) {
+            $entity['@id'] = get_term_link( $term->term_id ) . '#id';
+            if ( ! empty( $term->description ) ) {
+                $entity['description'] = $term->description;
+            }
 
-			return $entity;
+            return $entity;
 
-		}, $entities );
+        }, $entities );
 
-	}
+    }
 
-	public function wl_after_get_jsonld( $jsonld, $post_id ) {
+    public function wl_after_get_jsonld( $jsonld, $post_id ) {
 
-		if ( ! is_array( $jsonld ) || count( $jsonld ) === 0 ) {
-			return $jsonld;
-		}
+        if ( ! is_array( $jsonld ) || count( $jsonld ) === 0 ) {
+            return $jsonld;
+        }
 
-		foreach ( $jsonld as $key => $value ) {
-			if ( $value['@type'] === 'Article' && isset( $value['image'] ) ) {
-				$image = $value['image'];
-			}
-			if ( $value['@type'] === 'Recipe' && ! isset( $value['image'] ) ) {
-				$index = $key;
-			}
-		}
+        foreach ( $jsonld as $key => $value ) {
+            if ( $value['@type'] === 'Article' && isset( $value['image'] ) ) {
+                $image = $value['image'];
+            }
+            if ( $value['@type'] === 'Recipe' && ! isset( $value['image'] ) ) {
+                $index = $key;
+            }
+        }
 
-		if ( isset( $index ) && ! empty( $image ) ) {
-			$jsonld[ $index ]['image'] = $image;
-		}
+        if ( isset( $index ) && ! empty( $image ) ) {
+            $jsonld[ $index ]['image'] = $image;
+        }
 
-		return $jsonld;
+        return $jsonld;
 
-	}
+    }
 
 }

Spacing +29 added lines, -29 removed lines

@@ -15,16 +15,16 @@ 
 
 
 	public function enhance_post_jsonld() {
-		add_filter( 'wl_post_jsonld_array', array( $this, 'wl_post_jsonld_array' ), 11, 2 );
-		add_filter( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 11, 2 );
+		add_filter('wl_post_jsonld_array', array($this, 'wl_post_jsonld_array'), 11, 2);
+		add_filter('wl_after_get_jsonld', array($this, 'wl_after_get_jsonld'), 11, 2);
 	}
 
-	public function wl_post_jsonld_array( $arr, $post_id ) {
+	public function wl_post_jsonld_array($arr, $post_id) {
 
 		$jsonld     = $arr['jsonld'];
 		$references = $arr['references'];
 
-		$this->add_mentions( $post_id, $jsonld, $references );
+		$this->add_mentions($post_id, $jsonld, $references);
 
 		return array(
 			'jsonld'     => $jsonld,
@@ -33,42 +33,42 @@ 
 
 	}
 
-	public function add_mentions( $post_id, &$jsonld, &$references ) {
+	public function add_mentions($post_id, &$jsonld, &$references) {
 
 		$taxonomies = Terms_Compat::get_public_taxonomies();
 		$terms      = array();
 
-		foreach ( $taxonomies as $taxonomy ) {
-			$taxonomy_terms = get_the_terms( $post_id, $taxonomy );
-			if ( ! $taxonomy_terms ) {
+		foreach ($taxonomies as $taxonomy) {
+			$taxonomy_terms = get_the_terms($post_id, $taxonomy);
+			if ( ! $taxonomy_terms) {
 				continue;
 			}
-			$terms = array_merge( $taxonomy_terms, $terms );
+			$terms = array_merge($taxonomy_terms, $terms);
 		}
 
-		if ( ! $terms ) {
+		if ( ! $terms) {
 			return;
 		}
 
-		if ( ! array_key_exists( 'mentions', $jsonld ) ) {
+		if ( ! array_key_exists('mentions', $jsonld)) {
 			$jsonld['mentions'] = array();
 		}
 
-		foreach ( $terms as $term ) {
+		foreach ($terms as $term) {
 
-			$is_matched = intval( get_term_meta( $term->term_id, Entity_Rest_Endpoint::IGNORE_TAG_FROM_LISTING, true ) ) === 1;
+			$is_matched = intval(get_term_meta($term->term_id, Entity_Rest_Endpoint::IGNORE_TAG_FROM_LISTING, true)) === 1;
 
-			if ( ! $is_matched ) {
+			if ( ! $is_matched) {
 				continue;
 			}
 
-			$entities = Jsonld_Utils::get_matched_entities_for_term( $term->term_id );
+			$entities = Jsonld_Utils::get_matched_entities_for_term($term->term_id);
 
-			if ( count( $entities ) === 0 ) {
+			if (count($entities) === 0) {
 				continue;
 			}
 
-			$jsonld['mentions'] = array_merge( $jsonld['mentions'], self::add_additional_attrs( $term, $entities ) );
+			$jsonld['mentions'] = array_merge($jsonld['mentions'], self::add_additional_attrs($term, $entities));
 		}
 
 	}
@@ -79,37 +79,37 @@ 
 	 *
 	 * @return array
 	 */
-	public static function add_additional_attrs( $term, $entities ) {
+	public static function add_additional_attrs($term, $entities) {
 
-		return array_map( function ( $entity ) use ( $term ) {
-			$entity['@id'] = get_term_link( $term->term_id ) . '#id';
-			if ( ! empty( $term->description ) ) {
+		return array_map(function($entity) use ($term) {
+			$entity['@id'] = get_term_link($term->term_id).'#id';
+			if ( ! empty($term->description)) {
 				$entity['description'] = $term->description;
 			}
 
 			return $entity;
 
-		}, $entities );
+		}, $entities);
 
 	}
 
-	public function wl_after_get_jsonld( $jsonld, $post_id ) {
+	public function wl_after_get_jsonld($jsonld, $post_id) {
 
-		if ( ! is_array( $jsonld ) || count( $jsonld ) === 0 ) {
+		if ( ! is_array($jsonld) || count($jsonld) === 0) {
 			return $jsonld;
 		}
 
-		foreach ( $jsonld as $key => $value ) {
-			if ( $value['@type'] === 'Article' && isset( $value['image'] ) ) {
+		foreach ($jsonld as $key => $value) {
+			if ($value['@type'] === 'Article' && isset($value['image'])) {
 				$image = $value['image'];
 			}
-			if ( $value['@type'] === 'Recipe' && ! isset( $value['image'] ) ) {
+			if ($value['@type'] === 'Recipe' && ! isset($value['image'])) {
 				$index = $key;
 			}
 		}
 
-		if ( isset( $index ) && ! empty( $image ) ) {
-			$jsonld[ $index ]['image'] = $image;
+		if (isset($index) && ! empty($image)) {
+			$jsonld[$index]['image'] = $image;
 		}
 
 		return $jsonld;

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

Spacing +25 added lines, -25 removed lines

@@ -47,21 +47,21 @@ 
 	 * @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();
 
-		add_filter( 'wl_after_get_jsonld', array( $this, 'after_get_jsonld' ), PHP_INT_MAX - 100, 3 );
+		add_filter('wl_after_get_jsonld', array($this, 'after_get_jsonld'), PHP_INT_MAX - 100, 3);
 
 		$this->cached_postid_to_jsonld_converter = $cached_postid_to_jsonld_converter;
 
 		$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) {
 
-		if ( Jsonld_Context_Enum::PAGE !== $context || ! is_array( $jsonld ) || ! isset( $jsonld[0] )
-		     || ! is_array( $jsonld[0] ) ) {
+		if (Jsonld_Context_Enum::PAGE !== $context || ! is_array($jsonld) || ! isset($jsonld[0])
+		     || ! is_array($jsonld[0])) {
 			return $jsonld;
 		}
 
@@ -70,74 +70,74 @@ 
 
 		// 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;
 		}
 
 		// Convert the post as Article.
-		$article_jsonld = $this->post_to_jsonld_converter->convert( $post_id );
+		$article_jsonld = $this->post_to_jsonld_converter->convert($post_id);
 
-		$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);
 
-		$author_jsonld = $this->get_author_linked_entity( $article_jsonld );
+		$author_jsonld = $this->get_author_linked_entity($article_jsonld);
 
 		/**
 		 * The author entities can be present in graph for some entity types
 		 * for Person and Organization, so check before we add it to graph.
 		 * reference : https://schema.org/author
 		 */
-		if ( $author_jsonld && ! $this->is_author_entity_present_in_graph( $jsonld, $article_jsonld['author']['@id'] ) ) {
+		if ($author_jsonld && ! $this->is_author_entity_present_in_graph($jsonld, $article_jsonld['author']['@id'])) {
 			$jsonld[] = $author_jsonld;
 		}
 
 		return $jsonld;
 	}
 
-	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_linked_entity( $article_jsonld ) {
-		if ( ! array_key_exists( 'author', $article_jsonld ) ) {
+	private function get_author_linked_entity($article_jsonld) {
+		if ( ! array_key_exists('author', $article_jsonld)) {
 			return false;
 		}
 
 		$author = $article_jsonld['author'];
 
-		if ( count( array_keys( $author ) ) !== 1 || ! array_key_exists( '@id', $author ) ) {
+		if (count(array_keys($author)) !== 1 || ! array_key_exists('@id', $author)) {
 			return false;
 		}
 
 		$author_linked_entity_id = $author['@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;
 		}
 
-		return $this->cached_postid_to_jsonld_converter->convert( $author_entity_post->ID );
+		return $this->cached_postid_to_jsonld_converter->convert($author_entity_post->ID);
 
 	}
 
-	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;
 			}
 		}

Indentation +128 added lines, -128 removed lines

@@ -6,143 +6,143 @@
 
 class Jsonld_Article_Wrapper {
 
-	public static $article_types = array(
-		'Article',
-		'AdvertiserContentArticle',
-		'NewsArticle',
-		'AnalysisNewsArticle',
-		'AskPublicNewsArticle',
-		'BackgroundNewsArticle',
-		'OpinionNewsArticle',
-		'ReportageNewsArticle',
-		'ReviewNewsArticle',
-		'Report',
-		'SatiricalArticle',
-		'ScholarlyArticle',
-		'MedicalScholarlyArticle',
-		'SocialMediaPosting',
-		'BlogPosting',
-		'LiveBlogPosting',
-		'DiscussionForumPosting',
-		'TechArticle',
-		'APIReference'
-	);
-
-	/**
-	 * @var Wordlift_Post_To_Jsonld_Converter
-	 */
-	private $post_to_jsonld_converter;
-	/**
-	 * @var \Wordlift_Cached_Post_Converter
-	 */
-	private $cached_postid_to_jsonld_converter;
-	/**
-	 * @var \Wordlift_Entity_Uri_Service
-	 */
-	private $entity_uri_service;
-
-	/**
-	 * Jsonld_Article_Wrapper constructor.
-	 *
-	 * @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 ) {
-
-		$this->post_to_jsonld_converter = $post_to_jsonld_converter->new_instance_with_filters_disabled();
-
-		add_filter( 'wl_after_get_jsonld', array( $this, 'after_get_jsonld' ), PHP_INT_MAX - 100, 3 );
-
-		$this->cached_postid_to_jsonld_converter = $cached_postid_to_jsonld_converter;
-
-		$this->entity_uri_service = \Wordlift_Entity_Uri_Service::get_instance();
-	}
-
-	public function after_get_jsonld( $jsonld, $post_id, $context ) {
-
-		if ( Jsonld_Context_Enum::PAGE !== $context || ! is_array( $jsonld ) || ! isset( $jsonld[0] )
-		     || ! is_array( $jsonld[0] ) ) {
-			return $jsonld;
-		}
-
-		// Copy the 1st array element
-		$post_jsonld = $jsonld[0];
-
-		// 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'] ) ) {
-			return $jsonld;
-		}
-
-		// Convert the post as Article.
-		$article_jsonld = $this->post_to_jsonld_converter->convert( $post_id );
-
-		$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'] );
-
-
-		// Copy over the URLs.
-		if ( isset( $post_jsonld['url'] ) ) {
-			$article_jsonld['url'] = $post_jsonld['url'];
-		}
-
-		array_unshift( $jsonld, $article_jsonld );
-
-		$author_jsonld = $this->get_author_linked_entity( $article_jsonld );
-
-		/**
-		 * The author entities can be present in graph for some entity types
-		 * for Person and Organization, so check before we add it to graph.
-		 * reference : https://schema.org/author
-		 */
-		if ( $author_jsonld && ! $this->is_author_entity_present_in_graph( $jsonld, $article_jsonld['author']['@id'] ) ) {
-			$jsonld[] = $author_jsonld;
-		}
-
-		return $jsonld;
-	}
-
-	private function is_article( $schema_types ) {
-
-		$array_intersect = array_intersect( self::$article_types, ( array ) $schema_types );
-
-		return ! empty( $array_intersect );
-	}
-
-	private function get_author_linked_entity( $article_jsonld ) {
-		if ( ! array_key_exists( 'author', $article_jsonld ) ) {
-			return false;
-		}
+    public static $article_types = array(
+        'Article',
+        'AdvertiserContentArticle',
+        'NewsArticle',
+        'AnalysisNewsArticle',
+        'AskPublicNewsArticle',
+        'BackgroundNewsArticle',
+        'OpinionNewsArticle',
+        'ReportageNewsArticle',
+        'ReviewNewsArticle',
+        'Report',
+        'SatiricalArticle',
+        'ScholarlyArticle',
+        'MedicalScholarlyArticle',
+        'SocialMediaPosting',
+        'BlogPosting',
+        'LiveBlogPosting',
+        'DiscussionForumPosting',
+        'TechArticle',
+        'APIReference'
+    );
+
+    /**
+     * @var Wordlift_Post_To_Jsonld_Converter
+     */
+    private $post_to_jsonld_converter;
+    /**
+     * @var \Wordlift_Cached_Post_Converter
+     */
+    private $cached_postid_to_jsonld_converter;
+    /**
+     * @var \Wordlift_Entity_Uri_Service
+     */
+    private $entity_uri_service;
+
+    /**
+     * Jsonld_Article_Wrapper constructor.
+     *
+     * @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 ) {
+
+        $this->post_to_jsonld_converter = $post_to_jsonld_converter->new_instance_with_filters_disabled();
+
+        add_filter( 'wl_after_get_jsonld', array( $this, 'after_get_jsonld' ), PHP_INT_MAX - 100, 3 );
+
+        $this->cached_postid_to_jsonld_converter = $cached_postid_to_jsonld_converter;
+
+        $this->entity_uri_service = \Wordlift_Entity_Uri_Service::get_instance();
+    }
+
+    public function after_get_jsonld( $jsonld, $post_id, $context ) {
+
+        if ( Jsonld_Context_Enum::PAGE !== $context || ! is_array( $jsonld ) || ! isset( $jsonld[0] )
+             || ! is_array( $jsonld[0] ) ) {
+            return $jsonld;
+        }
+
+        // Copy the 1st array element
+        $post_jsonld = $jsonld[0];
+
+        // 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'] ) ) {
+            return $jsonld;
+        }
+
+        // Convert the post as Article.
+        $article_jsonld = $this->post_to_jsonld_converter->convert( $post_id );
+
+        $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'] );
+
+
+        // Copy over the URLs.
+        if ( isset( $post_jsonld['url'] ) ) {
+            $article_jsonld['url'] = $post_jsonld['url'];
+        }
+
+        array_unshift( $jsonld, $article_jsonld );
+
+        $author_jsonld = $this->get_author_linked_entity( $article_jsonld );
+
+        /**
+         * The author entities can be present in graph for some entity types
+         * for Person and Organization, so check before we add it to graph.
+         * reference : https://schema.org/author
+         */
+        if ( $author_jsonld && ! $this->is_author_entity_present_in_graph( $jsonld, $article_jsonld['author']['@id'] ) ) {
+            $jsonld[] = $author_jsonld;
+        }
+
+        return $jsonld;
+    }
+
+    private function is_article( $schema_types ) {
+
+        $array_intersect = array_intersect( self::$article_types, ( array ) $schema_types );
+
+        return ! empty( $array_intersect );
+    }
+
+    private function get_author_linked_entity( $article_jsonld ) {
+        if ( ! array_key_exists( 'author', $article_jsonld ) ) {
+            return false;
+        }
 
-		$author = $article_jsonld['author'];
+        $author = $article_jsonld['author'];
 
-		if ( count( array_keys( $author ) ) !== 1 || ! array_key_exists( '@id', $author ) ) {
-			return false;
-		}
+        if ( count( array_keys( $author ) ) !== 1 || ! array_key_exists( '@id', $author ) ) {
+            return false;
+        }
 
-		$author_linked_entity_id = $author['@id'];
+        $author_linked_entity_id = $author['@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 ) {
-			return false;
-		}
+        if ( ! $author_entity_post instanceof \WP_Post ) {
+            return false;
+        }
 
-		return $this->cached_postid_to_jsonld_converter->convert( $author_entity_post->ID );
+        return $this->cached_postid_to_jsonld_converter->convert( $author_entity_post->ID );
 
-	}
+    }
 
-	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 ) {
-				return true;
-			}
-		}
+        foreach ( $jsonld as $item ) {
+            if ( $item && array_key_exists( '@id', $item ) && $item['@id'] === $author_entity_id ) {
+                return true;
+            }
+        }
 
-		return false;
-	}
+        return false;
+    }
 
 }
\ No newline at end of file

src/includes/intf-wordlift-post-converter.php

Indentation +12 added lines, -12 removed lines

@@ -18,17 +18,17 @@
  */
 interface Wordlift_Post_Converter {
 
-	/**
-	 * Convert the specified post id.
-	 *
-	 * @param int $post_id The post id.
-	 * @param array $references An array of posts referenced by the specified post.
-	 * @param array $references_infos
-	 *
-	 * @return mixed The conversion result.
-	 * @since 3.16.0 $references argument added.
-	 * @since 3.10.0
-	 */
-	public function convert( $post_id, &$references = array(), &$references_infos = array() );
+    /**
+     * Convert the specified post id.
+     *
+     * @param int $post_id The post id.
+     * @param array $references An array of posts referenced by the specified post.
+     * @param array $references_infos
+     *
+     * @return mixed The conversion result.
+     * @since 3.16.0 $references argument added.
+     * @since 3.10.0
+     */
+    public function convert( $post_id, &$references = array(), &$references_infos = array() );
 
 }

Spacing +1 added lines, -1 removed lines

@@ -29,6 +29,6 @@
 	 * @since 3.16.0 $references argument added.
 	 * @since 3.10.0
 	 */
-	public function convert( $post_id, &$references = array(), &$references_infos = array() );
+	public function convert($post_id, &$references = array(), &$references_infos = array());
 
 }

src/wordlift/videoobject/sitemap/class-video-sitemap.php

Indentation +65 added lines, -65 removed lines

@@ -10,76 +10,76 @@
 
 class Video_Sitemap {
 
-	/**
-	 * @var Ttl_Cache
-	 */
-	private $sitemap_cache;
-
-	const XML_CACHE_KEY = 'video_sitemap';
-
-	public function __construct( $sitemap_cache ) {
-		$this->sitemap_cache = $sitemap_cache;
-	}
-
-	public function init() {
-		if ( self::is_video_sitemap_enabled() ) {
-			add_action( 'template_redirect', array( $this, 'print_video_sitemap' ), 1 );
-		}
-		add_action( 'wordlift_videoobject_video_storage_updated', array( $this, 'flush_cache' ) );
-	}
-
-	public function flush_cache() {
-		$this->sitemap_cache->flush();
-	}
-
-	/**
-	 * Print video sitemap.
-	 */
-	public function print_video_sitemap() {
-		global $wp;
-
-		$url = home_url( $wp->request );
-
-		$pattern = '/wl-video-sitemap\.xml$/m';
-
-		if ( preg_match( $pattern, $url ) !== 1 ) {
-			return;
-		}
-
-		header( "Content-type: text/xml" );
-		// set 200 status code.
-		status_header(200);
-
-		$xml = $this->sitemap_cache->get( self::XML_CACHE_KEY );
-
-		if ( ! $xml ) {
-			$xml = $this->get_sitemap_xml();
-			$this->sitemap_cache->put( self::XML_CACHE_KEY, $xml );
-		}
-
-		echo $xml;
-		die();
-	}
-
-	public static function is_video_sitemap_enabled() {
-		return intval( get_option( '_wl_video_sitemap_generation', false ) ) === 1;
-	}
-
-	/**
-	 * @return string
-	 */
-	private function get_sitemap_xml() {
-		$sitemap_start_tag = <<<EOF
+    /**
+     * @var Ttl_Cache
+     */
+    private $sitemap_cache;
+
+    const XML_CACHE_KEY = 'video_sitemap';
+
+    public function __construct( $sitemap_cache ) {
+        $this->sitemap_cache = $sitemap_cache;
+    }
+
+    public function init() {
+        if ( self::is_video_sitemap_enabled() ) {
+            add_action( 'template_redirect', array( $this, 'print_video_sitemap' ), 1 );
+        }
+        add_action( 'wordlift_videoobject_video_storage_updated', array( $this, 'flush_cache' ) );
+    }
+
+    public function flush_cache() {
+        $this->sitemap_cache->flush();
+    }
+
+    /**
+     * Print video sitemap.
+     */
+    public function print_video_sitemap() {
+        global $wp;
+
+        $url = home_url( $wp->request );
+
+        $pattern = '/wl-video-sitemap\.xml$/m';
+
+        if ( preg_match( $pattern, $url ) !== 1 ) {
+            return;
+        }
+
+        header( "Content-type: text/xml" );
+        // set 200 status code.
+        status_header(200);
+
+        $xml = $this->sitemap_cache->get( self::XML_CACHE_KEY );
+
+        if ( ! $xml ) {
+            $xml = $this->get_sitemap_xml();
+            $this->sitemap_cache->put( self::XML_CACHE_KEY, $xml );
+        }
+
+        echo $xml;
+        die();
+    }
+
+    public static function is_video_sitemap_enabled() {
+        return intval( get_option( '_wl_video_sitemap_generation', false ) ) === 1;
+    }
+
+    /**
+     * @return string
+     */
+    private function get_sitemap_xml() {
+        $sitemap_start_tag = <<<EOF
 <urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9"
         xmlns:video="http://www.google.com/schemas/sitemap-video/1.1">
 EOF;
-		$sitemap_body      = Xml_Generator::get_xml_for_all_posts_with_videos();
+        $sitemap_body      = Xml_Generator::get_xml_for_all_posts_with_videos();
 
-		$sitemap_end_tag = "</urlset>";
+        $sitemap_end_tag = "</urlset>";
 
-		$xml = $sitemap_start_tag . $sitemap_body . $sitemap_end_tag;
+        $xml = $sitemap_start_tag . $sitemap_body . $sitemap_end_tag;
 
-		return $xml;
-	}
+        return $xml;
+    }
 
 }
\ No newline at end of file

Spacing +13 added lines, -13 removed lines

@@ -17,15 +17,15 @@ 
 
 	const XML_CACHE_KEY = 'video_sitemap';
 
-	public function __construct( $sitemap_cache ) {
+	public function __construct($sitemap_cache) {
 		$this->sitemap_cache = $sitemap_cache;
 	}
 
 	public function init() {
-		if ( self::is_video_sitemap_enabled() ) {
-			add_action( 'template_redirect', array( $this, 'print_video_sitemap' ), 1 );
+		if (self::is_video_sitemap_enabled()) {
+			add_action('template_redirect', array($this, 'print_video_sitemap'), 1);
 		}
-		add_action( 'wordlift_videoobject_video_storage_updated', array( $this, 'flush_cache' ) );
+		add_action('wordlift_videoobject_video_storage_updated', array($this, 'flush_cache'));
 	}
 
 	public function flush_cache() {
@@ -38,23 +38,23 @@ 
 	public function print_video_sitemap() {
 		global $wp;
 
-		$url = home_url( $wp->request );
+		$url = home_url($wp->request);
 
 		$pattern = '/wl-video-sitemap\.xml$/m';
 
-		if ( preg_match( $pattern, $url ) !== 1 ) {
+		if (preg_match($pattern, $url) !== 1) {
 			return;
 		}
 
-		header( "Content-type: text/xml" );
+		header("Content-type: text/xml");
 		// set 200 status code.
 		status_header(200);
 
-		$xml = $this->sitemap_cache->get( self::XML_CACHE_KEY );
+		$xml = $this->sitemap_cache->get(self::XML_CACHE_KEY);
 
-		if ( ! $xml ) {
+		if ( ! $xml) {
 			$xml = $this->get_sitemap_xml();
-			$this->sitemap_cache->put( self::XML_CACHE_KEY, $xml );
+			$this->sitemap_cache->put(self::XML_CACHE_KEY, $xml);
 		}
 
 		echo $xml;
@@ -62,7 +62,7 @@ 
 	}
 
 	public static function is_video_sitemap_enabled() {
-		return intval( get_option( '_wl_video_sitemap_generation', false ) ) === 1;
+		return intval(get_option('_wl_video_sitemap_generation', false)) === 1;
 	}
 
 	/**
@@ -73,11 +73,11 @@ 
 <urlset xmlns="http://www.sitemaps.org/schemas/sitemap/0.9"
         xmlns:video="http://www.google.com/schemas/sitemap-video/1.1">
 EOF;
-		$sitemap_body      = Xml_Generator::get_xml_for_all_posts_with_videos();
+		$sitemap_body = Xml_Generator::get_xml_for_all_posts_with_videos();
 
 		$sitemap_end_tag = "</urlset>";
 
-		$xml = $sitemap_start_tag . $sitemap_body . $sitemap_end_tag;
+		$xml = $sitemap_start_tag.$sitemap_body.$sitemap_end_tag;
 
 		return $xml;
 	}

src/wordlift/videoobject/pages/class-import-videos-page.php

Indentation +29 added lines, -29 removed lines

@@ -4,40 +4,40 @@ 
 
 class Import_Videos_Page {
 
-	/**
-	 * Sync_Page constructor.
-	 */
-	public function __construct() {
+    /**
+     * Sync_Page constructor.
+     */
+    public function __construct() {
 
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+        add_action( 'admin_menu', array( $this, 'admin_menu' ) );
 
-	}
+    }
 
-	public function admin_menu() {
+    public function admin_menu() {
 
-		add_submenu_page( 'wl_admin_menu',
-			__( 'Import all videos', 'wordlift' ),
-			__( 'Import all videos', 'wordlift' ),
-			'manage_options', 'wl_videos_import', array(
-				$this,
-				'render'
-			) );
+        add_submenu_page( 'wl_admin_menu',
+            __( 'Import all videos', 'wordlift' ),
+            __( 'Import all videos', 'wordlift' ),
+            'manage_options', 'wl_videos_import', array(
+                $this,
+                'render'
+            ) );
 
-	}
+    }
 
-	public function render() {
+    public function render() {
 
-		wp_enqueue_script(
-			'wl-videos-sync-page',
-			plugin_dir_url( __FILE__ ) . 'assets/videoobject-import-page.js',
-			array( 'wp-api' ),
-			\Wordlift::get_instance()->get_version() );
-		wp_localize_script( 'wl-videos-sync-page', '_wlVideoObjectImportSettings', array(
-			'restUrl' => get_rest_url(),
-			'nonce'   => wp_create_nonce( 'wp_rest' )
-		) )
+        wp_enqueue_script(
+            'wl-videos-sync-page',
+            plugin_dir_url( __FILE__ ) . 'assets/videoobject-import-page.js',
+            array( 'wp-api' ),
+            \Wordlift::get_instance()->get_version() );
+        wp_localize_script( 'wl-videos-sync-page', '_wlVideoObjectImportSettings', array(
+            'restUrl' => get_rest_url(),
+            'nonce'   => wp_create_nonce( 'wp_rest' )
+        ) )
 
-		?>
+        ?>
         <div class="wrap">
             <h2><?php esc_html_e( 'Import all videos', 'wordlift' ); ?></h2>
 
@@ -47,12 +47,12 @@ 
             </div>
 
             <button id="wl-start-btn" type="button" class="button button-large button-primary"><?php
-				esc_html_e( 'Start', 'wordlift' ); ?></button>
+                esc_html_e( 'Start', 'wordlift' ); ?></button>
             <button id="wl-stop-btn" type="button" class="button button-large button-primary hidden"><?php
-				esc_html_e( 'Stop', 'wordlift' ); ?></button>
+                esc_html_e( 'Stop', 'wordlift' ); ?></button>
 
         </div>
 		<?php
-	}
+    }
 
 }
\ No newline at end of file

Spacing +13 added lines, -13 removed lines

@@ -9,19 +9,19 @@ 
 	 */
 	public function __construct() {
 
-		add_action( 'admin_menu', array( $this, 'admin_menu' ) );
+		add_action('admin_menu', array($this, 'admin_menu'));
 
 	}
 
 	public function admin_menu() {
 
-		add_submenu_page( 'wl_admin_menu',
-			__( 'Import all videos', 'wordlift' ),
-			__( 'Import all videos', 'wordlift' ),
+		add_submenu_page('wl_admin_menu',
+			__('Import all videos', 'wordlift'),
+			__('Import all videos', 'wordlift'),
 			'manage_options', 'wl_videos_import', array(
 				$this,
 				'render'
-			) );
+			));
 
 	}
 
@@ -29,17 +29,17 @@ 
 
 		wp_enqueue_script(
 			'wl-videos-sync-page',
-			plugin_dir_url( __FILE__ ) . 'assets/videoobject-import-page.js',
-			array( 'wp-api' ),
+			plugin_dir_url(__FILE__).'assets/videoobject-import-page.js',
+			array('wp-api'),
 			\Wordlift::get_instance()->get_version() );
-		wp_localize_script( 'wl-videos-sync-page', '_wlVideoObjectImportSettings', array(
+		wp_localize_script('wl-videos-sync-page', '_wlVideoObjectImportSettings', array(
 			'restUrl' => get_rest_url(),
-			'nonce'   => wp_create_nonce( 'wp_rest' )
-		) )
+			'nonce'   => wp_create_nonce('wp_rest')
+		))
 
 		?>
         <div class="wrap">
-            <h2><?php esc_html_e( 'Import all videos', 'wordlift' ); ?></h2>
+            <h2><?php esc_html_e('Import all videos', 'wordlift'); ?></h2>
 
             <div class="wl-task__progress" style="border: 1px solid #23282D; height: 20px; margin: 8px 0;">
                 <div class="wl-task__progress__bar"
@@ -47,9 +47,9 @@ 
             </div>
 
             <button id="wl-start-btn" type="button" class="button button-large button-primary"><?php
-				esc_html_e( 'Start', 'wordlift' ); ?></button>
+				esc_html_e('Start', 'wordlift'); ?></button>
             <button id="wl-stop-btn" type="button" class="button button-large button-primary hidden"><?php
-				esc_html_e( 'Stop', 'wordlift' ); ?></button>
+				esc_html_e('Stop', 'wordlift'); ?></button>
 
         </div>
 		<?php

src/public/class-wordlift-related-entities-cloud-shortcode.php

Indentation +152 added lines, -152 removed lines

@@ -16,164 +16,164 @@
  */
 class Wordlift_Related_Entities_Cloud_Shortcode extends Wordlift_Shortcode {
 
-	/**
-	 * {@inheritdoc}
-	 */
-	const SHORTCODE = 'wl_cloud';
-
-	/**
-	 * The {@link Wordlift_Relation_Service} instance.
-	 *
-	 * @since  3.15.0
-	 * @access private
-	 * @var \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
-	 */
-	private $relation_service;
-	/**
-	 * @var Wordlift_Entity_Service
-	 */
-	private $entity_service;
-
-	/**
-	 * Create a {@link Wordlift_Related_Entities_Cloud_Shortcode} instance.
-	 *
-	 * @param \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
-	 * @param \Wordlift_Entity_Service $entity_service
-	 *
-	 * @since 3.15.0
-	 *
-	 */
-	public function __construct( $relation_service, $entity_service ) {
-		parent::__construct();
-
-		$this->relation_service = $relation_service;
-
-		$this->entity_service = $entity_service;
-
-		$this->register_block_type();
-
-	}
-
-	/**
-	 * {@inheritdoc}
-	 */
-	public function render( $atts ) {
-
-		$tags = $this->get_related_entities_tags();
-
-		// Bail out if there are no associated entities.
-		if ( empty( $tags ) ) {
-			return '';
-		}
-
-		/*
+    /**
+     * {@inheritdoc}
+     */
+    const SHORTCODE = 'wl_cloud';
+
+    /**
+     * The {@link Wordlift_Relation_Service} instance.
+     *
+     * @since  3.15.0
+     * @access private
+     * @var \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
+     */
+    private $relation_service;
+    /**
+     * @var Wordlift_Entity_Service
+     */
+    private $entity_service;
+
+    /**
+     * Create a {@link Wordlift_Related_Entities_Cloud_Shortcode} instance.
+     *
+     * @param \Wordlift_Relation_Service $relation_service The {@link Wordlift_Relation_Service} instance.
+     * @param \Wordlift_Entity_Service $entity_service
+     *
+     * @since 3.15.0
+     *
+     */
+    public function __construct( $relation_service, $entity_service ) {
+        parent::__construct();
+
+        $this->relation_service = $relation_service;
+
+        $this->entity_service = $entity_service;
+
+        $this->register_block_type();
+
+    }
+
+    /**
+     * {@inheritdoc}
+     */
+    public function render( $atts ) {
+
+        $tags = $this->get_related_entities_tags();
+
+        // Bail out if there are no associated entities.
+        if ( empty( $tags ) ) {
+            return '';
+        }
+
+        /*
 		 * Since the output is use in the widget as well, we need to have the
 		 * same class as the core tagcloud widget, to easily inherit its styling.
 		 */
 
-		return '<div class="tagcloud wl-related-entities-cloud">' .
-		       wp_generate_tag_cloud( $tags ) .
-		       '</div>';
-	}
-
-	private function register_block_type() {
-
-		$scope = $this;
-
-		add_action( 'init', function () use ( $scope ) {
-			if ( ! function_exists( 'register_block_type' ) ) {
-				// Gutenberg is not active.
-				return;
-			}
-
-			register_block_type( 'wordlift/cloud', array(
-				'editor_script'   => 'wl-block-editor',
-				'render_callback' => function ( $attributes ) use ( $scope ) {
-					$attr_code = '';
-					foreach ( $attributes as $key => $value ) {
-						$attr_code .= $key . '="' . htmlentities( $value ) . '" ';
-					}
-
-					return '[' . $scope::SHORTCODE . ' ' . $attr_code . ']';
-				},
-				'attributes'      => array(
-					'preview'     => array(
-						'type'    => 'boolean',
-						'default' => false,
-					),
-					'preview_src' => array(
-						'type'    => 'string',
-						'default' => WP_CONTENT_URL . '/plugins/wordlift/images/block-previews/cloud.png',
-					),
-				),
-			) );
-		} );
-	}
-
-	/**
-	 * Find the related entities to the currently displayed post and
-	 * calculate the "tags" for them as wp_generate_tag_cloud expects to get.
-	 *
-	 * @return array    Array of tags. Empty array in case we re not in a context
-	 *                  of a post, or it has no related entities.
-	 * @since 3.11.0
-	 *
-	 */
-	public function get_related_entities_tags() {
-
-		// Define the supported types list.
-		$supported_types = Wordlift_Entity_Service::valid_entity_post_types();
-
-		// Show nothing if not on a post or entity page.
-		if ( ! is_singular( $supported_types ) ) {
-			return array();
-		}
-
-		// Get the IDs of entities related to current post.
-		$related_entities = wl_core_get_related_entity_ids( get_the_ID(), array( 'status' => 'publish' ) );
-
-		// Bail out if there are no associated entities.
-		if ( empty( $related_entities ) ) {
-			return array();
-		}
-
-		/*
+        return '<div class="tagcloud wl-related-entities-cloud">' .
+                wp_generate_tag_cloud( $tags ) .
+                '</div>';
+    }
+
+    private function register_block_type() {
+
+        $scope = $this;
+
+        add_action( 'init', function () use ( $scope ) {
+            if ( ! function_exists( 'register_block_type' ) ) {
+                // Gutenberg is not active.
+                return;
+            }
+
+            register_block_type( 'wordlift/cloud', array(
+                'editor_script'   => 'wl-block-editor',
+                'render_callback' => function ( $attributes ) use ( $scope ) {
+                    $attr_code = '';
+                    foreach ( $attributes as $key => $value ) {
+                        $attr_code .= $key . '="' . htmlentities( $value ) . '" ';
+                    }
+
+                    return '[' . $scope::SHORTCODE . ' ' . $attr_code . ']';
+                },
+                'attributes'      => array(
+                    'preview'     => array(
+                        'type'    => 'boolean',
+                        'default' => false,
+                    ),
+                    'preview_src' => array(
+                        'type'    => 'string',
+                        'default' => WP_CONTENT_URL . '/plugins/wordlift/images/block-previews/cloud.png',
+                    ),
+                ),
+            ) );
+        } );
+    }
+
+    /**
+     * Find the related entities to the currently displayed post and
+     * calculate the "tags" for them as wp_generate_tag_cloud expects to get.
+     *
+     * @return array    Array of tags. Empty array in case we re not in a context
+     *                  of a post, or it has no related entities.
+     * @since 3.11.0
+     *
+     */
+    public function get_related_entities_tags() {
+
+        // Define the supported types list.
+        $supported_types = Wordlift_Entity_Service::valid_entity_post_types();
+
+        // Show nothing if not on a post or entity page.
+        if ( ! is_singular( $supported_types ) ) {
+            return array();
+        }
+
+        // Get the IDs of entities related to current post.
+        $related_entities = wl_core_get_related_entity_ids( get_the_ID(), array( 'status' => 'publish' ) );
+
+        // Bail out if there are no associated entities.
+        if ( empty( $related_entities ) ) {
+            return array();
+        }
+
+        /*
 		 * Create an array of "tags" to feed to wp_generate_tag_cloud.
 		 * Use the number of posts and entities connected to the entity as a weight.
 		 */
-		$tags = array();
-
-		foreach ( array_unique( $related_entities ) as $entity_id ) {
-
-			$connected_entities = count( wl_core_get_related_entity_ids( $entity_id, array( 'status' => 'publish' ) ) );
-			$connected_posts    = count( $this->relation_service->get_article_subjects( $entity_id, '*', null, 'publish' ) );
-			/**
-			 * @since 3.31.5
-			 * if synonym exists, use it instead of entity name.
-			 */
-			$synonyms    = $this->get_synonyms( $entity_id );
-			$entity_name = count( $synonyms ) > 0 ? $synonyms[0] : get_the_title( $entity_id );
-
-			$tags[] = (object) array(
-				'id'    => $entity_id,
-				// Used to give a unique class on the tag.
-				'name'  => $entity_name,
-				// The text of the tag.
-				'slug'  => get_the_title( $entity_id ),
-				// Required but not seem to be relevant
-				'link'  => get_permalink( $entity_id ),
-				// the url the tag links to.
-				'count' => $connected_entities + $connected_posts,
-				// The weight.
-			);
-
-		}
-
-		return $tags;
-	}
-
-	private function get_synonyms( $entity_id ) {
-		return $this->entity_service->get_alternative_labels( $entity_id );
-	}
+        $tags = array();
+
+        foreach ( array_unique( $related_entities ) as $entity_id ) {
+
+            $connected_entities = count( wl_core_get_related_entity_ids( $entity_id, array( 'status' => 'publish' ) ) );
+            $connected_posts    = count( $this->relation_service->get_article_subjects( $entity_id, '*', null, 'publish' ) );
+            /**
+             * @since 3.31.5
+             * if synonym exists, use it instead of entity name.
+             */
+            $synonyms    = $this->get_synonyms( $entity_id );
+            $entity_name = count( $synonyms ) > 0 ? $synonyms[0] : get_the_title( $entity_id );
+
+            $tags[] = (object) array(
+                'id'    => $entity_id,
+                // Used to give a unique class on the tag.
+                'name'  => $entity_name,
+                // The text of the tag.
+                'slug'  => get_the_title( $entity_id ),
+                // Required but not seem to be relevant
+                'link'  => get_permalink( $entity_id ),
+                // the url the tag links to.
+                'count' => $connected_entities + $connected_posts,
+                // The weight.
+            );
+
+        }
+
+        return $tags;
+    }
+
+    private function get_synonyms( $entity_id ) {
+        return $this->entity_service->get_alternative_labels( $entity_id );
+    }
 
 }

Spacing +26 added lines, -26 removed lines

@@ -43,7 +43,7 @@ 
 	 * @since 3.15.0
 	 *
 	 */
-	public function __construct( $relation_service, $entity_service ) {
+	public function __construct($relation_service, $entity_service) {
 		parent::__construct();
 
 		$this->relation_service = $relation_service;
@@ -57,12 +57,12 @@ 
 	/**
 	 * {@inheritdoc}
 	 */
-	public function render( $atts ) {
+	public function render($atts) {
 
 		$tags = $this->get_related_entities_tags();
 
 		// Bail out if there are no associated entities.
-		if ( empty( $tags ) ) {
+		if (empty($tags)) {
 			return '';
 		}
 
@@ -71,8 +71,8 @@ 
 		 * same class as the core tagcloud widget, to easily inherit its styling.
 		 */
 
-		return '<div class="tagcloud wl-related-entities-cloud">' .
-		       wp_generate_tag_cloud( $tags ) .
+		return '<div class="tagcloud wl-related-entities-cloud">'.
+		       wp_generate_tag_cloud($tags).
 		       '</div>';
 	}
 
@@ -80,21 +80,21 @@ 
 
 		$scope = $this;
 
-		add_action( 'init', function () use ( $scope ) {
-			if ( ! function_exists( 'register_block_type' ) ) {
+		add_action('init', function() use ($scope) {
+			if ( ! function_exists('register_block_type')) {
 				// Gutenberg is not active.
 				return;
 			}
 
-			register_block_type( 'wordlift/cloud', array(
+			register_block_type('wordlift/cloud', array(
 				'editor_script'   => 'wl-block-editor',
-				'render_callback' => function ( $attributes ) use ( $scope ) {
+				'render_callback' => function($attributes) use ($scope) {
 					$attr_code = '';
-					foreach ( $attributes as $key => $value ) {
-						$attr_code .= $key . '="' . htmlentities( $value ) . '" ';
+					foreach ($attributes as $key => $value) {
+						$attr_code .= $key.'="'.htmlentities($value).'" ';
 					}
 
-					return '[' . $scope::SHORTCODE . ' ' . $attr_code . ']';
+					return '['.$scope::SHORTCODE.' '.$attr_code.']';
 				},
 				'attributes'      => array(
 					'preview'     => array(
@@ -103,10 +103,10 @@ 
 					),
 					'preview_src' => array(
 						'type'    => 'string',
-						'default' => WP_CONTENT_URL . '/plugins/wordlift/images/block-previews/cloud.png',
+						'default' => WP_CONTENT_URL.'/plugins/wordlift/images/block-previews/cloud.png',
 					),
 				),
-			) );
+			));
 		} );
 	}
 
@@ -125,15 +125,15 @@ 
 		$supported_types = Wordlift_Entity_Service::valid_entity_post_types();
 
 		// Show nothing if not on a post or entity page.
-		if ( ! is_singular( $supported_types ) ) {
+		if ( ! is_singular($supported_types)) {
 			return array();
 		}
 
 		// Get the IDs of entities related to current post.
-		$related_entities = wl_core_get_related_entity_ids( get_the_ID(), array( 'status' => 'publish' ) );
+		$related_entities = wl_core_get_related_entity_ids(get_the_ID(), array('status' => 'publish'));
 
 		// Bail out if there are no associated entities.
-		if ( empty( $related_entities ) ) {
+		if (empty($related_entities)) {
 			return array();
 		}
 
@@ -143,25 +143,25 @@ 
 		 */
 		$tags = array();
 
-		foreach ( array_unique( $related_entities ) as $entity_id ) {
+		foreach (array_unique($related_entities) as $entity_id) {
 
-			$connected_entities = count( wl_core_get_related_entity_ids( $entity_id, array( 'status' => 'publish' ) ) );
-			$connected_posts    = count( $this->relation_service->get_article_subjects( $entity_id, '*', null, 'publish' ) );
+			$connected_entities = count(wl_core_get_related_entity_ids($entity_id, array('status' => 'publish')));
+			$connected_posts    = count($this->relation_service->get_article_subjects($entity_id, '*', null, 'publish'));
 			/**
 			 * @since 3.31.5
 			 * if synonym exists, use it instead of entity name.
 			 */
-			$synonyms    = $this->get_synonyms( $entity_id );
-			$entity_name = count( $synonyms ) > 0 ? $synonyms[0] : get_the_title( $entity_id );
+			$synonyms    = $this->get_synonyms($entity_id);
+			$entity_name = count($synonyms) > 0 ? $synonyms[0] : get_the_title($entity_id);
 
 			$tags[] = (object) array(
 				'id'    => $entity_id,
 				// Used to give a unique class on the tag.
 				'name'  => $entity_name,
 				// The text of the tag.
-				'slug'  => get_the_title( $entity_id ),
+				'slug'  => get_the_title($entity_id),
 				// Required but not seem to be relevant
-				'link'  => get_permalink( $entity_id ),
+				'link'  => get_permalink($entity_id),
 				// the url the tag links to.
 				'count' => $connected_entities + $connected_posts,
 				// The weight.
@@ -172,8 +172,8 @@ 
 		return $tags;
 	}
 
-	private function get_synonyms( $entity_id ) {
-		return $this->entity_service->get_alternative_labels( $entity_id );
+	private function get_synonyms($entity_id) {
+		return $this->entity_service->get_alternative_labels($entity_id);
 	}
 
 }

src/wordlift/videoobject/sitemap/class-xml-generator.php

Indentation +105 added lines, -105 removed lines

@@ -12,110 +12,110 @@ 
 
 class Xml_Generator {
 
-	private static function iso8601_to_seconds( $iso8601_interval_string ) {
-		try {
-			$interval = new \DateInterval( $iso8601_interval_string );
-		} catch ( \Exception $e ) {
-			return 0;
-		}
-
-		$days_to_seconds    = $interval->d * 60 * 60 * 24;
-		$hours_to_seconds   = $interval->h * 60 * 60;
-		$minutes_to_seconds = $interval->i * 60;
-		$seconds            = $interval->s;
-
-		return $days_to_seconds + $hours_to_seconds + $minutes_to_seconds + $seconds;
-
-	}
-
-	public static function get_xml_for_all_posts_with_videos() {
-		/**
-		 * @since 3.31.6
-		 * Filter the query args, add support for all custom public post types.
-		 */
-		$video_sitemap_query_args = apply_filters( 'wl_videoobject_sitemap_query_args', array(
-			'fields'      => 'ids',
-			'numberposts' => - 1,
-			'post_type'   => get_post_types( array( 'public' => true ) ),
-			'meta_query'  => array(
-				array(
-					'key'     => Meta_Storage::META_KEY,
-					'compare' => 'EXISTS'
-				)
-			)
-		) );
-
-		$posts = get_posts( $video_sitemap_query_args );
-
-		$all_posts_xml = "";
-
-		if ( ! $posts ) {
-			return $all_posts_xml;
-		}
-
-		foreach ( $posts as $post_id ) {
-			$all_posts_xml .= self::get_xml_for_single_post( $post_id );
-		}
-
-		return $all_posts_xml;
-
-	}
-
-
-	/**
-	 * @param $post_id
-	 *
-	 * @return string XML string for single post.
-	 */
-	public static function get_xml_for_single_post( $post_id ) {
-		$videos = Video_Storage_Factory::get_storage()->get_all_videos( $post_id );
-		if ( ! $videos ) {
-			return "";
-		}
-		$single_post_xml = "";
-		foreach ( $videos as $video ) {
-			$single_post_xml .= self::get_xml_for_single_video( $video, $post_id );
-		}
-
-		return $single_post_xml;
-
-	}
-
-
-	/**
-	 * @param $video Video
-	 * @param $post_id int
-	 *
-	 * @return string
-	 */
-	public static function get_xml_for_single_video( $video, $post_id ) {
-
-		$permalink     = get_permalink( $post_id );
-		$title         = esc_html( $video->name );
-		$description   = esc_html( $video->description );
-		$thumbnail_url = $video->thumbnail_urls[0];
-
-		// If description is empty use title.
-		if ( ! $description ) {
-			$description = $title;
-		}
-
-		$optional_fields = array(
-			'content_loc' => $video->content_url,
-			'player_loc'  => $video->embed_url,
-			'duration'    => self::iso8601_to_seconds( $video->duration ),
-			'view_count'  => $video->views,
-			'live'        => $video->is_live_video ? 'yes' : 'no'
-		);
-
-		$optional_data = "";
-		foreach ( $optional_fields as $xml_key => $xml_value ) {
-			if ( $xml_value ) {
-				$optional_data .= "<video:${xml_key}>${xml_value}</video:${xml_key}>";
-			}
-		}
-
-		return <<<EOF
+    private static function iso8601_to_seconds( $iso8601_interval_string ) {
+        try {
+            $interval = new \DateInterval( $iso8601_interval_string );
+        } catch ( \Exception $e ) {
+            return 0;
+        }
+
+        $days_to_seconds    = $interval->d * 60 * 60 * 24;
+        $hours_to_seconds   = $interval->h * 60 * 60;
+        $minutes_to_seconds = $interval->i * 60;
+        $seconds            = $interval->s;
+
+        return $days_to_seconds + $hours_to_seconds + $minutes_to_seconds + $seconds;
+
+    }
+
+    public static function get_xml_for_all_posts_with_videos() {
+        /**
+         * @since 3.31.6
+         * Filter the query args, add support for all custom public post types.
+         */
+        $video_sitemap_query_args = apply_filters( 'wl_videoobject_sitemap_query_args', array(
+            'fields'      => 'ids',
+            'numberposts' => - 1,
+            'post_type'   => get_post_types( array( 'public' => true ) ),
+            'meta_query'  => array(
+                array(
+                    'key'     => Meta_Storage::META_KEY,
+                    'compare' => 'EXISTS'
+                )
+            )
+        ) );
+
+        $posts = get_posts( $video_sitemap_query_args );
+
+        $all_posts_xml = "";
+
+        if ( ! $posts ) {
+            return $all_posts_xml;
+        }
+
+        foreach ( $posts as $post_id ) {
+            $all_posts_xml .= self::get_xml_for_single_post( $post_id );
+        }
+
+        return $all_posts_xml;
+
+    }
+
+
+    /**
+     * @param $post_id
+     *
+     * @return string XML string for single post.
+     */
+    public static function get_xml_for_single_post( $post_id ) {
+        $videos = Video_Storage_Factory::get_storage()->get_all_videos( $post_id );
+        if ( ! $videos ) {
+            return "";
+        }
+        $single_post_xml = "";
+        foreach ( $videos as $video ) {
+            $single_post_xml .= self::get_xml_for_single_video( $video, $post_id );
+        }
+
+        return $single_post_xml;
+
+    }
+
+
+    /**
+     * @param $video Video
+     * @param $post_id int
+     *
+     * @return string
+     */
+    public static function get_xml_for_single_video( $video, $post_id ) {
+
+        $permalink     = get_permalink( $post_id );
+        $title         = esc_html( $video->name );
+        $description   = esc_html( $video->description );
+        $thumbnail_url = $video->thumbnail_urls[0];
+
+        // If description is empty use title.
+        if ( ! $description ) {
+            $description = $title;
+        }
+
+        $optional_fields = array(
+            'content_loc' => $video->content_url,
+            'player_loc'  => $video->embed_url,
+            'duration'    => self::iso8601_to_seconds( $video->duration ),
+            'view_count'  => $video->views,
+            'live'        => $video->is_live_video ? 'yes' : 'no'
+        );
+
+        $optional_data = "";
+        foreach ( $optional_fields as $xml_key => $xml_value ) {
+            if ( $xml_value ) {
+                $optional_data .= "<video:${xml_key}>${xml_value}</video:${xml_key}>";
+            }
+        }
+
+        return <<<EOF
    <url>
      <loc>${permalink}</loc>
      <video:video>
@@ -127,6 +127,6 @@ 
    </url>
 EOF;
 
-	}
+    }
 
 }
\ No newline at end of file

Spacing +24 added lines, -24 removed lines

@@ -12,10 +12,10 @@ 
 
 class Xml_Generator {
 
-	private static function iso8601_to_seconds( $iso8601_interval_string ) {
+	private static function iso8601_to_seconds($iso8601_interval_string) {
 		try {
-			$interval = new \DateInterval( $iso8601_interval_string );
-		} catch ( \Exception $e ) {
+			$interval = new \DateInterval($iso8601_interval_string);
+		} catch (\Exception $e) {
 			return 0;
 		}
 
@@ -33,28 +33,28 @@ 
 		 * @since 3.31.6
 		 * Filter the query args, add support for all custom public post types.
 		 */
-		$video_sitemap_query_args = apply_filters( 'wl_videoobject_sitemap_query_args', array(
+		$video_sitemap_query_args = apply_filters('wl_videoobject_sitemap_query_args', array(
 			'fields'      => 'ids',
-			'numberposts' => - 1,
-			'post_type'   => get_post_types( array( 'public' => true ) ),
+			'numberposts' => -1,
+			'post_type'   => get_post_types(array('public' => true)),
 			'meta_query'  => array(
 				array(
 					'key'     => Meta_Storage::META_KEY,
 					'compare' => 'EXISTS'
 				)
 			)
-		) );
+		));
 
-		$posts = get_posts( $video_sitemap_query_args );
+		$posts = get_posts($video_sitemap_query_args);
 
 		$all_posts_xml = "";
 
-		if ( ! $posts ) {
+		if ( ! $posts) {
 			return $all_posts_xml;
 		}
 
-		foreach ( $posts as $post_id ) {
-			$all_posts_xml .= self::get_xml_for_single_post( $post_id );
+		foreach ($posts as $post_id) {
+			$all_posts_xml .= self::get_xml_for_single_post($post_id);
 		}
 
 		return $all_posts_xml;
@@ -67,14 +67,14 @@ 
 	 *
 	 * @return string XML string for single post.
 	 */
-	public static function get_xml_for_single_post( $post_id ) {
-		$videos = Video_Storage_Factory::get_storage()->get_all_videos( $post_id );
-		if ( ! $videos ) {
+	public static function get_xml_for_single_post($post_id) {
+		$videos = Video_Storage_Factory::get_storage()->get_all_videos($post_id);
+		if ( ! $videos) {
 			return "";
 		}
 		$single_post_xml = "";
-		foreach ( $videos as $video ) {
-			$single_post_xml .= self::get_xml_for_single_video( $video, $post_id );
+		foreach ($videos as $video) {
+			$single_post_xml .= self::get_xml_for_single_video($video, $post_id);
 		}
 
 		return $single_post_xml;
@@ -88,29 +88,29 @@ 
 	 *
 	 * @return string
 	 */
-	public static function get_xml_for_single_video( $video, $post_id ) {
+	public static function get_xml_for_single_video($video, $post_id) {
 
-		$permalink     = get_permalink( $post_id );
-		$title         = esc_html( $video->name );
-		$description   = esc_html( $video->description );
+		$permalink     = get_permalink($post_id);
+		$title         = esc_html($video->name);
+		$description   = esc_html($video->description);
 		$thumbnail_url = $video->thumbnail_urls[0];
 
 		// If description is empty use title.
-		if ( ! $description ) {
+		if ( ! $description) {
 			$description = $title;
 		}
 
 		$optional_fields = array(
 			'content_loc' => $video->content_url,
 			'player_loc'  => $video->embed_url,
-			'duration'    => self::iso8601_to_seconds( $video->duration ),
+			'duration'    => self::iso8601_to_seconds($video->duration),
 			'view_count'  => $video->views,
 			'live'        => $video->is_live_video ? 'yes' : 'no'
 		);
 
 		$optional_data = "";
-		foreach ( $optional_fields as $xml_key => $xml_value ) {
-			if ( $xml_value ) {
+		foreach ($optional_fields as $xml_key => $xml_value) {
+			if ($xml_value) {
 				$optional_data .= "<video:${xml_key}>${xml_value}</video:${xml_key}>";
 			}
 		}