insideout10 / wordlift-plugin

src/wordlift/videoobject/data/video-storage/class-meta-storage.php

Indentation +42 added lines, -42 removed lines

@@ -8,57 +8,57 @@
 
 class Meta_Storage implements Storage {
 
-	const META_KEY = '_wl_video_object_videos';
+    const META_KEY = '_wl_video_object_videos';
 
-	public function add_video( $post_id, $video ) {
-		/**
-		 * @since 3.31.0
-		 * Fires when the video storage gets updated
-		 */
-		do_action( 'wordlift_videoobject_video_storage_updated' );
-		add_post_meta( $post_id, self::META_KEY, $video );
-	}
+    public function add_video( $post_id, $video ) {
+        /**
+         * @since 3.31.0
+         * Fires when the video storage gets updated
+         */
+        do_action( 'wordlift_videoobject_video_storage_updated' );
+        add_post_meta( $post_id, self::META_KEY, $video );
+    }
 
-	public function get_all_videos( $post_id ) {
+    public function get_all_videos( $post_id ) {
 
-		return get_post_meta( $post_id, self::META_KEY );
+        return get_post_meta( $post_id, self::META_KEY );
 
-	}
+    }
 
-	public function remove_videos( $videos_to_be_removed, $post_id ) {
-		/**
-		 * @since 3.31.0
-		 * Fires when the video storage gets updated
-		 */
-		do_action( 'wordlift_videoobject_video_storage_updated' );
-		$videos_to_be_removed_ids = array_map(
-			function ( $video ) {
-				return $video->id;
-			},
-			$videos_to_be_removed
-		);
+    public function remove_videos( $videos_to_be_removed, $post_id ) {
+        /**
+         * @since 3.31.0
+         * Fires when the video storage gets updated
+         */
+        do_action( 'wordlift_videoobject_video_storage_updated' );
+        $videos_to_be_removed_ids = array_map(
+            function ( $video ) {
+                return $video->id;
+            },
+            $videos_to_be_removed
+        );
 
-		$present_videos = $this->get_all_videos( $post_id );
+        $present_videos = $this->get_all_videos( $post_id );
 
-		$filtered_videos = array_filter(
-			$present_videos,
-			function ( $video ) use ( $videos_to_be_removed_ids ) {
-				// phpcs:ignore WordPress.PHP.StrictInArray.MissingTrueStrict
-				return ! in_array( $video->id, $videos_to_be_removed_ids );
-			}
-		);
+        $filtered_videos = array_filter(
+            $present_videos,
+            function ( $video ) use ( $videos_to_be_removed_ids ) {
+                // phpcs:ignore WordPress.PHP.StrictInArray.MissingTrueStrict
+                return ! in_array( $video->id, $videos_to_be_removed_ids );
+            }
+        );
 
-		// Remove all existing videos.
-		$this->remove_all_videos( $post_id );
+        // Remove all existing videos.
+        $this->remove_all_videos( $post_id );
 
-		// Save the remaining videos.
-		foreach ( $filtered_videos as $video ) {
-			$this->add_video( $post_id, $video );
-		}
+        // Save the remaining videos.
+        foreach ( $filtered_videos as $video ) {
+            $this->add_video( $post_id, $video );
+        }
 
-	}
+    }
 
-	public function remove_all_videos( $post_id ) {
-		delete_post_meta( $post_id, self::META_KEY );
-	}
+    public function remove_all_videos( $post_id ) {
+        delete_post_meta( $post_id, self::META_KEY );
+    }
 }

Spacing +16 added lines, -16 removed lines

@@ -10,55 +10,55 @@
 
 	const META_KEY = '_wl_video_object_videos';
 
-	public function add_video( $post_id, $video ) {
+	public function add_video($post_id, $video) {
 		/**
 		 * @since 3.31.0
 		 * Fires when the video storage gets updated
 		 */
-		do_action( 'wordlift_videoobject_video_storage_updated' );
-		add_post_meta( $post_id, self::META_KEY, $video );
+		do_action('wordlift_videoobject_video_storage_updated');
+		add_post_meta($post_id, self::META_KEY, $video);
 	}
 
-	public function get_all_videos( $post_id ) {
+	public function get_all_videos($post_id) {
 
-		return get_post_meta( $post_id, self::META_KEY );
+		return get_post_meta($post_id, self::META_KEY);
 
 	}
 
-	public function remove_videos( $videos_to_be_removed, $post_id ) {
+	public function remove_videos($videos_to_be_removed, $post_id) {
 		/**
 		 * @since 3.31.0
 		 * Fires when the video storage gets updated
 		 */
-		do_action( 'wordlift_videoobject_video_storage_updated' );
+		do_action('wordlift_videoobject_video_storage_updated');
 		$videos_to_be_removed_ids = array_map(
-			function ( $video ) {
+			function($video) {
 				return $video->id;
 			},
 			$videos_to_be_removed
 		);
 
-		$present_videos = $this->get_all_videos( $post_id );
+		$present_videos = $this->get_all_videos($post_id);
 
 		$filtered_videos = array_filter(
 			$present_videos,
-			function ( $video ) use ( $videos_to_be_removed_ids ) {
+			function($video) use ($videos_to_be_removed_ids) {
 				// phpcs:ignore WordPress.PHP.StrictInArray.MissingTrueStrict
-				return ! in_array( $video->id, $videos_to_be_removed_ids );
+				return ! in_array($video->id, $videos_to_be_removed_ids);
 			}
 		);
 
 		// Remove all existing videos.
-		$this->remove_all_videos( $post_id );
+		$this->remove_all_videos($post_id);
 
 		// Save the remaining videos.
-		foreach ( $filtered_videos as $video ) {
-			$this->add_video( $post_id, $video );
+		foreach ($filtered_videos as $video) {
+			$this->add_video($post_id, $video);
 		}
 
 	}
 
-	public function remove_all_videos( $post_id ) {
-		delete_post_meta( $post_id, self::META_KEY );
+	public function remove_all_videos($post_id) {
+		delete_post_meta($post_id, self::META_KEY);
 	}
 }

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

Indentation +106 added lines, -106 removed lines

@@ -12,111 +12,111 @@ 
 
 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 "
+    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 "
    <url>
      <loc>${permalink}</loc>
      <video:video>
@@ -128,6 +128,6 @@ 
    </url>
 ";
 
-	}
+    }
 
 }

Spacing +22 added lines, -22 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;
 		}
 
@@ -37,8 +37,8 @@ 
 			'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,
@@ -48,16 +48,16 @@ 
 			)
 		);
 
-		$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;
@@ -69,14 +69,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;
@@ -89,29 +89,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/class-video-processor.php

Indentation +244 added lines, -244 removed lines

@@ -12,249 +12,249 @@
 
 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 ) {
-
-		$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, $that ) {
-				/**
-				 * If the video id doesn't 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 );
-
-		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 ) {
+
+        $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, $that ) {
+                /**
+                 * If the video id doesn't 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 );
+
+        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;
+            }
+        );
+    }
 
 }

Spacing +56 added lines, -56 removed lines

@@ -19,41 +19,41 @@ 
 
 	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 ) {
+	private function get_data_for_videos($embedded_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 );
+		$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);
 	}
 
 	/**
@@ -61,10 +61,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);
 
 	}
 
@@ -75,10 +75,10 @@ 
 	 *
 	 * @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 );
+	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 ) {
+			function($embedded_video) {
 				/**
 				 * @var $embedded_video Embedded_Video
 				 */
@@ -92,20 +92,20 @@ 
 		 * 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, $that ) {
+			function($video) use ($embedded_video_ids, $that) {
 				/**
 				 * If the video id doesn't 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 ) ),
+						$that->get_video_ids(array($video->id)),
 						$embedded_video_ids
 					)
 				) === 0;
@@ -118,11 +118,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
@@ -134,30 +134,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 );
+		$videos = $this->get_data_for_videos($embedded_videos);
 
-		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);
 		}
 	}
 
@@ -168,29 +168,29 @@ 
 	 *
 	 * @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 );
+	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 ) {
+			function($video) {
 				return $video->id;
 			},
 			$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 ) {
+			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() ) ),
+						$that->get_video_ids(array($embedded_video->get_url())),
 						$video_ids_in_store
 					)
 				) === 0;
@@ -203,10 +203,10 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_youtube_videos( $embedded_videos ) {
+	private function get_youtube_videos($embedded_videos) {
 		return array_filter(
 			$embedded_videos,
-			function ( $embedded_video ) {
+			function($embedded_video) {
 				/**
 				 * it should have youtube.com or youtu.be in the url
 				 *
@@ -214,8 +214,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;
 			}
 		);
 	}
@@ -225,10 +225,10 @@ 
 	 *
 	 * @return array
 	 */
-	private function get_vimeo_videos( $embedded_videos ) {
+	private function get_vimeo_videos($embedded_videos) {
 		return array_filter(
 			$embedded_videos,
-			function ( $embedded_video ) {
+			function($embedded_video) {
 				/**
 				 * it should have vimeo.com in the url
 				 *
@@ -236,15 +236,15 @@ 
 				 */
 				$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 ) {
+	private function get_jw_player_videos($embedded_videos) {
 		return array_filter(
 			$embedded_videos,
-			function ( $embedded_video ) {
+			function($embedded_video) {
 				/**
 				 * it should have vimeo.com in the url
 				 *
@@ -252,7 +252,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/ui/class-post-edit-screen.php

Indentation +28 added lines, -28 removed lines

@@ -10,34 +10,34 @@
 
 class Post_Edit_Screen {
 
-	public function init() {
-		$callback = array( $this, 'enqueue_scripts' );
-		add_action( 'enqueue_block_editor_assets', $callback );
-		add_action( 'admin_print_scripts-post.php', $callback );
-		add_action( 'admin_print_scripts-post-new.php', $callback );
-	}
+    public function init() {
+        $callback = array( $this, 'enqueue_scripts' );
+        add_action( 'enqueue_block_editor_assets', $callback );
+        add_action( 'admin_print_scripts-post.php', $callback );
+        add_action( 'admin_print_scripts-post-new.php', $callback );
+    }
 
-	public function enqueue_scripts() {
-		Scripts_Helper::enqueue_based_on_wordpress_version(
-			'wl-videoobject',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . '/js/dist/videoobject',
-			array( 'react', 'react-dom', 'wp-hooks', 'wp-i18n', 'wp-polyfill' )
-		);
-		wp_enqueue_style(
-			'wl-videoobject',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . '/js/dist/videoobject.css',
-			array(),
-			WORDLIFT_VERSION
-		);
-		wp_localize_script(
-			'wl-videoobject',
-			'_wlVideoobjectConfig',
-			array(
-				'restUrl' => get_rest_url( null, '/wordlift/v1/videos' ),
-				'nonce'   => wp_create_nonce( 'wp_rest' ),
-				'postId'  => get_the_ID(),
-			)
-		);
-	}
+    public function enqueue_scripts() {
+        Scripts_Helper::enqueue_based_on_wordpress_version(
+            'wl-videoobject',
+            plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . '/js/dist/videoobject',
+            array( 'react', 'react-dom', 'wp-hooks', 'wp-i18n', 'wp-polyfill' )
+        );
+        wp_enqueue_style(
+            'wl-videoobject',
+            plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . '/js/dist/videoobject.css',
+            array(),
+            WORDLIFT_VERSION
+        );
+        wp_localize_script(
+            'wl-videoobject',
+            '_wlVideoobjectConfig',
+            array(
+                'restUrl' => get_rest_url( null, '/wordlift/v1/videos' ),
+                'nonce'   => wp_create_nonce( 'wp_rest' ),
+                'postId'  => get_the_ID(),
+            )
+        );
+    }
 
 }

Spacing +9 added lines, -9 removed lines

@@ -11,21 +11,21 @@ 
 class Post_Edit_Screen {
 
 	public function init() {
-		$callback = array( $this, 'enqueue_scripts' );
-		add_action( 'enqueue_block_editor_assets', $callback );
-		add_action( 'admin_print_scripts-post.php', $callback );
-		add_action( 'admin_print_scripts-post-new.php', $callback );
+		$callback = array($this, 'enqueue_scripts');
+		add_action('enqueue_block_editor_assets', $callback);
+		add_action('admin_print_scripts-post.php', $callback);
+		add_action('admin_print_scripts-post-new.php', $callback);
 	}
 
 	public function enqueue_scripts() {
 		Scripts_Helper::enqueue_based_on_wordpress_version(
 			'wl-videoobject',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . '/js/dist/videoobject',
-			array( 'react', 'react-dom', 'wp-hooks', 'wp-i18n', 'wp-polyfill' )
+			plugin_dir_url(dirname(dirname(__DIR__))).'/js/dist/videoobject',
+			array('react', 'react-dom', 'wp-hooks', 'wp-i18n', 'wp-polyfill')
 		);
 		wp_enqueue_style(
 			'wl-videoobject',
-			plugin_dir_url( dirname( dirname( __DIR__ ) ) ) . '/js/dist/videoobject.css',
+			plugin_dir_url(dirname(dirname(__DIR__))).'/js/dist/videoobject.css',
 			array(),
 			WORDLIFT_VERSION
 		);
@@ -33,8 +33,8 @@ 
 			'wl-videoobject',
 			'_wlVideoobjectConfig',
 			array(
-				'restUrl' => get_rest_url( null, '/wordlift/v1/videos' ),
-				'nonce'   => wp_create_nonce( 'wp_rest' ),
+				'restUrl' => get_rest_url(null, '/wordlift/v1/videos'),
+				'nonce'   => wp_create_nonce('wp_rest'),
 				'postId'  => get_the_ID(),
 			)
 		);

src/wordlift/videoobject/tabs/class-settings-tab.php

Indentation +12 added lines, -12 removed lines

@@ -7,19 +7,19 @@
 
 class Settings_Tab {
 
-	public function init() {
+    public function init() {
 
-		add_filter(
-			'wl_admin_page_tabs',
-			function ( $tabs ) {
-				$tabs[] = array(
-					'slug'  => 'videoobject-settings',
-					'title' => __( 'Video Settings', 'wordlift' ),
-				);
-				return $tabs;
-			}
-		);
+        add_filter(
+            'wl_admin_page_tabs',
+            function ( $tabs ) {
+                $tabs[] = array(
+                    'slug'  => 'videoobject-settings',
+                    'title' => __( 'Video Settings', 'wordlift' ),
+                );
+                return $tabs;
+            }
+        );
 
-	}
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -11,10 +11,10 @@
 
 		add_filter(
 			'wl_admin_page_tabs',
-			function ( $tabs ) {
+			function($tabs) {
 				$tabs[] = array(
 					'slug'  => 'videoobject-settings',
-					'title' => __( 'Video Settings', 'wordlift' ),
+					'title' => __('Video Settings', 'wordlift'),
 				);
 				return $tabs;
 			}

src/wordlift/videoobject/api/class-rest-controller.php

Indentation +160 added lines, -160 removed lines

@@ -12,165 +12,165 @@
 use WP_REST_Server;
 
 class Rest_Controller {
-	/**
-	 * @var Videoobject_Background_Process
-	 */
-	private $background_process;
-
-	/**
-	 * Rest_Controller constructor.
-	 *
-	 * @param $background_process Videoobject_Background_Process
-	 */
-	public function __construct( $background_process ) {
-		$this->background_process = $background_process;
-	}
-
-	public function register_all_routes() {
-		$that = $this;
-		add_action(
-			'rest_api_init',
-			function () use ( $that ) {
-				$that->register_get_all_videos_route();
-				$that->register_save_all_videos_route();
-				$that->register_get_sync_state_endpoint();
-				$that->register_background_process_start_endpoint();
-				$that->register_background_process_stop_endpoint();
-			}
-		);
-	}
-
-	public function get_all_videos( $request ) {
-		$data    = $request->get_params();
-		$post_id = (int) $data['post_id'];
-		$storage = Video_Storage_Factory::get_storage();
-
-		return $storage->get_all_videos( $post_id );
-	}
-
-	public function save_all_videos( $request ) {
-		$data    = $request->get_params();
-		$post_id = (int) $data['post_id'];
-		$videos  = (array) $data['videos'];
-		if ( ! $videos ) {
-			return;
-		}
-		$storage = Video_Storage_Factory::get_storage();
-		$storage->remove_all_videos( $post_id );
-
-		foreach ( $videos as $video ) {
-			$video_obj = new Video();
-			$video_obj->from( (array) $video );
-			$storage->add_video( $post_id, $video_obj );
-		}
-
-	}
-
-	private function register_get_all_videos_route() {
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/videos',
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => array( $this, 'get_all_videos' ),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-				'args'                => array(
-					'post_id' => array(
-						// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_numeric( $param ) && $param;
-						},
-						'required'          => true,
-					),
-				),
-			)
-		);
-	}
-
-	private function register_save_all_videos_route() {
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/videos/save',
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => array( $this, 'save_all_videos' ),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-				'args'                => array(
-					'post_id' => array(
-						// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_numeric( $param ) && $param;
-						},
-						'required'          => true,
-					),
-					'videos'  => array(
-						// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_array( $param ) && $param;
-						},
-						'required'          => true,
-					),
-				),
-			)
-		);
-	}
-
-	public function register_get_sync_state_endpoint() {
-		$that = $this;
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/videos/background/get_state',
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => function () use ( $that ) {
-					return $that->background_process->get_state()->get_array();
-				},
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-
-	}
-
-	public function register_background_process_start_endpoint() {
-		$that = $this;
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/videos/background/start',
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => function () use ( $that ) {
-					$that->background_process->start();
-				},
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-
-	}
-
-	public function register_background_process_stop_endpoint() {
-		$that = $this;
-		register_rest_route(
-			WL_REST_ROUTE_DEFAULT_NAMESPACE,
-			'/videos/background/stop',
-			array(
-				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => function () use ( $that ) {
-					$that->background_process->cancel();
-				},
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
-				},
-			)
-		);
-
-	}
+    /**
+     * @var Videoobject_Background_Process
+     */
+    private $background_process;
+
+    /**
+     * Rest_Controller constructor.
+     *
+     * @param $background_process Videoobject_Background_Process
+     */
+    public function __construct( $background_process ) {
+        $this->background_process = $background_process;
+    }
+
+    public function register_all_routes() {
+        $that = $this;
+        add_action(
+            'rest_api_init',
+            function () use ( $that ) {
+                $that->register_get_all_videos_route();
+                $that->register_save_all_videos_route();
+                $that->register_get_sync_state_endpoint();
+                $that->register_background_process_start_endpoint();
+                $that->register_background_process_stop_endpoint();
+            }
+        );
+    }
+
+    public function get_all_videos( $request ) {
+        $data    = $request->get_params();
+        $post_id = (int) $data['post_id'];
+        $storage = Video_Storage_Factory::get_storage();
+
+        return $storage->get_all_videos( $post_id );
+    }
+
+    public function save_all_videos( $request ) {
+        $data    = $request->get_params();
+        $post_id = (int) $data['post_id'];
+        $videos  = (array) $data['videos'];
+        if ( ! $videos ) {
+            return;
+        }
+        $storage = Video_Storage_Factory::get_storage();
+        $storage->remove_all_videos( $post_id );
+
+        foreach ( $videos as $video ) {
+            $video_obj = new Video();
+            $video_obj->from( (array) $video );
+            $storage->add_video( $post_id, $video_obj );
+        }
+
+    }
+
+    private function register_get_all_videos_route() {
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/videos',
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => array( $this, 'get_all_videos' ),
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+                'args'                => array(
+                    'post_id' => array(
+                        // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+                        'validate_callback' => function ( $param, $request, $key ) {
+                            return is_numeric( $param ) && $param;
+                        },
+                        'required'          => true,
+                    ),
+                ),
+            )
+        );
+    }
+
+    private function register_save_all_videos_route() {
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/videos/save',
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => array( $this, 'save_all_videos' ),
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+                'args'                => array(
+                    'post_id' => array(
+                        // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+                        'validate_callback' => function ( $param, $request, $key ) {
+                            return is_numeric( $param ) && $param;
+                        },
+                        'required'          => true,
+                    ),
+                    'videos'  => array(
+                        // phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
+                        'validate_callback' => function ( $param, $request, $key ) {
+                            return is_array( $param ) && $param;
+                        },
+                        'required'          => true,
+                    ),
+                ),
+            )
+        );
+    }
+
+    public function register_get_sync_state_endpoint() {
+        $that = $this;
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/videos/background/get_state',
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => function () use ( $that ) {
+                    return $that->background_process->get_state()->get_array();
+                },
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+
+    }
+
+    public function register_background_process_start_endpoint() {
+        $that = $this;
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/videos/background/start',
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => function () use ( $that ) {
+                    $that->background_process->start();
+                },
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+
+    }
+
+    public function register_background_process_stop_endpoint() {
+        $that = $this;
+        register_rest_route(
+            WL_REST_ROUTE_DEFAULT_NAMESPACE,
+            '/videos/background/stop',
+            array(
+                'methods'             => WP_REST_Server::CREATABLE,
+                'callback'            => function () use ( $that ) {
+                    $that->background_process->cancel();
+                },
+                'permission_callback' => function () {
+                    return current_user_can( 'manage_options' );
+                },
+            )
+        );
+
+    }
 
 }

Spacing +31 added lines, -31 removed lines

@@ -22,7 +22,7 @@ 
 	 *
 	 * @param $background_process Videoobject_Background_Process
 	 */
-	public function __construct( $background_process ) {
+	public function __construct($background_process) {
 		$this->background_process = $background_process;
 	}
 
@@ -30,7 +30,7 @@ 
 		$that = $this;
 		add_action(
 			'rest_api_init',
-			function () use ( $that ) {
+			function() use ($that) {
 				$that->register_get_all_videos_route();
 				$that->register_save_all_videos_route();
 				$that->register_get_sync_state_endpoint();
@@ -40,28 +40,28 @@ 
 		);
 	}
 
-	public function get_all_videos( $request ) {
+	public function get_all_videos($request) {
 		$data    = $request->get_params();
 		$post_id = (int) $data['post_id'];
 		$storage = Video_Storage_Factory::get_storage();
 
-		return $storage->get_all_videos( $post_id );
+		return $storage->get_all_videos($post_id);
 	}
 
-	public function save_all_videos( $request ) {
+	public function save_all_videos($request) {
 		$data    = $request->get_params();
 		$post_id = (int) $data['post_id'];
 		$videos  = (array) $data['videos'];
-		if ( ! $videos ) {
+		if ( ! $videos) {
 			return;
 		}
 		$storage = Video_Storage_Factory::get_storage();
-		$storage->remove_all_videos( $post_id );
+		$storage->remove_all_videos($post_id);
 
-		foreach ( $videos as $video ) {
+		foreach ($videos as $video) {
 			$video_obj = new Video();
-			$video_obj->from( (array) $video );
-			$storage->add_video( $post_id, $video_obj );
+			$video_obj->from((array) $video);
+			$storage->add_video($post_id, $video_obj);
 		}
 
 	}
@@ -72,15 +72,15 @@ 
 			'/videos',
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => array( $this, 'get_all_videos' ),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'callback'            => array($this, 'get_all_videos'),
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 				'args'                => array(
 					'post_id' => array(
 						// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_numeric( $param ) && $param;
+						'validate_callback' => function($param, $request, $key) {
+							return is_numeric($param) && $param;
 						},
 						'required'          => true,
 					),
@@ -95,22 +95,22 @@ 
 			'/videos/save',
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => array( $this, 'save_all_videos' ),
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'callback'            => array($this, 'save_all_videos'),
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 				'args'                => array(
 					'post_id' => array(
 						// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_numeric( $param ) && $param;
+						'validate_callback' => function($param, $request, $key) {
+							return is_numeric($param) && $param;
 						},
 						'required'          => true,
 					),
 					'videos'  => array(
 						// phpcs:ignore VariableAnalysis.CodeAnalysis.VariableAnalysis.UnusedVariable
-						'validate_callback' => function ( $param, $request, $key ) {
-							return is_array( $param ) && $param;
+						'validate_callback' => function($param, $request, $key) {
+							return is_array($param) && $param;
 						},
 						'required'          => true,
 					),
@@ -126,11 +126,11 @@ 
 			'/videos/background/get_state',
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => function () use ( $that ) {
+				'callback'            => function() use ($that) {
 					return $that->background_process->get_state()->get_array();
 				},
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -144,11 +144,11 @@ 
 			'/videos/background/start',
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => function () use ( $that ) {
+				'callback'            => function() use ($that) {
 					$that->background_process->start();
 				},
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);
@@ -162,11 +162,11 @@ 
 			'/videos/background/stop',
 			array(
 				'methods'             => WP_REST_Server::CREATABLE,
-				'callback'            => function () use ( $that ) {
+				'callback'            => function() use ($that) {
 					$that->background_process->cancel();
 				},
-				'permission_callback' => function () {
-					return current_user_can( 'manage_options' );
+				'permission_callback' => function() {
+					return current_user_can('manage_options');
 				},
 			)
 		);

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

Indentation +129 added lines, -129 removed lines

@@ -11,134 +11,134 @@
 
 class Youtube extends Api_Provider {
 
-	const YT_API_FIELD_NAME = '__wl_video_object_youtube_api_key';
-
-	private static function get_thumbnails( $api_thumbnail_data ) {
-		return array_map(
-			function ( $item ) {
-				return $item['url'];
-
-			},
-			$api_thumbnail_data
-		);
-	}
-
-	public function get_videos_data( $videos ) {
-		$urls = array_map(
-			function ( $video ) {
-				/**
-				 * @param $video Video
-				 */
-				return $video->get_url();
-			},
-			$videos
-		);
-
-		return $this->get_data( $urls );
-
-	}
-
-	/**
-	 * @param $video_urls
-	 *
-	 * @return bool | array<Video>
-	 */
-	public function get_data( $video_urls ) {
-		// extract ids from the url list.
-		if ( ! is_array( $video_urls ) ) {
-			return array();
-		}
-		$response_body = $this->api_client->get_data( $video_urls );
-
-		return $this->parse_youtube_video_data_from_response( $response_body );
-	}
-
-	/**
-	 * @param $response_body string
-	 *
-	 * @return array<Video>
-	 */
-	private function parse_youtube_video_data_from_response( $response_body ) {
-		$result = json_decode( $response_body, true );
-		if ( ! is_array( $result ) ) {
-			// Return empty array since the response body is invalid.
-			return array();
-		}
-		if ( ! array_key_exists( 'items', $result ) ) {
-			return array();
-		}
-		$videos_json_data = $result['items'];
-		$videos           = array();
-		foreach ( $videos_json_data as $single_video_json_data ) {
-			$videos[] = self::create_video_from_youtube_data( $single_video_json_data );
-		}
-
-		return $videos;
-	}
-
-	public static function create_video_from_youtube_data( $video_data ) {
-
-		$video = new Video();
-		if ( array_key_exists( 'contentDetails', $video_data ) ) {
-			$video->duration = $video_data['contentDetails']['duration'];
-		}
-
-		if ( array_key_exists( 'id', $video_data ) ) {
-			$video_id           = $video_data['id'];
-			$video->embed_url   = "https://www.youtube.com/embed/${video_id}";
-			$video->content_url = "https://www.youtube.com/watch?v=${video_id}";
-		}
-		if ( ! array_key_exists( 'snippet', $video_data ) ) {
-			return false;
-		}
-
-		$video->name        = $video_data['snippet']['title'];
-		$video->description = $video_data['snippet']['description'];
-
-		/**
-		 * @since 3.30.0
-		 * Use title as fallback if description is not present.
-		 */
-		if ( ! $video->description ) {
-			$video->description = $video->name;
-		}
-
-		$video->upload_date = $video_data['snippet']['publishedAt'];
-
-		if ( array_key_exists( 'thumbnails', $video_data['snippet'] ) ) {
-			$api_thumbnail_data    = array_values( $video_data['snippet']['thumbnails'] );
-			$video->thumbnail_urls = self::get_thumbnails( $api_thumbnail_data );
-		}
-
-		if ( array_key_exists( 'statistics', $video_data )
-			 && array_key_exists( 'viewCount', $video_data['statistics'] ) ) {
-			$video->views = $video_data['statistics']['viewCount'];
-		}
-
-		if ( array_key_exists( 'liveStreamingDetails', $video_data ) &&
-			 array_key_exists( 'scheduledStartTime', $video_data['liveStreamingDetails'] ) ) {
-			$video->is_live_video         = true;
-			$video->live_video_start_date = $video_data['liveStreamingDetails']['scheduledStartTime'];
-			try {
-				$end_date = new \DateTime( $video->live_video_start_date );
-				/**
-				 * the google doc says :
-				 * It is required to provide the endDate once the video has finished and is no longer live.
-				 * If the expected endDate is unknown prior to the livestream starting, we recommend providing an approximate endDate.
-				 */
-				// we add 1 day to start date
-				$end_date->add( new DateInterval( 'P1D' ) );
-				$video->live_video_end_date = $end_date->format( 'c' );
-			// phpcs:ignore Generic.CodeAnalysis.EmptyStatement.DetectedCatch
-			} catch ( \Exception $e ) {
-				// No need to do anything here.
-			}
-		}
-
-		$video->id = $video->content_url;
-
-		return $video;
-
-	}
+    const YT_API_FIELD_NAME = '__wl_video_object_youtube_api_key';
+
+    private static function get_thumbnails( $api_thumbnail_data ) {
+        return array_map(
+            function ( $item ) {
+                return $item['url'];
+
+            },
+            $api_thumbnail_data
+        );
+    }
+
+    public function get_videos_data( $videos ) {
+        $urls = array_map(
+            function ( $video ) {
+                /**
+                 * @param $video Video
+                 */
+                return $video->get_url();
+            },
+            $videos
+        );
+
+        return $this->get_data( $urls );
+
+    }
+
+    /**
+     * @param $video_urls
+     *
+     * @return bool | array<Video>
+     */
+    public function get_data( $video_urls ) {
+        // extract ids from the url list.
+        if ( ! is_array( $video_urls ) ) {
+            return array();
+        }
+        $response_body = $this->api_client->get_data( $video_urls );
+
+        return $this->parse_youtube_video_data_from_response( $response_body );
+    }
+
+    /**
+     * @param $response_body string
+     *
+     * @return array<Video>
+     */
+    private function parse_youtube_video_data_from_response( $response_body ) {
+        $result = json_decode( $response_body, true );
+        if ( ! is_array( $result ) ) {
+            // Return empty array since the response body is invalid.
+            return array();
+        }
+        if ( ! array_key_exists( 'items', $result ) ) {
+            return array();
+        }
+        $videos_json_data = $result['items'];
+        $videos           = array();
+        foreach ( $videos_json_data as $single_video_json_data ) {
+            $videos[] = self::create_video_from_youtube_data( $single_video_json_data );
+        }
+
+        return $videos;
+    }
+
+    public static function create_video_from_youtube_data( $video_data ) {
+
+        $video = new Video();
+        if ( array_key_exists( 'contentDetails', $video_data ) ) {
+            $video->duration = $video_data['contentDetails']['duration'];
+        }
+
+        if ( array_key_exists( 'id', $video_data ) ) {
+            $video_id           = $video_data['id'];
+            $video->embed_url   = "https://www.youtube.com/embed/${video_id}";
+            $video->content_url = "https://www.youtube.com/watch?v=${video_id}";
+        }
+        if ( ! array_key_exists( 'snippet', $video_data ) ) {
+            return false;
+        }
+
+        $video->name        = $video_data['snippet']['title'];
+        $video->description = $video_data['snippet']['description'];
+
+        /**
+         * @since 3.30.0
+         * Use title as fallback if description is not present.
+         */
+        if ( ! $video->description ) {
+            $video->description = $video->name;
+        }
+
+        $video->upload_date = $video_data['snippet']['publishedAt'];
+
+        if ( array_key_exists( 'thumbnails', $video_data['snippet'] ) ) {
+            $api_thumbnail_data    = array_values( $video_data['snippet']['thumbnails'] );
+            $video->thumbnail_urls = self::get_thumbnails( $api_thumbnail_data );
+        }
+
+        if ( array_key_exists( 'statistics', $video_data )
+             && array_key_exists( 'viewCount', $video_data['statistics'] ) ) {
+            $video->views = $video_data['statistics']['viewCount'];
+        }
+
+        if ( array_key_exists( 'liveStreamingDetails', $video_data ) &&
+             array_key_exists( 'scheduledStartTime', $video_data['liveStreamingDetails'] ) ) {
+            $video->is_live_video         = true;
+            $video->live_video_start_date = $video_data['liveStreamingDetails']['scheduledStartTime'];
+            try {
+                $end_date = new \DateTime( $video->live_video_start_date );
+                /**
+                 * the google doc says :
+                 * It is required to provide the endDate once the video has finished and is no longer live.
+                 * If the expected endDate is unknown prior to the livestream starting, we recommend providing an approximate endDate.
+                 */
+                // we add 1 day to start date
+                $end_date->add( new DateInterval( 'P1D' ) );
+                $video->live_video_end_date = $end_date->format( 'c' );
+            // phpcs:ignore Generic.CodeAnalysis.EmptyStatement.DetectedCatch
+            } catch ( \Exception $e ) {
+                // No need to do anything here.
+            }
+        }
+
+        $video->id = $video->content_url;
+
+        return $video;
+
+    }
 
 }

Spacing +31 added lines, -31 removed lines

@@ -13,9 +13,9 @@ 
 
 	const YT_API_FIELD_NAME = '__wl_video_object_youtube_api_key';
 
-	private static function get_thumbnails( $api_thumbnail_data ) {
+	private static function get_thumbnails($api_thumbnail_data) {
 		return array_map(
-			function ( $item ) {
+			function($item) {
 				return $item['url'];
 
 			},
@@ -23,9 +23,9 @@ 
 		);
 	}
 
-	public function get_videos_data( $videos ) {
+	public function get_videos_data($videos) {
 		$urls = array_map(
-			function ( $video ) {
+			function($video) {
 				/**
 				 * @param $video Video
 				 */
@@ -34,7 +34,7 @@ 
 			$videos
 		);
 
-		return $this->get_data( $urls );
+		return $this->get_data($urls);
 
 	}
 
@@ -43,14 +43,14 @@ 
 	 *
 	 * @return bool | array<Video>
 	 */
-	public function get_data( $video_urls ) {
+	public function get_data($video_urls) {
 		// extract ids from the url list.
-		if ( ! is_array( $video_urls ) ) {
+		if ( ! is_array($video_urls)) {
 			return array();
 		}
-		$response_body = $this->api_client->get_data( $video_urls );
+		$response_body = $this->api_client->get_data($video_urls);
 
-		return $this->parse_youtube_video_data_from_response( $response_body );
+		return $this->parse_youtube_video_data_from_response($response_body);
 	}
 
 	/**
@@ -58,37 +58,37 @@ 
 	 *
 	 * @return array<Video>
 	 */
-	private function parse_youtube_video_data_from_response( $response_body ) {
-		$result = json_decode( $response_body, true );
-		if ( ! is_array( $result ) ) {
+	private function parse_youtube_video_data_from_response($response_body) {
+		$result = json_decode($response_body, true);
+		if ( ! is_array($result)) {
 			// Return empty array since the response body is invalid.
 			return array();
 		}
-		if ( ! array_key_exists( 'items', $result ) ) {
+		if ( ! array_key_exists('items', $result)) {
 			return array();
 		}
 		$videos_json_data = $result['items'];
 		$videos           = array();
-		foreach ( $videos_json_data as $single_video_json_data ) {
-			$videos[] = self::create_video_from_youtube_data( $single_video_json_data );
+		foreach ($videos_json_data as $single_video_json_data) {
+			$videos[] = self::create_video_from_youtube_data($single_video_json_data);
 		}
 
 		return $videos;
 	}
 
-	public static function create_video_from_youtube_data( $video_data ) {
+	public static function create_video_from_youtube_data($video_data) {
 
 		$video = new Video();
-		if ( array_key_exists( 'contentDetails', $video_data ) ) {
+		if (array_key_exists('contentDetails', $video_data)) {
 			$video->duration = $video_data['contentDetails']['duration'];
 		}
 
-		if ( array_key_exists( 'id', $video_data ) ) {
+		if (array_key_exists('id', $video_data)) {
 			$video_id           = $video_data['id'];
 			$video->embed_url   = "https://www.youtube.com/embed/${video_id}";
 			$video->content_url = "https://www.youtube.com/watch?v=${video_id}";
 		}
-		if ( ! array_key_exists( 'snippet', $video_data ) ) {
+		if ( ! array_key_exists('snippet', $video_data)) {
 			return false;
 		}
 
@@ -99,38 +99,38 @@ 
 		 * @since 3.30.0
 		 * Use title as fallback if description is not present.
 		 */
-		if ( ! $video->description ) {
+		if ( ! $video->description) {
 			$video->description = $video->name;
 		}
 
 		$video->upload_date = $video_data['snippet']['publishedAt'];
 
-		if ( array_key_exists( 'thumbnails', $video_data['snippet'] ) ) {
-			$api_thumbnail_data    = array_values( $video_data['snippet']['thumbnails'] );
-			$video->thumbnail_urls = self::get_thumbnails( $api_thumbnail_data );
+		if (array_key_exists('thumbnails', $video_data['snippet'])) {
+			$api_thumbnail_data    = array_values($video_data['snippet']['thumbnails']);
+			$video->thumbnail_urls = self::get_thumbnails($api_thumbnail_data);
 		}
 
-		if ( array_key_exists( 'statistics', $video_data )
-			 && array_key_exists( 'viewCount', $video_data['statistics'] ) ) {
+		if (array_key_exists('statistics', $video_data)
+			 && array_key_exists('viewCount', $video_data['statistics'])) {
 			$video->views = $video_data['statistics']['viewCount'];
 		}
 
-		if ( array_key_exists( 'liveStreamingDetails', $video_data ) &&
-			 array_key_exists( 'scheduledStartTime', $video_data['liveStreamingDetails'] ) ) {
+		if (array_key_exists('liveStreamingDetails', $video_data) &&
+			 array_key_exists('scheduledStartTime', $video_data['liveStreamingDetails'])) {
 			$video->is_live_video         = true;
 			$video->live_video_start_date = $video_data['liveStreamingDetails']['scheduledStartTime'];
 			try {
-				$end_date = new \DateTime( $video->live_video_start_date );
+				$end_date = new \DateTime($video->live_video_start_date);
 				/**
 				 * the google doc says :
 				 * It is required to provide the endDate once the video has finished and is no longer live.
 				 * If the expected endDate is unknown prior to the livestream starting, we recommend providing an approximate endDate.
 				 */
 				// we add 1 day to start date
-				$end_date->add( new DateInterval( 'P1D' ) );
-				$video->live_video_end_date = $end_date->format( 'c' );
+				$end_date->add(new DateInterval('P1D'));
+				$video->live_video_end_date = $end_date->format('c');
 			// phpcs:ignore Generic.CodeAnalysis.EmptyStatement.DetectedCatch
-			} catch ( \Exception $e ) {
+			} catch (\Exception $e) {
 				// No need to do anything here.
 			}
 		}

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

Indentation +12 added lines, -12 removed lines

@@ -10,21 +10,21 @@
  */
 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 ) );
-		} elseif ( self::VIMEO === $provider_name ) {
-			return new Vimeo( Client_Factory::get_client( Client_Factory::VIMEO ) );
-		} elseif ( 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 ) );
+        } elseif ( self::VIMEO === $provider_name ) {
+            return new Vimeo( Client_Factory::get_client( Client_Factory::VIMEO ) );
+        } elseif ( self::JWPLAYER === $provider_name ) {
+            return new Jw_Player( Client_Factory::get_client( Client_Factory::JWPLAYER ) );
+        }
 
-	}
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -16,13 +16,13 @@
 
 	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 ) );
-		} elseif ( self::VIMEO === $provider_name ) {
-			return new Vimeo( Client_Factory::get_client( Client_Factory::VIMEO ) );
-		} elseif ( 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));
+		} elseif (self::VIMEO === $provider_name) {
+			return new Vimeo(Client_Factory::get_client(Client_Factory::VIMEO));
+		} elseif (self::JWPLAYER === $provider_name) {
+			return new Jw_Player(Client_Factory::get_client(Client_Factory::JWPLAYER));
 		}
 
 	}

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

Indentation +92 added lines, -92 removed lines

@@ -15,97 +15,97 @@
  */
 class Vimeo extends Api_Provider {
 
-	const API_FIELD_NAME = '_wl_videoobject_vimeo_api_key';
-
-	public static function get_api_key() {
-		return get_option( self::API_FIELD_NAME, false );
-	}
-
-	public function get_videos_data( $videos ) {
-
-		$key = $this->get_api_key();
-
-		if ( ! $key ) {
-			return array();
-		}
-
-		$urls = array_map(
-			function ( $video ) {
-				return $video->get_url();
-			},
-			$videos
-		);
-
-		$response_body = $this->api_client->get_data( $urls );
-
-		if ( ! is_string( $response_body ) || ! $response_body ) {
-			return array();
-		}
-
-		$response = json_decode( $response_body, true );
-
-		$video_list = $response['data'];
-
-		if ( ! is_array( $video_list ) ) {
-			// Return if we cant parse the response.
-			return array();
-		}
-
-		return array_filter( array_map( array( $this, 'get_video_from_video_data' ), $video_list ) );
-
-	}
-
-	public function get_video_from_video_data( $vimeo_video_data ) {
-
-		if ( ! $vimeo_video_data ) {
-			// If valid data not supplied dont init the object.
-			return false;
-		}
-		$video              = new Video();
-		$video->name        = $vimeo_video_data['name'];
-		$video->description = $vimeo_video_data['description'];
-
-		$video->content_url = $vimeo_video_data['link'];
-		$video->embed_url   = 'https://player.vimeo.com/video/' . $this->get_id( $vimeo_video_data );
-		if ( is_numeric( $vimeo_video_data['duration'] ) ) {
-			$video->duration = 'PT' . $vimeo_video_data['duration'] . 'S';
-		}
-		$video->upload_date    = $vimeo_video_data['release_time'];
-		$video->thumbnail_urls = $this->set_thumbnail_urls( $vimeo_video_data );
-		$video->id             = $video->content_url;
-
-		if ( array_key_exists( 'stats', $vimeo_video_data )
-			 && array_key_exists( 'plays', $vimeo_video_data['stats'] ) ) {
-			$video->views = (int) $vimeo_video_data['stats']['plays'];
-		}
-
-		return $video;
-	}
-
-	private function get_id( $api_response_data ) {
-		return str_replace( '/videos/', '', $api_response_data['uri'] );
-	}
-
-	private function set_thumbnail_urls( $api_response_data ) {
-
-		if ( ! array_key_exists( 'pictures', $api_response_data ) || ! array_key_exists(
-			'sizes',
-			$api_response_data['pictures']
-		) ) {
-			return array();
-		}
-		if ( ! is_array( $api_response_data['pictures']['sizes'] ) ) {
-			return array();
-		}
-		$pictures = $api_response_data['pictures']['sizes'];
-
-		return array_map(
-			function ( $picture_data ) {
-				return $picture_data['link'];
-			},
-			$pictures
-		);
-
-	}
+    const API_FIELD_NAME = '_wl_videoobject_vimeo_api_key';
+
+    public static function get_api_key() {
+        return get_option( self::API_FIELD_NAME, false );
+    }
+
+    public function get_videos_data( $videos ) {
+
+        $key = $this->get_api_key();
+
+        if ( ! $key ) {
+            return array();
+        }
+
+        $urls = array_map(
+            function ( $video ) {
+                return $video->get_url();
+            },
+            $videos
+        );
+
+        $response_body = $this->api_client->get_data( $urls );
+
+        if ( ! is_string( $response_body ) || ! $response_body ) {
+            return array();
+        }
+
+        $response = json_decode( $response_body, true );
+
+        $video_list = $response['data'];
+
+        if ( ! is_array( $video_list ) ) {
+            // Return if we cant parse the response.
+            return array();
+        }
+
+        return array_filter( array_map( array( $this, 'get_video_from_video_data' ), $video_list ) );
+
+    }
+
+    public function get_video_from_video_data( $vimeo_video_data ) {
+
+        if ( ! $vimeo_video_data ) {
+            // If valid data not supplied dont init the object.
+            return false;
+        }
+        $video              = new Video();
+        $video->name        = $vimeo_video_data['name'];
+        $video->description = $vimeo_video_data['description'];
+
+        $video->content_url = $vimeo_video_data['link'];
+        $video->embed_url   = 'https://player.vimeo.com/video/' . $this->get_id( $vimeo_video_data );
+        if ( is_numeric( $vimeo_video_data['duration'] ) ) {
+            $video->duration = 'PT' . $vimeo_video_data['duration'] . 'S';
+        }
+        $video->upload_date    = $vimeo_video_data['release_time'];
+        $video->thumbnail_urls = $this->set_thumbnail_urls( $vimeo_video_data );
+        $video->id             = $video->content_url;
+
+        if ( array_key_exists( 'stats', $vimeo_video_data )
+             && array_key_exists( 'plays', $vimeo_video_data['stats'] ) ) {
+            $video->views = (int) $vimeo_video_data['stats']['plays'];
+        }
+
+        return $video;
+    }
+
+    private function get_id( $api_response_data ) {
+        return str_replace( '/videos/', '', $api_response_data['uri'] );
+    }
+
+    private function set_thumbnail_urls( $api_response_data ) {
+
+        if ( ! array_key_exists( 'pictures', $api_response_data ) || ! array_key_exists(
+            'sizes',
+            $api_response_data['pictures']
+        ) ) {
+            return array();
+        }
+        if ( ! is_array( $api_response_data['pictures']['sizes'] ) ) {
+            return array();
+        }
+        $pictures = $api_response_data['pictures']['sizes'];
+
+        return array_map(
+            function ( $picture_data ) {
+                return $picture_data['link'];
+            },
+            $pictures
+        );
+
+    }
 
 }

Spacing +24 added lines, -24 removed lines

@@ -18,46 +18,46 @@ 
 	const API_FIELD_NAME = '_wl_videoobject_vimeo_api_key';
 
 	public static function get_api_key() {
-		return get_option( self::API_FIELD_NAME, false );
+		return get_option(self::API_FIELD_NAME, false);
 	}
 
-	public function get_videos_data( $videos ) {
+	public function get_videos_data($videos) {
 
 		$key = $this->get_api_key();
 
-		if ( ! $key ) {
+		if ( ! $key) {
 			return array();
 		}
 
 		$urls = array_map(
-			function ( $video ) {
+			function($video) {
 				return $video->get_url();
 			},
 			$videos
 		);
 
-		$response_body = $this->api_client->get_data( $urls );
+		$response_body = $this->api_client->get_data($urls);
 
-		if ( ! is_string( $response_body ) || ! $response_body ) {
+		if ( ! is_string($response_body) || ! $response_body) {
 			return array();
 		}
 
-		$response = json_decode( $response_body, true );
+		$response = json_decode($response_body, true);
 
 		$video_list = $response['data'];
 
-		if ( ! is_array( $video_list ) ) {
+		if ( ! is_array($video_list)) {
 			// Return if we cant parse the response.
 			return array();
 		}
 
-		return array_filter( array_map( array( $this, 'get_video_from_video_data' ), $video_list ) );
+		return array_filter(array_map(array($this, 'get_video_from_video_data'), $video_list));
 
 	}
 
-	public function get_video_from_video_data( $vimeo_video_data ) {
+	public function get_video_from_video_data($vimeo_video_data) {
 
-		if ( ! $vimeo_video_data ) {
+		if ( ! $vimeo_video_data) {
 			// If valid data not supplied dont init the object.
 			return false;
 		}
@@ -66,41 +66,41 @@ 
 		$video->description = $vimeo_video_data['description'];
 
 		$video->content_url = $vimeo_video_data['link'];
-		$video->embed_url   = 'https://player.vimeo.com/video/' . $this->get_id( $vimeo_video_data );
-		if ( is_numeric( $vimeo_video_data['duration'] ) ) {
-			$video->duration = 'PT' . $vimeo_video_data['duration'] . 'S';
+		$video->embed_url   = 'https://player.vimeo.com/video/'.$this->get_id($vimeo_video_data);
+		if (is_numeric($vimeo_video_data['duration'])) {
+			$video->duration = 'PT'.$vimeo_video_data['duration'].'S';
 		}
 		$video->upload_date    = $vimeo_video_data['release_time'];
-		$video->thumbnail_urls = $this->set_thumbnail_urls( $vimeo_video_data );
+		$video->thumbnail_urls = $this->set_thumbnail_urls($vimeo_video_data);
 		$video->id             = $video->content_url;
 
-		if ( array_key_exists( 'stats', $vimeo_video_data )
-			 && array_key_exists( 'plays', $vimeo_video_data['stats'] ) ) {
+		if (array_key_exists('stats', $vimeo_video_data)
+			 && array_key_exists('plays', $vimeo_video_data['stats'])) {
 			$video->views = (int) $vimeo_video_data['stats']['plays'];
 		}
 
 		return $video;
 	}
 
-	private function get_id( $api_response_data ) {
-		return str_replace( '/videos/', '', $api_response_data['uri'] );
+	private function get_id($api_response_data) {
+		return str_replace('/videos/', '', $api_response_data['uri']);
 	}
 
-	private function set_thumbnail_urls( $api_response_data ) {
+	private function set_thumbnail_urls($api_response_data) {
 
-		if ( ! array_key_exists( 'pictures', $api_response_data ) || ! array_key_exists(
+		if ( ! array_key_exists('pictures', $api_response_data) || ! array_key_exists(
 			'sizes',
 			$api_response_data['pictures']
-		) ) {
+		)) {
 			return array();
 		}
-		if ( ! is_array( $api_response_data['pictures']['sizes'] ) ) {
+		if ( ! is_array($api_response_data['pictures']['sizes'])) {
 			return array();
 		}
 		$pictures = $api_response_data['pictures']['sizes'];
 
 		return array_map(
-			function ( $picture_data ) {
+			function($picture_data) {
 				return $picture_data['link'];
 			},
 			$pictures