insideout10 / wordlift-plugin

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}>";
 			}
 		}

src/wordlift/videoobject/jsonld/class-jsonld.php

Indentation +166 added lines, -166 removed lines

@@ -11,172 +11,172 @@
 use Wordlift\Videoobject\Data\Video_Storage\Storage;
 
 class Jsonld {
-	/**
-	 * @var Storage
-	 */
-	private $video_storage;
-
-	/**
-	 * Jsonld constructor.
-	 *
-	 * @param $video_storage Storage
-	 */
-	public function __construct( $video_storage ) {
-		add_action( 'wl_post_jsonld', array( $this, 'wl_post_jsonld' ), 10, 3 );
-		add_action( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 10, 3 );
-		$this->video_storage = $video_storage;
-	}
-
-
-	public function wl_after_get_jsonld( $jsonld, $post_id, $context ) {
-		if ( 0 === count( $jsonld ) ) {
-			return $jsonld;
-		}
-		$current_item = $jsonld[0];
-
-		if ( ! array_key_exists( '@type', $current_item ) ) {
-			// Cant determine type return early.
-			return $jsonld;
-		}
-
-		$type = $current_item['@type'];
-		if ( ( is_string( $type ) && $type === 'Article' ) ||
-		     ( is_array( $type ) && in_array( 'Article', $type ) ) ) {
-			return $jsonld;
-		}
-
-		$videos_jsonld = $this->get_videos_jsonld( $post_id );
-		if ( 0 === count( $videos_jsonld ) ) {
-			return $jsonld;
-		}
-
-		// check if we have @id in jsonld for first item.
-		$id = array_key_exists( '@id', $current_item ) ? $current_item['@id'] : '';
-
-		foreach ( $videos_jsonld as &$video_jsonld ) {
-			if ( ! $id ) {
-				continue;
-			}
-			if ( ! array_key_exists( 'mentions', $video_jsonld ) ) {
-				$video_jsonld['mentions'] = array( '@id' => $id );
-			} else {
-				$video_jsonld['mentions'] = array_merge( $video_jsonld['mentions'], array( '@id' => $id ) );
-			}
-		}
-
-		return array_merge( $jsonld, $videos_jsonld );
-	}
-
-	/**
-	 * @param $existing_video_data string | array associative or sequential array.
-	 * @param $new_video_data array Sequential array.
-	 *
-	 * @return array
-	 */
-	private function merge_video_data( $existing_video_data, $new_video_data ) {
-		if ( ! is_array( $existing_video_data ) ) {
-			$new_video_data[] = $existing_video_data;
-
-			return $new_video_data;
-		}
-
-		if ( $this->is_associative_array( $existing_video_data ) ) {
-			$new_video_data[] = $existing_video_data;
-
-			return $new_video_data;
-		}
-
-		return array_merge( $existing_video_data, $new_video_data );
-	}
-
-	public function wl_post_jsonld( $jsonld, $post_id, $references ) {
-
-		$video_jsonld = $this->get_videos_jsonld( $post_id );
-		if ( count( $video_jsonld ) === 0 ) {
-			return $jsonld;
-		}
-		// Before adding the video jsonld check if the key
-		// is present and additional data might be present,
-		// if not present just add the data and return early.
-		if ( ! array_key_exists( 'video', $jsonld ) ) {
-			$jsonld['video'] = $video_jsonld;
-
-			return $jsonld;
-		}
-
-		// since key exists, we need to merge the data based on type.
-		$previous_video_data = $jsonld['video'];
-		$jsonld['video']     = $this->merge_video_data( $previous_video_data, $video_jsonld );
-
-		return $jsonld;
-	}
-
-
-	/**
-	 * @param $post_id int Post id.
-	 *
-	 * @return array
-	 */
-	public function get_videos_jsonld( $post_id ) {
-
-		$videos = $this->video_storage->get_all_videos( $post_id );
-
-		$jsonld = array();
-
-		foreach ( $videos as $video ) {
-			/**
-			 * @var $video Video
-			 */
-			$description = $video->description;
-			if ( ! $video->description ) {
-				// If description is empty then use the video title as description
-				$description = $video->name;
-			}
-			$single_jsonld = array(
-				'@context'     => 'http://schema.org',
-				'@type'        => 'VideoObject',
-				'name'         => $video->name,
-				'description'  => $description,
-				'contentUrl'   => $video->content_url,
-				'embedUrl'     => $video->embed_url,
-				'uploadDate'   => $video->upload_date,
-				'thumbnailUrl' => $video->thumbnail_urls,
-				'duration'     => $video->duration,
-			);
-
-			if ( $video->views ) {
-				$single_jsonld['interactionStatistic'] = array(
-					'@type'                => 'InteractionCounter',
-					'interactionType'      => array(
-						'@type' => 'http://schema.org/WatchAction'
-					),
-					'userInteractionCount' => $video->views
-				);
-			}
-
-			if ( $video->is_live_video ) {
-				$single_jsonld['publication'] = array(
-					'@type'           => 'BroadcastEvent',
-					'isLiveBroadcast' => true,
-					'startDate'       => $video->live_video_start_date,
-					'endDate'         => $video->live_video_end_date
-				);
-			}
-
-			$jsonld[] = $single_jsonld;
-		}
-
-		return $jsonld;
-	}
-
-
-	private function is_associative_array( $arr ) {
-		if ( array() === $arr ) {
-			return false;
-		}
-
-		return array_keys( $arr ) !== range( 0, count( $arr ) - 1 );
-	}
+    /**
+     * @var Storage
+     */
+    private $video_storage;
+
+    /**
+     * Jsonld constructor.
+     *
+     * @param $video_storage Storage
+     */
+    public function __construct( $video_storage ) {
+        add_action( 'wl_post_jsonld', array( $this, 'wl_post_jsonld' ), 10, 3 );
+        add_action( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 10, 3 );
+        $this->video_storage = $video_storage;
+    }
+
+
+    public function wl_after_get_jsonld( $jsonld, $post_id, $context ) {
+        if ( 0 === count( $jsonld ) ) {
+            return $jsonld;
+        }
+        $current_item = $jsonld[0];
+
+        if ( ! array_key_exists( '@type', $current_item ) ) {
+            // Cant determine type return early.
+            return $jsonld;
+        }
+
+        $type = $current_item['@type'];
+        if ( ( is_string( $type ) && $type === 'Article' ) ||
+             ( is_array( $type ) && in_array( 'Article', $type ) ) ) {
+            return $jsonld;
+        }
+
+        $videos_jsonld = $this->get_videos_jsonld( $post_id );
+        if ( 0 === count( $videos_jsonld ) ) {
+            return $jsonld;
+        }
+
+        // check if we have @id in jsonld for first item.
+        $id = array_key_exists( '@id', $current_item ) ? $current_item['@id'] : '';
+
+        foreach ( $videos_jsonld as &$video_jsonld ) {
+            if ( ! $id ) {
+                continue;
+            }
+            if ( ! array_key_exists( 'mentions', $video_jsonld ) ) {
+                $video_jsonld['mentions'] = array( '@id' => $id );
+            } else {
+                $video_jsonld['mentions'] = array_merge( $video_jsonld['mentions'], array( '@id' => $id ) );
+            }
+        }
+
+        return array_merge( $jsonld, $videos_jsonld );
+    }
+
+    /**
+     * @param $existing_video_data string | array associative or sequential array.
+     * @param $new_video_data array Sequential array.
+     *
+     * @return array
+     */
+    private function merge_video_data( $existing_video_data, $new_video_data ) {
+        if ( ! is_array( $existing_video_data ) ) {
+            $new_video_data[] = $existing_video_data;
+
+            return $new_video_data;
+        }
+
+        if ( $this->is_associative_array( $existing_video_data ) ) {
+            $new_video_data[] = $existing_video_data;
+
+            return $new_video_data;
+        }
+
+        return array_merge( $existing_video_data, $new_video_data );
+    }
+
+    public function wl_post_jsonld( $jsonld, $post_id, $references ) {
+
+        $video_jsonld = $this->get_videos_jsonld( $post_id );
+        if ( count( $video_jsonld ) === 0 ) {
+            return $jsonld;
+        }
+        // Before adding the video jsonld check if the key
+        // is present and additional data might be present,
+        // if not present just add the data and return early.
+        if ( ! array_key_exists( 'video', $jsonld ) ) {
+            $jsonld['video'] = $video_jsonld;
+
+            return $jsonld;
+        }
+
+        // since key exists, we need to merge the data based on type.
+        $previous_video_data = $jsonld['video'];
+        $jsonld['video']     = $this->merge_video_data( $previous_video_data, $video_jsonld );
+
+        return $jsonld;
+    }
+
+
+    /**
+     * @param $post_id int Post id.
+     *
+     * @return array
+     */
+    public function get_videos_jsonld( $post_id ) {
+
+        $videos = $this->video_storage->get_all_videos( $post_id );
+
+        $jsonld = array();
+
+        foreach ( $videos as $video ) {
+            /**
+             * @var $video Video
+             */
+            $description = $video->description;
+            if ( ! $video->description ) {
+                // If description is empty then use the video title as description
+                $description = $video->name;
+            }
+            $single_jsonld = array(
+                '@context'     => 'http://schema.org',
+                '@type'        => 'VideoObject',
+                'name'         => $video->name,
+                'description'  => $description,
+                'contentUrl'   => $video->content_url,
+                'embedUrl'     => $video->embed_url,
+                'uploadDate'   => $video->upload_date,
+                'thumbnailUrl' => $video->thumbnail_urls,
+                'duration'     => $video->duration,
+            );
+
+            if ( $video->views ) {
+                $single_jsonld['interactionStatistic'] = array(
+                    '@type'                => 'InteractionCounter',
+                    'interactionType'      => array(
+                        '@type' => 'http://schema.org/WatchAction'
+                    ),
+                    'userInteractionCount' => $video->views
+                );
+            }
+
+            if ( $video->is_live_video ) {
+                $single_jsonld['publication'] = array(
+                    '@type'           => 'BroadcastEvent',
+                    'isLiveBroadcast' => true,
+                    'startDate'       => $video->live_video_start_date,
+                    'endDate'         => $video->live_video_end_date
+                );
+            }
+
+            $jsonld[] = $single_jsonld;
+        }
+
+        return $jsonld;
+    }
+
+
+    private function is_associative_array( $arr ) {
+        if ( array() === $arr ) {
+            return false;
+        }
+
+        return array_keys( $arr ) !== range( 0, count( $arr ) - 1 );
+    }
 
 
 }

Spacing +35 added lines, -35 removed lines

@@ -21,50 +21,50 @@ 
 	 *
 	 * @param $video_storage Storage
 	 */
-	public function __construct( $video_storage ) {
-		add_action( 'wl_post_jsonld', array( $this, 'wl_post_jsonld' ), 10, 3 );
-		add_action( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 10, 3 );
+	public function __construct($video_storage) {
+		add_action('wl_post_jsonld', array($this, 'wl_post_jsonld'), 10, 3);
+		add_action('wl_after_get_jsonld', array($this, 'wl_after_get_jsonld'), 10, 3);
 		$this->video_storage = $video_storage;
 	}
 
 
-	public function wl_after_get_jsonld( $jsonld, $post_id, $context ) {
-		if ( 0 === count( $jsonld ) ) {
+	public function wl_after_get_jsonld($jsonld, $post_id, $context) {
+		if (0 === count($jsonld)) {
 			return $jsonld;
 		}
 		$current_item = $jsonld[0];
 
-		if ( ! array_key_exists( '@type', $current_item ) ) {
+		if ( ! array_key_exists('@type', $current_item)) {
 			// Cant determine type return early.
 			return $jsonld;
 		}
 
 		$type = $current_item['@type'];
-		if ( ( is_string( $type ) && $type === 'Article' ) ||
-		     ( is_array( $type ) && in_array( 'Article', $type ) ) ) {
+		if ((is_string($type) && $type === 'Article') ||
+		     (is_array($type) && in_array('Article', $type))) {
 			return $jsonld;
 		}
 
-		$videos_jsonld = $this->get_videos_jsonld( $post_id );
-		if ( 0 === count( $videos_jsonld ) ) {
+		$videos_jsonld = $this->get_videos_jsonld($post_id);
+		if (0 === count($videos_jsonld)) {
 			return $jsonld;
 		}
 
 		// check if we have @id in jsonld for first item.
-		$id = array_key_exists( '@id', $current_item ) ? $current_item['@id'] : '';
+		$id = array_key_exists('@id', $current_item) ? $current_item['@id'] : '';
 
-		foreach ( $videos_jsonld as &$video_jsonld ) {
-			if ( ! $id ) {
+		foreach ($videos_jsonld as &$video_jsonld) {
+			if ( ! $id) {
 				continue;
 			}
-			if ( ! array_key_exists( 'mentions', $video_jsonld ) ) {
-				$video_jsonld['mentions'] = array( '@id' => $id );
+			if ( ! array_key_exists('mentions', $video_jsonld)) {
+				$video_jsonld['mentions'] = array('@id' => $id);
 			} else {
-				$video_jsonld['mentions'] = array_merge( $video_jsonld['mentions'], array( '@id' => $id ) );
+				$video_jsonld['mentions'] = array_merge($video_jsonld['mentions'], array('@id' => $id));
 			}
 		}
 
-		return array_merge( $jsonld, $videos_jsonld );
+		return array_merge($jsonld, $videos_jsonld);
 	}
 
 	/**
@@ -73,32 +73,32 @@ 
 	 *
 	 * @return array
 	 */
-	private function merge_video_data( $existing_video_data, $new_video_data ) {
-		if ( ! is_array( $existing_video_data ) ) {
+	private function merge_video_data($existing_video_data, $new_video_data) {
+		if ( ! is_array($existing_video_data)) {
 			$new_video_data[] = $existing_video_data;
 
 			return $new_video_data;
 		}
 
-		if ( $this->is_associative_array( $existing_video_data ) ) {
+		if ($this->is_associative_array($existing_video_data)) {
 			$new_video_data[] = $existing_video_data;
 
 			return $new_video_data;
 		}
 
-		return array_merge( $existing_video_data, $new_video_data );
+		return array_merge($existing_video_data, $new_video_data);
 	}
 
-	public function wl_post_jsonld( $jsonld, $post_id, $references ) {
+	public function wl_post_jsonld($jsonld, $post_id, $references) {
 
-		$video_jsonld = $this->get_videos_jsonld( $post_id );
-		if ( count( $video_jsonld ) === 0 ) {
+		$video_jsonld = $this->get_videos_jsonld($post_id);
+		if (count($video_jsonld) === 0) {
 			return $jsonld;
 		}
 		// Before adding the video jsonld check if the key
 		// is present and additional data might be present,
 		// if not present just add the data and return early.
-		if ( ! array_key_exists( 'video', $jsonld ) ) {
+		if ( ! array_key_exists('video', $jsonld)) {
 			$jsonld['video'] = $video_jsonld;
 
 			return $jsonld;
@@ -106,7 +106,7 @@ 
 
 		// since key exists, we need to merge the data based on type.
 		$previous_video_data = $jsonld['video'];
-		$jsonld['video']     = $this->merge_video_data( $previous_video_data, $video_jsonld );
+		$jsonld['video']     = $this->merge_video_data($previous_video_data, $video_jsonld);
 
 		return $jsonld;
 	}
@@ -117,18 +117,18 @@ 
 	 *
 	 * @return array
 	 */
-	public function get_videos_jsonld( $post_id ) {
+	public function get_videos_jsonld($post_id) {
 
-		$videos = $this->video_storage->get_all_videos( $post_id );
+		$videos = $this->video_storage->get_all_videos($post_id);
 
 		$jsonld = array();
 
-		foreach ( $videos as $video ) {
+		foreach ($videos as $video) {
 			/**
 			 * @var $video Video
 			 */
 			$description = $video->description;
-			if ( ! $video->description ) {
+			if ( ! $video->description) {
 				// If description is empty then use the video title as description
 				$description = $video->name;
 			}
@@ -144,7 +144,7 @@ 
 				'duration'     => $video->duration,
 			);
 
-			if ( $video->views ) {
+			if ($video->views) {
 				$single_jsonld['interactionStatistic'] = array(
 					'@type'                => 'InteractionCounter',
 					'interactionType'      => array(
@@ -154,7 +154,7 @@ 
 				);
 			}
 
-			if ( $video->is_live_video ) {
+			if ($video->is_live_video) {
 				$single_jsonld['publication'] = array(
 					'@type'           => 'BroadcastEvent',
 					'isLiveBroadcast' => true,
@@ -170,12 +170,12 @@ 
 	}
 
 
-	private function is_associative_array( $arr ) {
-		if ( array() === $arr ) {
+	private function is_associative_array($arr) {
+		if (array() === $arr) {
 			return false;
 		}
 
-		return array_keys( $arr ) !== range( 0, count( $arr ) - 1 );
+		return array_keys($arr) !== range(0, count($arr) - 1);
 	}