insideout10 / wordlift-plugin

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

Indentation +173 added lines, -173 removed lines

@@ -12,179 +12,179 @@
 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 = array( $type );
-		}
-		// If its a article or descendant of article, then dont add the
-		// videoobject in this hook, they will be already added to video property.
-		if ( array_intersect( Jsonld_Article_Wrapper::$article_types, $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,
-				'uploadDate'   => $video->upload_date,
-				'thumbnailUrl' => $video->thumbnail_urls,
-				'duration'     => $video->duration,
-			);
-
-			if ( $video->embed_url ) {
-				$single_jsonld['embedUrl'] = $video->embed_url;
-			}
-
-			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 = array( $type );
+        }
+        // If its a article or descendant of article, then dont add the
+        // videoobject in this hook, they will be already added to video property.
+        if ( array_intersect( Jsonld_Article_Wrapper::$article_types, $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,
+                'uploadDate'   => $video->upload_date,
+                'thumbnailUrl' => $video->thumbnail_urls,
+                'duration'     => $video->duration,
+            );
+
+            if ( $video->embed_url ) {
+                $single_jsonld['embedUrl'] = $video->embed_url;
+            }
+
+            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 +37 added lines, -37 removed lines

@@ -22,54 +22,54 @@ 
 	 *
 	 * @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 = array( $type );
+		if (is_string($type)) {
+			$type = array($type);
 		}
 		// If its a article or descendant of article, then dont add the
 		// videoobject in this hook, they will be already added to video property.
-		if ( array_intersect( Jsonld_Article_Wrapper::$article_types, $type ) ) {
+		if (array_intersect(Jsonld_Article_Wrapper::$article_types, $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);
 	}
 
 	/**
@@ -78,32 +78,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;
@@ -111,7 +111,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;
 	}
@@ -122,18 +122,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;
 			}
@@ -148,11 +148,11 @@ 
 				'duration'     => $video->duration,
 			);
 
-			if ( $video->embed_url ) {
+			if ($video->embed_url) {
 				$single_jsonld['embedUrl'] = $video->embed_url;
 			}
 
-			if ( $video->views ) {
+			if ($video->views) {
 				$single_jsonld['interactionStatistic'] = array(
 					'@type'                => 'InteractionCounter',
 					'interactionType'      => array(
@@ -162,7 +162,7 @@ 
 				);
 			}
 
-			if ( $video->is_live_video ) {
+			if ($video->is_live_video) {
 				$single_jsonld['publication'] = array(
 					'@type'           => 'BroadcastEvent',
 					'isLiveBroadcast' => true,
@@ -178,12 +178,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);
 	}
 
 

src/wordlift/videoobject/filters/class-jw-player-capture.php

Indentation +21 added lines, -21 removed lines

@@ -13,31 +13,31 @@
 
 class Jw_Player_Capture {
 
-	public function init() {
-		add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
-	}
+    public function init() {
+        add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
+    }
 
 
-	public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
-		// we cant reliably determine count for external plugin without
-		// this method.
-		global $wpdb;
-		$post_meta_table_name = $wpdb->postmeta;
-		$query_template = <<<EOF
+    public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
+        // we cant reliably determine count for external plugin without
+        // this method.
+        global $wpdb;
+        $post_meta_table_name = $wpdb->postmeta;
+        $query_template = <<<EOF
 SELECT meta_value FROM $post_meta_table_name WHERE meta_key LIKE '_jwppp-video-url-%' AND post_id=%d
 EOF;
-		$query                = $wpdb->prepare( $query_template, $post_id );
-		$video_ids            = $wpdb->get_col( $query );
-		if ( ! $video_ids ) {
-			return array();
-		}
-
-		$jw_player_videos = array_map( function ( $video_id ) {
-			return new Default_Embedded_Video( 'https://cdn.jwplayer.com/v2/media/' . $video_id );
-		}, $video_ids );
-
-		return array_merge( $embedded_videos, $jw_player_videos );
-	}
+        $query                = $wpdb->prepare( $query_template, $post_id );
+        $video_ids            = $wpdb->get_col( $query );
+        if ( ! $video_ids ) {
+            return array();
+        }
+
+        $jw_player_videos = array_map( function ( $video_id ) {
+            return new Default_Embedded_Video( 'https://cdn.jwplayer.com/v2/media/' . $video_id );
+        }, $video_ids );
+
+        return array_merge( $embedded_videos, $jw_player_videos );
+    }
 
 
 }
\ No newline at end of file

Spacing +9 added lines, -9 removed lines

@@ -14,11 +14,11 @@ 
 class Jw_Player_Capture {
 
 	public function init() {
-		add_filter( 'wl_videoobject_embedded_videos', array( $this, 'wl_videoobject_embedded_videos' ), 10, 2 );
+		add_filter('wl_videoobject_embedded_videos', array($this, 'wl_videoobject_embedded_videos'), 10, 2);
 	}
 
 
-	public function wl_videoobject_embedded_videos( $embedded_videos, $post_id ) {
+	public function wl_videoobject_embedded_videos($embedded_videos, $post_id) {
 		// we cant reliably determine count for external plugin without
 		// this method.
 		global $wpdb;
@@ -26,17 +26,17 @@ 
 		$query_template = <<<EOF
 SELECT meta_value FROM $post_meta_table_name WHERE meta_key LIKE '_jwppp-video-url-%' AND post_id=%d
 EOF;
-		$query                = $wpdb->prepare( $query_template, $post_id );
-		$video_ids            = $wpdb->get_col( $query );
-		if ( ! $video_ids ) {
+		$query                = $wpdb->prepare($query_template, $post_id);
+		$video_ids            = $wpdb->get_col($query);
+		if ( ! $video_ids) {
 			return array();
 		}
 
-		$jw_player_videos = array_map( function ( $video_id ) {
-			return new Default_Embedded_Video( 'https://cdn.jwplayer.com/v2/media/' . $video_id );
-		}, $video_ids );
+		$jw_player_videos = array_map(function($video_id) {
+			return new Default_Embedded_Video('https://cdn.jwplayer.com/v2/media/'.$video_id);
+		}, $video_ids);
 
-		return array_merge( $embedded_videos, $jw_player_videos );
+		return array_merge($embedded_videos, $jw_player_videos);
 	}
 
 

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

Indentation +221 added lines, -221 removed lines

@@ -14,227 +14,227 @@
 
 class Video_Processor {
 
-	/**
-	 * @var array<Client>
-	 */
-	private $api_clients;
-
-	public function __construct() {
-		$this->api_clients = array(
-			Client_Factory::get_client( Client_Factory::YOUTUBE ),
-			Client_Factory::get_client( Client_Factory::VIMEO ),
-			Client_Factory::get_client( Client_Factory::JWPLAYER )
-		);
-	}
-
-	private function get_data_for_videos( $embedded_videos, $post_id ) {
-
-		$youtube_videos   = $this->get_youtube_videos( $embedded_videos );
-		$youtube_provider = Provider_Factory::get_provider( Provider_Factory::YOUTUBE );
-		$youtube_videos   = $youtube_provider->get_videos_data( $youtube_videos );
-
-		$vimeo_videos   = $this->get_vimeo_videos( $embedded_videos );
-		$vimeo_provider = Provider_Factory::get_provider( Provider_Factory::VIMEO );
-		$vimeo_videos   = $vimeo_provider->get_videos_data( $vimeo_videos );
-
-		$jwplayer_videos   = $this->get_jw_player_videos( $embedded_videos );
-		$jwplayer_provider = Provider_Factory::get_provider( Provider_Factory::JWPLAYER );
-		$jwplayer_videos   = $jwplayer_provider->get_videos_data( $jwplayer_videos );
-
-		return array_merge( $youtube_videos, $vimeo_videos, $jwplayer_videos );
-
-	}
-
-
-	private function get_video_ids( $video_urls ) {
-		$clients   = $this->api_clients;
-		$video_ids = array();
-		foreach ( $clients as $client ) {
-			$ids = $client->get_video_ids( $video_urls );
-			if ( $ids ) {
-				$video_ids = array_merge( $video_ids, $ids );
-			}
-		}
-
-		return array_unique( $video_ids );
-	}
-
-
-	/**
-	 * @param $storage Storage
-	 * @param $post_id int
-	 * @param $embedded_videos array<Embedded_Video>
-	 */
-	private function remove_videos_from_store_if_not_present_in_content( $storage, $post_id, $embedded_videos ) {
-
-		$videos_to_be_deleted = $this->get_videos_to_be_deleted( $storage, $post_id, $embedded_videos );
-		$storage->remove_videos( $videos_to_be_deleted, $post_id );
-
-	}
-
-	/**
-	 * @param Storage $storage
-	 * @param $post_id
-	 * @param array $embedded_videos
-	 *
-	 * @return array An array of videos which exist on storage, not on post content.
-	 */
-	private function get_videos_to_be_deleted( Storage $storage, $post_id, array $embedded_videos ) {
-		$videos_in_store     = $storage->get_all_videos( $post_id );
-		$embedded_video_urls = array_map( function ( $embedded_video ) {
-			/**
-			 * @var $embedded_video Embedded_Video
-			 */
-			return $embedded_video->get_url();
-		}, $embedded_videos );
-
-		/**
-		 * Previously we are checking if the captured url is content_url to delete it from the storage
-		 * but this might not work well if we want to support multiple URL formats, we extract the video
-		 * ids for embedded URLs.
-		 */
-		$embedded_video_ids = $this->get_video_ids( $embedded_video_urls );
-
-		$that = $this;
-
-		return array_filter( $videos_in_store, function ( $video ) use ( $embedded_video_ids, $embedded_video_urls, $that ) {
-			/**
-			 * If the video id doesnt exist on the content then we need to return it
-			 * in order to delete that video.
-			 */
-			return count( array_intersect(
-					$that->get_video_ids( array( $video->id ) ),
-					$embedded_video_ids
-				) ) === 0;
-
-		} );
-	}
-
-	/**
-	 * @param \WP_Post $post
-	 * @param $post_id
-	 */
-	public function process_video_urls( \WP_Post $post, $post_id ) {
-
-		$parser = Parser_Factory::get_parser_from_content( $post->post_content );
-
-		$embedded_videos = $parser->get_videos( $post_id );
-		/**
-		 * Filters the embedded videos on post contet, custom plugins can add their video urls
-		 * by constructing \Default_Embedded_Video or implement Embedded_Video class
-		 *
-		 * @param $embedded_videos array<Embedded_Video>
-		 * @param $post_id int The post id for the videoobject is processed.
-		 *
-		 * @return array<Embedded_Video>
-		 * @since 3.31.4
-		 * Filter name : wl_videoobject_embedded_videos
-		 */
-		$embedded_videos = apply_filters( 'wl_videoobject_embedded_videos', $embedded_videos, $post_id );
-
-		$storage = Video_Storage_Factory::get_storage();
-
-		// Before sending api requests we need to check if there are any videos in
-		// store which is not present on post content, remove them if there are
-		// any
-		$this->remove_videos_from_store_if_not_present_in_content( $storage, $post_id, $embedded_videos );
-
-		$embedded_videos = $this->get_videos_without_existing_data( $storage, $post_id, $embedded_videos );
-
-		// Return early after removing all the videos.
-		if ( ! $embedded_videos ) {
-			return;
-		}
-
-		$videos = $this->get_data_for_videos( $embedded_videos, $post_id );
-
-		if ( ! $videos ) {
-			return;
-		}
-
-		foreach ( $videos as $video ) {
-			$storage->add_video( $post_id, $video );
-		}
-	}
-
-	/**
-	 * @param $storage Storage
-	 * @param $post_id int
-	 * @param $embedded_videos array<Embedded_Video>
-	 *
-	 * @return array<Embedded_Video> Return array of embedded videos which are not in store.
-	 */
-	private function get_videos_without_existing_data( $storage, $post_id, $embedded_videos ) {
-		$videos_in_store     = $storage->get_all_videos( $post_id );
-		$video_urls_in_store = array_map( function ( $video ) {
-			return $video->id;
-		}, $videos_in_store );
-
-		$video_ids_in_store = $this->get_video_ids( $video_urls_in_store );
-
-		$that = $this;
-
-		return array_filter( $embedded_videos, function ( $embedded_video ) use ( $video_ids_in_store, $that ) {
-			/**
-			 * If the video id exist on content, not on storage then
-			 * we need to fetch the data.
-			 */
-			return count( array_intersect(
-					$that->get_video_ids( array( $embedded_video->get_url() ) ),
-					$video_ids_in_store
-				) ) === 0;
-		} );
-	}
-
-	/**
-	 * @param $embedded_videos
-	 *
-	 * @return array
-	 */
-	private function get_youtube_videos( $embedded_videos ) {
-		return array_filter( $embedded_videos, function ( $embedded_video ) {
-			/**
-			 * it should have youtube.com or youtu.be in the url
-			 *
-			 * @param $embedded_video Embedded_Video
-			 */
-			$video_url = $embedded_video->get_url();
-
-			return strpos( $video_url, "youtube.com" ) !== false ||
-			       strpos( $video_url, "youtu.be" ) !== false;
-		} );
-	}
-
-	/**
-	 * @param $embedded_videos
-	 *
-	 * @return array
-	 */
-	private function get_vimeo_videos( $embedded_videos ) {
-		return array_filter( $embedded_videos, function ( $embedded_video ) {
-			/**
-			 * it should have vimeo.com in the url
-			 *
-			 * @param $embedded_video Embedded_Video
-			 */
-			$video_url = $embedded_video->get_url();
-
-			return strpos( $video_url, "vimeo.com" ) !== false;
-		} );
-	}
-
-	private function get_jw_player_videos( $embedded_videos ) {
-		return array_filter( $embedded_videos, function ( $embedded_video ) {
-			/**
-			 * it should have vimeo.com in the url
-			 *
-			 * @param $embedded_video Embedded_Video
-			 */
-			$video_url = $embedded_video->get_url();
-
-			return strpos( $video_url, 'https://cdn.jwplayer.com/v2/media/', 0 ) !== false;
-		} );
-	}
+    /**
+     * @var array<Client>
+     */
+    private $api_clients;
+
+    public function __construct() {
+        $this->api_clients = array(
+            Client_Factory::get_client( Client_Factory::YOUTUBE ),
+            Client_Factory::get_client( Client_Factory::VIMEO ),
+            Client_Factory::get_client( Client_Factory::JWPLAYER )
+        );
+    }
+
+    private function get_data_for_videos( $embedded_videos, $post_id ) {
+
+        $youtube_videos   = $this->get_youtube_videos( $embedded_videos );
+        $youtube_provider = Provider_Factory::get_provider( Provider_Factory::YOUTUBE );
+        $youtube_videos   = $youtube_provider->get_videos_data( $youtube_videos );
+
+        $vimeo_videos   = $this->get_vimeo_videos( $embedded_videos );
+        $vimeo_provider = Provider_Factory::get_provider( Provider_Factory::VIMEO );
+        $vimeo_videos   = $vimeo_provider->get_videos_data( $vimeo_videos );
+
+        $jwplayer_videos   = $this->get_jw_player_videos( $embedded_videos );
+        $jwplayer_provider = Provider_Factory::get_provider( Provider_Factory::JWPLAYER );
+        $jwplayer_videos   = $jwplayer_provider->get_videos_data( $jwplayer_videos );
+
+        return array_merge( $youtube_videos, $vimeo_videos, $jwplayer_videos );
+
+    }
+
+
+    private function get_video_ids( $video_urls ) {
+        $clients   = $this->api_clients;
+        $video_ids = array();
+        foreach ( $clients as $client ) {
+            $ids = $client->get_video_ids( $video_urls );
+            if ( $ids ) {
+                $video_ids = array_merge( $video_ids, $ids );
+            }
+        }
+
+        return array_unique( $video_ids );
+    }
+
+
+    /**
+     * @param $storage Storage
+     * @param $post_id int
+     * @param $embedded_videos array<Embedded_Video>
+     */
+    private function remove_videos_from_store_if_not_present_in_content( $storage, $post_id, $embedded_videos ) {
+
+        $videos_to_be_deleted = $this->get_videos_to_be_deleted( $storage, $post_id, $embedded_videos );
+        $storage->remove_videos( $videos_to_be_deleted, $post_id );
+
+    }
+
+    /**
+     * @param Storage $storage
+     * @param $post_id
+     * @param array $embedded_videos
+     *
+     * @return array An array of videos which exist on storage, not on post content.
+     */
+    private function get_videos_to_be_deleted( Storage $storage, $post_id, array $embedded_videos ) {
+        $videos_in_store     = $storage->get_all_videos( $post_id );
+        $embedded_video_urls = array_map( function ( $embedded_video ) {
+            /**
+             * @var $embedded_video Embedded_Video
+             */
+            return $embedded_video->get_url();
+        }, $embedded_videos );
+
+        /**
+         * Previously we are checking if the captured url is content_url to delete it from the storage
+         * but this might not work well if we want to support multiple URL formats, we extract the video
+         * ids for embedded URLs.
+         */
+        $embedded_video_ids = $this->get_video_ids( $embedded_video_urls );
+
+        $that = $this;
+
+        return array_filter( $videos_in_store, function ( $video ) use ( $embedded_video_ids, $embedded_video_urls, $that ) {
+            /**
+             * If the video id doesnt exist on the content then we need to return it
+             * in order to delete that video.
+             */
+            return count( array_intersect(
+                    $that->get_video_ids( array( $video->id ) ),
+                    $embedded_video_ids
+                ) ) === 0;
+
+        } );
+    }
+
+    /**
+     * @param \WP_Post $post
+     * @param $post_id
+     */
+    public function process_video_urls( \WP_Post $post, $post_id ) {
+
+        $parser = Parser_Factory::get_parser_from_content( $post->post_content );
+
+        $embedded_videos = $parser->get_videos( $post_id );
+        /**
+         * Filters the embedded videos on post contet, custom plugins can add their video urls
+         * by constructing \Default_Embedded_Video or implement Embedded_Video class
+         *
+         * @param $embedded_videos array<Embedded_Video>
+         * @param $post_id int The post id for the videoobject is processed.
+         *
+         * @return array<Embedded_Video>
+         * @since 3.31.4
+         * Filter name : wl_videoobject_embedded_videos
+         */
+        $embedded_videos = apply_filters( 'wl_videoobject_embedded_videos', $embedded_videos, $post_id );
+
+        $storage = Video_Storage_Factory::get_storage();
+
+        // Before sending api requests we need to check if there are any videos in
+        // store which is not present on post content, remove them if there are
+        // any
+        $this->remove_videos_from_store_if_not_present_in_content( $storage, $post_id, $embedded_videos );
+
+        $embedded_videos = $this->get_videos_without_existing_data( $storage, $post_id, $embedded_videos );
+
+        // Return early after removing all the videos.
+        if ( ! $embedded_videos ) {
+            return;
+        }
+
+        $videos = $this->get_data_for_videos( $embedded_videos, $post_id );
+
+        if ( ! $videos ) {
+            return;
+        }
+
+        foreach ( $videos as $video ) {
+            $storage->add_video( $post_id, $video );
+        }
+    }
+
+    /**
+     * @param $storage Storage
+     * @param $post_id int
+     * @param $embedded_videos array<Embedded_Video>
+     *
+     * @return array<Embedded_Video> Return array of embedded videos which are not in store.
+     */
+    private function get_videos_without_existing_data( $storage, $post_id, $embedded_videos ) {
+        $videos_in_store     = $storage->get_all_videos( $post_id );
+        $video_urls_in_store = array_map( function ( $video ) {
+            return $video->id;
+        }, $videos_in_store );
+
+        $video_ids_in_store = $this->get_video_ids( $video_urls_in_store );
+
+        $that = $this;
+
+        return array_filter( $embedded_videos, function ( $embedded_video ) use ( $video_ids_in_store, $that ) {
+            /**
+             * If the video id exist on content, not on storage then
+             * we need to fetch the data.
+             */
+            return count( array_intersect(
+                    $that->get_video_ids( array( $embedded_video->get_url() ) ),
+                    $video_ids_in_store
+                ) ) === 0;
+        } );
+    }
+
+    /**
+     * @param $embedded_videos
+     *
+     * @return array
+     */
+    private function get_youtube_videos( $embedded_videos ) {
+        return array_filter( $embedded_videos, function ( $embedded_video ) {
+            /**
+             * it should have youtube.com or youtu.be in the url
+             *
+             * @param $embedded_video Embedded_Video
+             */
+            $video_url = $embedded_video->get_url();
+
+            return strpos( $video_url, "youtube.com" ) !== false ||
+                   strpos( $video_url, "youtu.be" ) !== false;
+        } );
+    }
+
+    /**
+     * @param $embedded_videos
+     *
+     * @return array
+     */
+    private function get_vimeo_videos( $embedded_videos ) {
+        return array_filter( $embedded_videos, function ( $embedded_video ) {
+            /**
+             * it should have vimeo.com in the url
+             *
+             * @param $embedded_video Embedded_Video
+             */
+            $video_url = $embedded_video->get_url();
+
+            return strpos( $video_url, "vimeo.com" ) !== false;
+        } );
+    }
+
+    private function get_jw_player_videos( $embedded_videos ) {
+        return array_filter( $embedded_videos, function ( $embedded_video ) {
+            /**
+             * it should have vimeo.com in the url
+             *
+             * @param $embedded_video Embedded_Video
+             */
+            $video_url = $embedded_video->get_url();
+
+            return strpos( $video_url, 'https://cdn.jwplayer.com/v2/media/', 0 ) !== false;
+        } );
+    }
 
 
 }
\ No newline at end of file

Spacing +62 added lines, -62 removed lines

@@ -21,42 +21,42 @@ 
 
 	public function __construct() {
 		$this->api_clients = array(
-			Client_Factory::get_client( Client_Factory::YOUTUBE ),
-			Client_Factory::get_client( Client_Factory::VIMEO ),
-			Client_Factory::get_client( Client_Factory::JWPLAYER )
+			Client_Factory::get_client(Client_Factory::YOUTUBE),
+			Client_Factory::get_client(Client_Factory::VIMEO),
+			Client_Factory::get_client(Client_Factory::JWPLAYER)
 		);
 	}
 
-	private function get_data_for_videos( $embedded_videos, $post_id ) {
+	private function get_data_for_videos($embedded_videos, $post_id) {
 
-		$youtube_videos   = $this->get_youtube_videos( $embedded_videos );
-		$youtube_provider = Provider_Factory::get_provider( Provider_Factory::YOUTUBE );
-		$youtube_videos   = $youtube_provider->get_videos_data( $youtube_videos );
+		$youtube_videos   = $this->get_youtube_videos($embedded_videos);
+		$youtube_provider = Provider_Factory::get_provider(Provider_Factory::YOUTUBE);
+		$youtube_videos   = $youtube_provider->get_videos_data($youtube_videos);
 
-		$vimeo_videos   = $this->get_vimeo_videos( $embedded_videos );
-		$vimeo_provider = Provider_Factory::get_provider( Provider_Factory::VIMEO );
-		$vimeo_videos   = $vimeo_provider->get_videos_data( $vimeo_videos );
+		$vimeo_videos   = $this->get_vimeo_videos($embedded_videos);
+		$vimeo_provider = Provider_Factory::get_provider(Provider_Factory::VIMEO);
+		$vimeo_videos   = $vimeo_provider->get_videos_data($vimeo_videos);
 
-		$jwplayer_videos   = $this->get_jw_player_videos( $embedded_videos );
-		$jwplayer_provider = Provider_Factory::get_provider( Provider_Factory::JWPLAYER );
-		$jwplayer_videos   = $jwplayer_provider->get_videos_data( $jwplayer_videos );
+		$jwplayer_videos   = $this->get_jw_player_videos($embedded_videos);
+		$jwplayer_provider = Provider_Factory::get_provider(Provider_Factory::JWPLAYER);
+		$jwplayer_videos   = $jwplayer_provider->get_videos_data($jwplayer_videos);
 
-		return array_merge( $youtube_videos, $vimeo_videos, $jwplayer_videos );
+		return array_merge($youtube_videos, $vimeo_videos, $jwplayer_videos);
 
 	}
 
 
-	private function get_video_ids( $video_urls ) {
+	private function get_video_ids($video_urls) {
 		$clients   = $this->api_clients;
 		$video_ids = array();
-		foreach ( $clients as $client ) {
-			$ids = $client->get_video_ids( $video_urls );
-			if ( $ids ) {
-				$video_ids = array_merge( $video_ids, $ids );
+		foreach ($clients as $client) {
+			$ids = $client->get_video_ids($video_urls);
+			if ($ids) {
+				$video_ids = array_merge($video_ids, $ids);
 			}
 		}
 
-		return array_unique( $video_ids );
+		return array_unique($video_ids);
 	}
 
 
@@ -65,10 +65,10 @@ 
 	 * @param $post_id int
 	 * @param $embedded_videos array<Embedded_Video>
 	 */
-	private function remove_videos_from_store_if_not_present_in_content( $storage, $post_id, $embedded_videos ) {
+	private function remove_videos_from_store_if_not_present_in_content($storage, $post_id, $embedded_videos) {
 
-		$videos_to_be_deleted = $this->get_videos_to_be_deleted( $storage, $post_id, $embedded_videos );
-		$storage->remove_videos( $videos_to_be_deleted, $post_id );
+		$videos_to_be_deleted = $this->get_videos_to_be_deleted($storage, $post_id, $embedded_videos);
+		$storage->remove_videos($videos_to_be_deleted, $post_id);
 
 	}
 
@@ -79,33 +79,33 @@ 
 	 *
 	 * @return array An array of videos which exist on storage, not on post content.
 	 */
-	private function get_videos_to_be_deleted( Storage $storage, $post_id, array $embedded_videos ) {
-		$videos_in_store     = $storage->get_all_videos( $post_id );
-		$embedded_video_urls = array_map( function ( $embedded_video ) {
+	private function get_videos_to_be_deleted(Storage $storage, $post_id, array $embedded_videos) {
+		$videos_in_store     = $storage->get_all_videos($post_id);
+		$embedded_video_urls = array_map(function($embedded_video) {
 			/**
 			 * @var $embedded_video Embedded_Video
 			 */
 			return $embedded_video->get_url();
-		}, $embedded_videos );
+		}, $embedded_videos);
 
 		/**
 		 * Previously we are checking if the captured url is content_url to delete it from the storage
 		 * but this might not work well if we want to support multiple URL formats, we extract the video
 		 * ids for embedded URLs.
 		 */
-		$embedded_video_ids = $this->get_video_ids( $embedded_video_urls );
+		$embedded_video_ids = $this->get_video_ids($embedded_video_urls);
 
 		$that = $this;
 
-		return array_filter( $videos_in_store, function ( $video ) use ( $embedded_video_ids, $embedded_video_urls, $that ) {
+		return array_filter($videos_in_store, function($video) use ($embedded_video_ids, $embedded_video_urls, $that) {
 			/**
 			 * If the video id doesnt exist on the content then we need to return it
 			 * in order to delete that video.
 			 */
-			return count( array_intersect(
-					$that->get_video_ids( array( $video->id ) ),
+			return count(array_intersect(
+					$that->get_video_ids(array($video->id)),
 					$embedded_video_ids
-				) ) === 0;
+				)) === 0;
 
 		} );
 	}
@@ -114,11 +114,11 @@ 
 	 * @param \WP_Post $post
 	 * @param $post_id
 	 */
-	public function process_video_urls( \WP_Post $post, $post_id ) {
+	public function process_video_urls(\WP_Post $post, $post_id) {
 
-		$parser = Parser_Factory::get_parser_from_content( $post->post_content );
+		$parser = Parser_Factory::get_parser_from_content($post->post_content);
 
-		$embedded_videos = $parser->get_videos( $post_id );
+		$embedded_videos = $parser->get_videos($post_id);
 		/**
 		 * Filters the embedded videos on post contet, custom plugins can add their video urls
 		 * by constructing \Default_Embedded_Video or implement Embedded_Video class
@@ -130,30 +130,30 @@ 
 		 * @since 3.31.4
 		 * Filter name : wl_videoobject_embedded_videos
 		 */
-		$embedded_videos = apply_filters( 'wl_videoobject_embedded_videos', $embedded_videos, $post_id );
+		$embedded_videos = apply_filters('wl_videoobject_embedded_videos', $embedded_videos, $post_id);
 
 		$storage = Video_Storage_Factory::get_storage();
 
 		// Before sending api requests we need to check if there are any videos in
 		// store which is not present on post content, remove them if there are
 		// any
-		$this->remove_videos_from_store_if_not_present_in_content( $storage, $post_id, $embedded_videos );
+		$this->remove_videos_from_store_if_not_present_in_content($storage, $post_id, $embedded_videos);
 
-		$embedded_videos = $this->get_videos_without_existing_data( $storage, $post_id, $embedded_videos );
+		$embedded_videos = $this->get_videos_without_existing_data($storage, $post_id, $embedded_videos);
 
 		// Return early after removing all the videos.
-		if ( ! $embedded_videos ) {
+		if ( ! $embedded_videos) {
 			return;
 		}
 
-		$videos = $this->get_data_for_videos( $embedded_videos, $post_id );
+		$videos = $this->get_data_for_videos($embedded_videos, $post_id);
 
-		if ( ! $videos ) {
+		if ( ! $videos) {
 			return;
 		}
 
-		foreach ( $videos as $video ) {
-			$storage->add_video( $post_id, $video );
+		foreach ($videos as $video) {
+			$storage->add_video($post_id, $video);
 		}
 	}
 
@@ -164,25 +164,25 @@ 
 	 *
 	 * @return array<Embedded_Video> Return array of embedded videos which are not in store.
 	 */
-	private function get_videos_without_existing_data( $storage, $post_id, $embedded_videos ) {
-		$videos_in_store     = $storage->get_all_videos( $post_id );
-		$video_urls_in_store = array_map( function ( $video ) {
+	private function get_videos_without_existing_data($storage, $post_id, $embedded_videos) {
+		$videos_in_store     = $storage->get_all_videos($post_id);
+		$video_urls_in_store = array_map(function($video) {
 			return $video->id;
-		}, $videos_in_store );
+		}, $videos_in_store);
 
-		$video_ids_in_store = $this->get_video_ids( $video_urls_in_store );
+		$video_ids_in_store = $this->get_video_ids($video_urls_in_store);
 
 		$that = $this;
 
-		return array_filter( $embedded_videos, function ( $embedded_video ) use ( $video_ids_in_store, $that ) {
+		return array_filter($embedded_videos, function($embedded_video) use ($video_ids_in_store, $that) {
 			/**
 			 * If the video id exist on content, not on storage then
 			 * we need to fetch the data.
 			 */
-			return count( array_intersect(
-					$that->get_video_ids( array( $embedded_video->get_url() ) ),
+			return count(array_intersect(
+					$that->get_video_ids(array($embedded_video->get_url())),
 					$video_ids_in_store
-				) ) === 0;
+				)) === 0;
 		} );
 	}
 
@@ -191,8 +191,8 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_youtube_videos( $embedded_videos ) {
-		return array_filter( $embedded_videos, function ( $embedded_video ) {
+	private function get_youtube_videos($embedded_videos) {
+		return array_filter($embedded_videos, function($embedded_video) {
 			/**
 			 * it should have youtube.com or youtu.be in the url
 			 *
@@ -200,8 +200,8 @@ 
 			 */
 			$video_url = $embedded_video->get_url();
 
-			return strpos( $video_url, "youtube.com" ) !== false ||
-			       strpos( $video_url, "youtu.be" ) !== false;
+			return strpos($video_url, "youtube.com") !== false ||
+			       strpos($video_url, "youtu.be") !== false;
 		} );
 	}
 
@@ -210,8 +210,8 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_vimeo_videos( $embedded_videos ) {
-		return array_filter( $embedded_videos, function ( $embedded_video ) {
+	private function get_vimeo_videos($embedded_videos) {
+		return array_filter($embedded_videos, function($embedded_video) {
 			/**
 			 * it should have vimeo.com in the url
 			 *
@@ -219,12 +219,12 @@ 
 			 */
 			$video_url = $embedded_video->get_url();
 
-			return strpos( $video_url, "vimeo.com" ) !== false;
+			return strpos($video_url, "vimeo.com") !== false;
 		} );
 	}
 
-	private function get_jw_player_videos( $embedded_videos ) {
-		return array_filter( $embedded_videos, function ( $embedded_video ) {
+	private function get_jw_player_videos($embedded_videos) {
+		return array_filter($embedded_videos, function($embedded_video) {
 			/**
 			 * it should have vimeo.com in the url
 			 *
@@ -232,7 +232,7 @@ 
 			 */
 			$video_url = $embedded_video->get_url();
 
-			return strpos( $video_url, 'https://cdn.jwplayer.com/v2/media/', 0 ) !== false;
+			return strpos($video_url, 'https://cdn.jwplayer.com/v2/media/', 0) !== false;
 		} );
 	}
 

src/wordlift/videoobject/class-loader.php

Indentation +39 added lines, -39 removed lines

@@ -24,57 +24,57 @@
  */
 class Loader extends Default_Loader {
 
-	public function init_all_dependencies() {
-		$video_storage = Video_Storage_Factory::get_storage();
-		new Jsonld( $video_storage );
+    public function init_all_dependencies() {
+        $video_storage = Video_Storage_Factory::get_storage();
+        new Jsonld( $video_storage );
 
-		$sitemap_cache = new Ttl_Cache( "wl_video_sitemap", 86400 );
+        $sitemap_cache = new Ttl_Cache( "wl_video_sitemap", 86400 );
 
-		$video_processor = new Video_Processor();
-		// Hook in to save_post to save the videos
-		$post_filter = new Post_Filter( $video_processor );
-		$post_filter->init();
-		// Add entry to wordlift admin tabs
-		$settings_tab = new Settings_Tab();
-		$settings_tab->init();
+        $video_processor = new Video_Processor();
+        // Hook in to save_post to save the videos
+        $post_filter = new Post_Filter( $video_processor );
+        $post_filter->init();
+        // Add entry to wordlift admin tabs
+        $settings_tab = new Settings_Tab();
+        $settings_tab->init();
 
 
-		$video_sitemap = new Video_Sitemap( $sitemap_cache );
-		$video_sitemap->init();
+        $video_sitemap = new Video_Sitemap( $sitemap_cache );
+        $video_sitemap->init();
 
-		$background_process_data_source = new Videos_Data_Source( '__wl_videoobject_import_state' );
-		$background_process             = new Videoobject_Background_Process( $video_processor, $background_process_data_source );
+        $background_process_data_source = new Videos_Data_Source( '__wl_videoobject_import_state' );
+        $background_process             = new Videoobject_Background_Process( $video_processor, $background_process_data_source );
 
-		$rest_controller = new Rest_Controller( $background_process );
-		$rest_controller->register_all_routes();
+        $rest_controller = new Rest_Controller( $background_process );
+        $rest_controller->register_all_routes();
 
-		$post_edit_screen = new Post_Edit_Screen();
-		$post_edit_screen->init();
+        $post_edit_screen = new Post_Edit_Screen();
+        $post_edit_screen->init();
 
-		new Import_Videos_Page();
+        new Import_Videos_Page();
 
-		/**
-		 * @since 3.32.0
-		 * Allow videoobject to capture embed shortcode.
-		 */
-		$embed_shortcode_capture = new Embed_Shortcode_Capture();
-		$embed_shortcode_capture->init();
+        /**
+         * @since 3.32.0
+         * Allow videoobject to capture embed shortcode.
+         */
+        $embed_shortcode_capture = new Embed_Shortcode_Capture();
+        $embed_shortcode_capture->init();
 
 
-		/**
-		 * @since 3.32.0
-		 * Get videos from jw player.
-		 */
-		$jw_player_capture_videos = new Jw_Player_Capture();
-		$jw_player_capture_videos->init();
+        /**
+         * @since 3.32.0
+         * Get videos from jw player.
+         */
+        $jw_player_capture_videos = new Jw_Player_Capture();
+        $jw_player_capture_videos->init();
 
-	}
+    }
 
-	public function get_feature_slug() {
-		return 'videoobject';
-	}
+    public function get_feature_slug() {
+        return 'videoobject';
+    }
 
-	public function get_feature_default_value() {
-		return false;
-	}
+    public function get_feature_default_value() {
+        return false;
+    }
 }
\ No newline at end of file

src/wordlift/videoobject/provider/client/class-jw-player-client.php

Indentation +31 added lines, -31 removed lines

@@ -11,48 +11,48 @@
  */
 class Jw_Player_Client extends Singleton implements Client {
 
-	public function get_data( $video_urls ) {
+    public function get_data( $video_urls ) {
 
-		$videos_data = array();
+        $videos_data = array();
 
-		foreach ( $video_urls as $video_url ) {
-			$data = wp_remote_get( $video_url );
-			if ( is_wp_error( $data ) ) {
-				continue;
-			}
-			$json_body  = wp_remote_retrieve_body( $data );
+        foreach ( $video_urls as $video_url ) {
+            $data = wp_remote_get( $video_url );
+            if ( is_wp_error( $data ) ) {
+                continue;
+            }
+            $json_body  = wp_remote_retrieve_body( $data );
 
-			$video_data = json_decode( $json_body, true );
-			if ( ! $video_data ) {
-				continue;
-			}
-			$video_data['id'] = $video_url;
-			$videos_data[]    = $video_data;
-		}
+            $video_data = json_decode( $json_body, true );
+            if ( ! $video_data ) {
+                continue;
+            }
+            $video_data['id'] = $video_url;
+            $videos_data[]    = $video_data;
+        }
 
-		return $videos_data;
+        return $videos_data;
 
-	}
+    }
 
-	public function get_video_ids( $video_urls ) {
+    public function get_video_ids( $video_urls ) {
 
-		return array_filter( $video_urls, function ( $item ) {
-			return strpos( $item, 'https://cdn.jwplayer.com/v2/media/', 0 );
-		} );
+        return array_filter( $video_urls, function ( $item ) {
+            return strpos( $item, 'https://cdn.jwplayer.com/v2/media/', 0 );
+        } );
 
-	}
+    }
 
-	public static function get_api_key() {
-		// Method not implemented, since we dont need api key
-	}
+    public static function get_api_key() {
+        // Method not implemented, since we dont need api key
+    }
 
-	public static function get_api_key_option_name() {
-		// Method not implemented, since we dont need api key
-	}
+    public static function get_api_key_option_name() {
+        // Method not implemented, since we dont need api key
+    }
 
-	public function get_api_url() {
-		// Method not implemented, since we dont need api key
-	}
+    public function get_api_url() {
+        // Method not implemented, since we dont need api key
+    }
 
 
 }
\ No newline at end of file

Spacing +10 added lines, -10 removed lines

@@ -11,19 +11,19 @@ 
  */
 class Jw_Player_Client extends Singleton implements Client {
 
-	public function get_data( $video_urls ) {
+	public function get_data($video_urls) {
 
 		$videos_data = array();
 
-		foreach ( $video_urls as $video_url ) {
-			$data = wp_remote_get( $video_url );
-			if ( is_wp_error( $data ) ) {
+		foreach ($video_urls as $video_url) {
+			$data = wp_remote_get($video_url);
+			if (is_wp_error($data)) {
 				continue;
 			}
-			$json_body  = wp_remote_retrieve_body( $data );
+			$json_body  = wp_remote_retrieve_body($data);
 
-			$video_data = json_decode( $json_body, true );
-			if ( ! $video_data ) {
+			$video_data = json_decode($json_body, true);
+			if ( ! $video_data) {
 				continue;
 			}
 			$video_data['id'] = $video_url;
@@ -34,10 +34,10 @@ 
 
 	}
 
-	public function get_video_ids( $video_urls ) {
+	public function get_video_ids($video_urls) {
 
-		return array_filter( $video_urls, function ( $item ) {
-			return strpos( $item, 'https://cdn.jwplayer.com/v2/media/', 0 );
+		return array_filter($video_urls, function($item) {
+			return strpos($item, 'https://cdn.jwplayer.com/v2/media/', 0);
 		} );
 
 	}

src/wordlift/videoobject/provider/client/class-client-factory.php

Indentation +13 added lines, -13 removed lines

@@ -8,21 +8,21 @@
  */
 class Client_Factory {
 
-	const YOUTUBE = 'youtube';
+    const YOUTUBE = 'youtube';
 
-	const VIMEO = 'vimeo';
+    const VIMEO = 'vimeo';
 
-	const JWPLAYER = 'jwplayer';
+    const JWPLAYER = 'jwplayer';
 
-	public static function get_client( $config ) {
-		if ( self::YOUTUBE === $config ) {
-			return Youtube_Client::get_instance();
-		} else if ( self::VIMEO === $config ) {
-			return Vimeo_Client::get_instance();
-		}
-		else if ( self::JWPLAYER === $config ) {
-			return Jw_Player_Client::get_instance();
-		}
+    public static function get_client( $config ) {
+        if ( self::YOUTUBE === $config ) {
+            return Youtube_Client::get_instance();
+        } else if ( self::VIMEO === $config ) {
+            return Vimeo_Client::get_instance();
+        }
+        else if ( self::JWPLAYER === $config ) {
+            return Jw_Player_Client::get_instance();
+        }
 
-	}
+    }
 }
\ No newline at end of file

Spacing +4 added lines, -4 removed lines

@@ -14,13 +14,13 @@
 
 	const JWPLAYER = 'jwplayer';
 
-	public static function get_client( $config ) {
-		if ( self::YOUTUBE === $config ) {
+	public static function get_client($config) {
+		if (self::YOUTUBE === $config) {
 			return Youtube_Client::get_instance();
-		} else if ( self::VIMEO === $config ) {
+		} else if (self::VIMEO === $config) {
 			return Vimeo_Client::get_instance();
 		}
-		else if ( self::JWPLAYER === $config ) {
+		else if (self::JWPLAYER === $config) {
 			return Jw_Player_Client::get_instance();
 		}
 

Braces +1 added lines, -2 removed lines

@@ -19,8 +19,7 @@
 			return Youtube_Client::get_instance();
 		} else if ( self::VIMEO === $config ) {
 			return Vimeo_Client::get_instance();
-		}
-		else if ( self::JWPLAYER === $config ) {
+		} else if ( self::JWPLAYER === $config ) {
 			return Jw_Player_Client::get_instance();
 		}
 

src/wordlift/videoobject/provider/class-jw-player.php

Indentation +40 added lines, -40 removed lines

@@ -12,62 +12,62 @@
 class Jw_Player extends Api_Provider {
 
 
-	/**
-	 * @param $videos array<Embedded_Video>
-	 *
-	 * @return array<Video>
-	 */
-	public function get_videos_data( $videos ) {
+    /**
+     * @param $videos array<Embedded_Video>
+     *
+     * @return array<Video>
+     */
+    public function get_videos_data( $videos ) {
 
-		$video_urls = array_map( function ( $video ) {
-			return $video->get_url();
-		}, $videos );
+        $video_urls = array_map( function ( $video ) {
+            return $video->get_url();
+        }, $videos );
 
-		$videos_data = $this->api_client->get_data( $video_urls );
+        $videos_data = $this->api_client->get_data( $video_urls );
 
-		return array_filter( array_map( function ( $video_data ) {
+        return array_filter( array_map( function ( $video_data ) {
 
-			if ( ! array_key_exists( 'playlist', $video_data )
+            if ( ! array_key_exists( 'playlist', $video_data )
 
-			     || ! is_array( $video_data['playlist'] )
+                 || ! is_array( $video_data['playlist'] )
 
-			     || count( $video_data['playlist'] ) === 0 ) {
+                 || count( $video_data['playlist'] ) === 0 ) {
 
-				return false;
+                return false;
 
-			}
+            }
 
-			$video                 = new Video();
-			$video->id             = $video_data['id'];
-			$playlist_item_data    = $video_data['playlist'][0];
-			$video->name           = $playlist_item_data['title'];
-			$video->description    = $playlist_item_data['description'];
-			$video->thumbnail_urls = array_filter( array_map( function ( $item ) {
-				return $item['src'];
-			}, $playlist_item_data['images'] ) );
+            $video                 = new Video();
+            $video->id             = $video_data['id'];
+            $playlist_item_data    = $video_data['playlist'][0];
+            $video->name           = $playlist_item_data['title'];
+            $video->description    = $playlist_item_data['description'];
+            $video->thumbnail_urls = array_filter( array_map( function ( $item ) {
+                return $item['src'];
+            }, $playlist_item_data['images'] ) );
 
-			$video->duration    = "PT" . (int) $playlist_item_data['duration'] . "S";
-			$video->upload_date = date( 'c', (int) $playlist_item_data['pubdate'] );
+            $video->duration    = "PT" . (int) $playlist_item_data['duration'] . "S";
+            $video->upload_date = date( 'c', (int) $playlist_item_data['pubdate'] );
 
-			$video_content_urls_data = array_filter( $playlist_item_data['sources'], function ( $item ) {
-				return strpos( $item['type'], 'video/' ) !== false;
-			} );
+            $video_content_urls_data = array_filter( $playlist_item_data['sources'], function ( $item ) {
+                return strpos( $item['type'], 'video/' ) !== false;
+            } );
 
-			$video_content_urls = array_map( function ( $item ) {
-				return $item['file'];
-			}, $video_content_urls_data );
+            $video_content_urls = array_map( function ( $item ) {
+                return $item['file'];
+            }, $video_content_urls_data );
 
-			// Content url is a single field, so we pick the video with
-			// high resolution.
-			$video->content_url = array_pop( $video_content_urls );
+            // Content url is a single field, so we pick the video with
+            // high resolution.
+            $video->content_url = array_pop( $video_content_urls );
 
-			// We dont have embed url for JW Player.
-			$video->embed_url = '';
+            // We dont have embed url for JW Player.
+            $video->embed_url = '';
 
-			return $video;
+            return $video;
 
-		}, $videos_data ) );
+        }, $videos_data ) );
 
-	}
+    }
 
 }

Spacing +18 added lines, -18 removed lines

@@ -17,21 +17,21 @@ 
 	 *
 	 * @return array<Video>
 	 */
-	public function get_videos_data( $videos ) {
+	public function get_videos_data($videos) {
 
-		$video_urls = array_map( function ( $video ) {
+		$video_urls = array_map(function($video) {
 			return $video->get_url();
-		}, $videos );
+		}, $videos);
 
-		$videos_data = $this->api_client->get_data( $video_urls );
+		$videos_data = $this->api_client->get_data($video_urls);
 
-		return array_filter( array_map( function ( $video_data ) {
+		return array_filter(array_map(function($video_data) {
 
-			if ( ! array_key_exists( 'playlist', $video_data )
+			if ( ! array_key_exists('playlist', $video_data)
 
-			     || ! is_array( $video_data['playlist'] )
+			     || ! is_array($video_data['playlist'])
 
-			     || count( $video_data['playlist'] ) === 0 ) {
+			     || count($video_data['playlist']) === 0) {
 
 				return false;
 
@@ -42,31 +42,31 @@ 
 			$playlist_item_data    = $video_data['playlist'][0];
 			$video->name           = $playlist_item_data['title'];
 			$video->description    = $playlist_item_data['description'];
-			$video->thumbnail_urls = array_filter( array_map( function ( $item ) {
+			$video->thumbnail_urls = array_filter(array_map(function($item) {
 				return $item['src'];
-			}, $playlist_item_data['images'] ) );
+			}, $playlist_item_data['images']));
 
-			$video->duration    = "PT" . (int) $playlist_item_data['duration'] . "S";
-			$video->upload_date = date( 'c', (int) $playlist_item_data['pubdate'] );
+			$video->duration    = "PT".(int) $playlist_item_data['duration']."S";
+			$video->upload_date = date('c', (int) $playlist_item_data['pubdate']);
 
-			$video_content_urls_data = array_filter( $playlist_item_data['sources'], function ( $item ) {
-				return strpos( $item['type'], 'video/' ) !== false;
+			$video_content_urls_data = array_filter($playlist_item_data['sources'], function($item) {
+				return strpos($item['type'], 'video/') !== false;
 			} );
 
-			$video_content_urls = array_map( function ( $item ) {
+			$video_content_urls = array_map(function($item) {
 				return $item['file'];
-			}, $video_content_urls_data );
+			}, $video_content_urls_data);
 
 			// Content url is a single field, so we pick the video with
 			// high resolution.
-			$video->content_url = array_pop( $video_content_urls );
+			$video->content_url = array_pop($video_content_urls);
 
 			// We dont have embed url for JW Player.
 			$video->embed_url = '';
 
 			return $video;
 
-		}, $videos_data ) );
+		}, $videos_data));
 
 	}
 

src/wordlift/videoobject/provider/class-provider-factory.php

Indentation +13 added lines, -13 removed lines

@@ -10,23 +10,23 @@
  */
 class Provider_Factory {
 
-	const YOUTUBE = 'youtube';
+    const YOUTUBE = 'youtube';
 
-	const VIMEO = 'vimeo';
+    const VIMEO = 'vimeo';
 
-	const JWPLAYER = 'jwplayer';
+    const JWPLAYER = 'jwplayer';
 
 
-	public static function get_provider( $provider_name ) {
-		if ( self::YOUTUBE === $provider_name ) {
-			return new Youtube( Client_Factory::get_client( Client_Factory::YOUTUBE ) );
-		} else if (  self::VIMEO === $provider_name ) {
-			return new Vimeo( Client_Factory::get_client( Client_Factory::VIMEO ) );
-		}
-		else if ( self::JWPLAYER === $provider_name ) {
-			return new Jw_Player( Client_Factory::get_client( Client_Factory::JWPLAYER ) );
-		}
+    public static function get_provider( $provider_name ) {
+        if ( self::YOUTUBE === $provider_name ) {
+            return new Youtube( Client_Factory::get_client( Client_Factory::YOUTUBE ) );
+        } else if (  self::VIMEO === $provider_name ) {
+            return new Vimeo( Client_Factory::get_client( Client_Factory::VIMEO ) );
+        }
+        else if ( self::JWPLAYER === $provider_name ) {
+            return new Jw_Player( Client_Factory::get_client( Client_Factory::JWPLAYER ) );
+        }
 
-	}
+    }
 
 }
\ No newline at end of file

Spacing +7 added lines, -7 removed lines

@@ -17,14 +17,14 @@
 	const JWPLAYER = 'jwplayer';
 
 
-	public static function get_provider( $provider_name ) {
-		if ( self::YOUTUBE === $provider_name ) {
-			return new Youtube( Client_Factory::get_client( Client_Factory::YOUTUBE ) );
-		} else if (  self::VIMEO === $provider_name ) {
-			return new Vimeo( Client_Factory::get_client( Client_Factory::VIMEO ) );
+	public static function get_provider($provider_name) {
+		if (self::YOUTUBE === $provider_name) {
+			return new Youtube(Client_Factory::get_client(Client_Factory::YOUTUBE));
+		} else if (self::VIMEO === $provider_name) {
+			return new Vimeo(Client_Factory::get_client(Client_Factory::VIMEO));
 		}
-		else if ( self::JWPLAYER === $provider_name ) {
-			return new Jw_Player( Client_Factory::get_client( Client_Factory::JWPLAYER ) );
+		else if (self::JWPLAYER === $provider_name) {
+			return new Jw_Player(Client_Factory::get_client(Client_Factory::JWPLAYER));
 		}
 
 	}

Braces +1 added lines, -2 removed lines

@@ -22,8 +22,7 @@
 			return new Youtube( Client_Factory::get_client( Client_Factory::YOUTUBE ) );
 		} else if (  self::VIMEO === $provider_name ) {
 			return new Vimeo( Client_Factory::get_client( Client_Factory::VIMEO ) );
-		}
-		else if ( self::JWPLAYER === $provider_name ) {
+		} else if ( self::JWPLAYER === $provider_name ) {
 			return new Jw_Player( Client_Factory::get_client( Client_Factory::JWPLAYER ) );
 		}