insideout10 / wordlift-plugin

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

Indentation +34 added lines, -34 removed lines

@@ -11,50 +11,50 @@
  */
 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 ) !== false;
-			}
-		);
+        return array_filter(
+            $video_urls,
+            function ( $item ) {
+                return strpos( $item, 'https://cdn.jwplayer.com/v2/media/', 0 ) !== false;
+            }
+        );
 
-	}
+    }
 
-	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
+    }
 
 }

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,12 +34,12 @@ 
 
 	}
 
-	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 ) !== false;
+			function($item) {
+				return strpos($item, 'https://cdn.jwplayer.com/v2/media/', 0) !== false;
 			}
 		);
 

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

Indentation +12 added lines, -12 removed lines

@@ -9,20 +9,20 @@
  */
 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();
-		} elseif ( self::VIMEO === $config ) {
-			return Vimeo_Client::get_instance();
-		} elseif ( self::JWPLAYER === $config ) {
-			return Jw_Player_Client::get_instance();
-		}
+    public static function get_client( $config ) {
+        if ( self::YOUTUBE === $config ) {
+            return Youtube_Client::get_instance();
+        } elseif ( self::VIMEO === $config ) {
+            return Vimeo_Client::get_instance();
+        } elseif ( self::JWPLAYER === $config ) {
+            return Jw_Player_Client::get_instance();
+        }
 
-	}
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -15,12 +15,12 @@
 
 	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();
-		} elseif ( self::VIMEO === $config ) {
+		} elseif (self::VIMEO === $config) {
 			return Vimeo_Client::get_instance();
-		} elseif ( self::JWPLAYER === $config ) {
+		} elseif (self::JWPLAYER === $config) {
 			return Jw_Player_Client::get_instance();
 		}
 

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

Indentation +52 added lines, -52 removed lines

@@ -11,75 +11,75 @@
  */
 class Youtube_Client extends Singleton implements Client {
 
-	const YOUTUBE_REGEX = '/(?:https?:\/\/)?(?:youtu\.be\/|(?:www\.|m\.)?youtube\.com\/(?:watch|v|embed)(?:\.php)?(?:\?.*v=|\/))([a-zA-Z0-9\_-]+)/m';
+    const YOUTUBE_REGEX = '/(?:https?:\/\/)?(?:youtu\.be\/|(?:www\.|m\.)?youtube\.com\/(?:watch|v|embed)(?:\.php)?(?:\?.*v=|\/))([a-zA-Z0-9\_-]+)/m';
 
-	public static $requests_sent = 0;
+    public static $requests_sent = 0;
 
-	public static function get_api_key_option_name() {
-		return '__wl_video_object_youtube_api_key';
-	}
+    public static function get_api_key_option_name() {
+        return '__wl_video_object_youtube_api_key';
+    }
 
-	public static function get_api_key() {
-		return get_option( self::get_api_key_option_name(), false );
-	}
+    public static function get_api_key() {
+        return get_option( self::get_api_key_option_name(), false );
+    }
 
-	public function get_api_url() {
-		return 'https://www.googleapis.com/youtube/v3/videos';
-	}
+    public function get_api_url() {
+        return 'https://www.googleapis.com/youtube/v3/videos';
+    }
 
-	public function get_data( $video_urls ) {
-		$video_ids = $this->get_video_ids_as_string( $video_urls );
-		$url       = add_query_arg(
-			array(
-				'part' => 'snippet,contentDetails,statistics,liveStreamingDetails',
-				'id'   => $video_ids,
-				'key'  => $this->get_api_key(),
-			),
-			$this->get_api_url()
-		);
+    public function get_data( $video_urls ) {
+        $video_ids = $this->get_video_ids_as_string( $video_urls );
+        $url       = add_query_arg(
+            array(
+                'part' => 'snippet,contentDetails,statistics,liveStreamingDetails',
+                'id'   => $video_ids,
+                'key'  => $this->get_api_key(),
+            ),
+            $this->get_api_url()
+        );
 
-		$response = wp_remote_get( $url );
+        $response = wp_remote_get( $url );
 
-		++ self::$requests_sent;
+        ++ self::$requests_sent;
 
-		return wp_remote_retrieve_body( $response );
-	}
+        return wp_remote_retrieve_body( $response );
+    }
 
-	private function get_video_ids_as_string( $video_urls ) {
-		// validate the urls.
-		$video_urls = array_filter(
-			$video_urls,
-			function ( $url ) {
-				return filter_var( $url, FILTER_VALIDATE_URL );
-			}
-		);
+    private function get_video_ids_as_string( $video_urls ) {
+        // validate the urls.
+        $video_urls = array_filter(
+            $video_urls,
+            function ( $url ) {
+                return filter_var( $url, FILTER_VALIDATE_URL );
+            }
+        );
 
-		// extract the video ids.
-		return join( ',', $this->get_video_ids( $video_urls ) );
-	}
+        // extract the video ids.
+        return join( ',', $this->get_video_ids( $video_urls ) );
+    }
 
-	public function get_video_ids( $video_urls ) {
+    public function get_video_ids( $video_urls ) {
 
-		$regex = self::YOUTUBE_REGEX;
+        $regex = self::YOUTUBE_REGEX;
 
-		$video_ids = array_map(
-			function ( $url ) use ( $regex ) {
-				$matches = array();
-				preg_match( $regex, $url, $matches );
+        $video_ids = array_map(
+            function ( $url ) use ( $regex ) {
+                $matches = array();
+                preg_match( $regex, $url, $matches );
 
-				// Return video id or return false.
-				if ( isset( $matches[1] ) && is_string( $matches[1] ) ) {
-					  return $matches[1];
-				}
+                // Return video id or return false.
+                if ( isset( $matches[1] ) && is_string( $matches[1] ) ) {
+                        return $matches[1];
+                }
 
-				return false;
+                return false;
 
-			},
-			$video_urls
-		);
+            },
+            $video_urls
+        );
 
-		return array_values( array_filter( $video_ids ) );
+        return array_values( array_filter( $video_ids ) );
 
-	}
+    }
 
 }

Spacing +14 added lines, -16 removed lines

@@ -20,15 +20,15 @@ 
 	}
 
 	public static function get_api_key() {
-		return get_option( self::get_api_key_option_name(), false );
+		return get_option(self::get_api_key_option_name(), false);
 	}
 
 	public function get_api_url() {
 		return 'https://www.googleapis.com/youtube/v3/videos';
 	}
 
-	public function get_data( $video_urls ) {
-		$video_ids = $this->get_video_ids_as_string( $video_urls );
+	public function get_data($video_urls) {
+		$video_ids = $this->get_video_ids_as_string($video_urls);
 		$url       = add_query_arg(
 			array(
 				'part' => 'snippet,contentDetails,statistics,liveStreamingDetails',
@@ -38,37 +38,35 @@ 
 			$this->get_api_url()
 		);
 
-		$response = wp_remote_get( $url );
+		$response = wp_remote_get($url);++ self::$requests_sent;
 
-		++ self::$requests_sent;
-
-		return wp_remote_retrieve_body( $response );
+		return wp_remote_retrieve_body($response);
 	}
 
-	private function get_video_ids_as_string( $video_urls ) {
+	private function get_video_ids_as_string($video_urls) {
 		// validate the urls.
 		$video_urls = array_filter(
 			$video_urls,
-			function ( $url ) {
-				return filter_var( $url, FILTER_VALIDATE_URL );
+			function($url) {
+				return filter_var($url, FILTER_VALIDATE_URL);
 			}
 		);
 
 		// extract the video ids.
-		return join( ',', $this->get_video_ids( $video_urls ) );
+		return join(',', $this->get_video_ids($video_urls));
 	}
 
-	public function get_video_ids( $video_urls ) {
+	public function get_video_ids($video_urls) {
 
 		$regex = self::YOUTUBE_REGEX;
 
 		$video_ids = array_map(
-			function ( $url ) use ( $regex ) {
+			function($url) use ($regex) {
 				$matches = array();
-				preg_match( $regex, $url, $matches );
+				preg_match($regex, $url, $matches);
 
 				// Return video id or return false.
-				if ( isset( $matches[1] ) && is_string( $matches[1] ) ) {
+				if (isset($matches[1]) && is_string($matches[1])) {
 					  return $matches[1];
 				}
 
@@ -78,7 +76,7 @@ 
 			$video_urls
 		);
 
-		return array_values( array_filter( $video_ids ) );
+		return array_values(array_filter($video_ids));
 
 	}
 

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

Indentation +60 added lines, -60 removed lines

@@ -11,83 +11,83 @@
  */
 class Vimeo_Client extends Singleton implements Client {
 
-	public static $requests_sent = 0;
+    public static $requests_sent = 0;
 
-	const VIMEO_URL_REGEX = '/https?:\/\/(?:www\.|player\.)?vimeo.com\/(?:channels\/(?:\w+\/)?|groups\/([^\/]*)\/videos\/|album\/(\d+)\/video\/|video\/|)(\d+)(?:$|\/|\?)/';
+    const VIMEO_URL_REGEX = '/https?:\/\/(?:www\.|player\.)?vimeo.com\/(?:channels\/(?:\w+\/)?|groups\/([^\/]*)\/videos\/|album\/(\d+)\/video\/|video\/|)(\d+)(?:$|\/|\?)/';
 
-	public function get_data( $video_urls ) {
+    public function get_data( $video_urls ) {
 
-		$vimeo_ids = $this->get_video_ids( $video_urls );
+        $vimeo_ids = $this->get_video_ids( $video_urls );
 
-		$vimeo_api_ids = array_map(
-			function ( $item ) {
-				return '/videos/' . $item;
-			},
-			$vimeo_ids
-		);
+        $vimeo_api_ids = array_map(
+            function ( $item ) {
+                return '/videos/' . $item;
+            },
+            $vimeo_ids
+        );
 
-		$ids = join( ',', $vimeo_api_ids );
+        $ids = join( ',', $vimeo_api_ids );
 
-		if ( ! $ids ) {
-			return array();
-		}
+        if ( ! $ids ) {
+            return array();
+        }
 
-		$api_url = $this->get_api_url() . '/videos/';
-		$api_url = add_query_arg(
-			array(
-				'uris'   => $ids,
-				'fields' => 'name,description,link,uri,duration,release_time,pictures,stats',
-			),
-			$api_url
-		);
+        $api_url = $this->get_api_url() . '/videos/';
+        $api_url = add_query_arg(
+            array(
+                'uris'   => $ids,
+                'fields' => 'name,description,link,uri,duration,release_time,pictures,stats',
+            ),
+            $api_url
+        );
 
-		$response = wp_remote_get(
-			$api_url,
-			array(
-				'headers' => array(
-					'Authorization' => 'bearer ' . $this->get_api_key(),
-				),
-			)
-		);
+        $response = wp_remote_get(
+            $api_url,
+            array(
+                'headers' => array(
+                    'Authorization' => 'bearer ' . $this->get_api_key(),
+                ),
+            )
+        );
 
-		++ self::$requests_sent;
+        ++ self::$requests_sent;
 
-		return wp_remote_retrieve_body( $response );
-	}
+        return wp_remote_retrieve_body( $response );
+    }
 
-	public static function get_api_key() {
-		return get_option( self::get_api_key_option_name(), false );
-	}
+    public static function get_api_key() {
+        return get_option( self::get_api_key_option_name(), false );
+    }
 
-	public static function get_api_key_option_name() {
-		return '_wl_videoobject_vimeo_api_key';
-	}
+    public static function get_api_key_option_name() {
+        return '_wl_videoobject_vimeo_api_key';
+    }
 
-	public function get_api_url() {
-		return 'https://api.vimeo.com';
-	}
+    public function get_api_url() {
+        return 'https://api.vimeo.com';
+    }
 
-	public function get_video_ids( $video_urls ) {
+    public function get_video_ids( $video_urls ) {
 
-		$that = $this;
+        $that = $this;
 
-		return array_filter(
-			array_map(
-				function ( $video_url ) use ( $that ) {
-					if ( ! $video_url ) {
-						  return false;
-					}
-					preg_match( $that::VIMEO_URL_REGEX, $video_url, $matches );
+        return array_filter(
+            array_map(
+                function ( $video_url ) use ( $that ) {
+                    if ( ! $video_url ) {
+                            return false;
+                    }
+                    preg_match( $that::VIMEO_URL_REGEX, $video_url, $matches );
 
-					if ( ! array_key_exists( 3, $matches ) ) {
-						return false;
-					}
+                    if ( ! array_key_exists( 3, $matches ) ) {
+                        return false;
+                    }
 
-					return $matches[3];
+                    return $matches[3];
 
-				},
-				$video_urls
-			)
-		);
-	}
+                },
+                $video_urls
+            )
+        );
+    }
 }

Spacing +16 added lines, -18 removed lines

@@ -15,24 +15,24 @@ 
 
 	const VIMEO_URL_REGEX = '/https?:\/\/(?:www\.|player\.)?vimeo.com\/(?:channels\/(?:\w+\/)?|groups\/([^\/]*)\/videos\/|album\/(\d+)\/video\/|video\/|)(\d+)(?:$|\/|\?)/';
 
-	public function get_data( $video_urls ) {
+	public function get_data($video_urls) {
 
-		$vimeo_ids = $this->get_video_ids( $video_urls );
+		$vimeo_ids = $this->get_video_ids($video_urls);
 
 		$vimeo_api_ids = array_map(
-			function ( $item ) {
-				return '/videos/' . $item;
+			function($item) {
+				return '/videos/'.$item;
 			},
 			$vimeo_ids
 		);
 
-		$ids = join( ',', $vimeo_api_ids );
+		$ids = join(',', $vimeo_api_ids);
 
-		if ( ! $ids ) {
+		if ( ! $ids) {
 			return array();
 		}
 
-		$api_url = $this->get_api_url() . '/videos/';
+		$api_url = $this->get_api_url().'/videos/';
 		$api_url = add_query_arg(
 			array(
 				'uris'   => $ids,
@@ -45,18 +45,16 @@ 
 			$api_url,
 			array(
 				'headers' => array(
-					'Authorization' => 'bearer ' . $this->get_api_key(),
+					'Authorization' => 'bearer '.$this->get_api_key(),
 				),
 			)
-		);
-
-		++ self::$requests_sent;
+		);++ self::$requests_sent;
 
-		return wp_remote_retrieve_body( $response );
+		return wp_remote_retrieve_body($response);
 	}
 
 	public static function get_api_key() {
-		return get_option( self::get_api_key_option_name(), false );
+		return get_option(self::get_api_key_option_name(), false);
 	}
 
 	public static function get_api_key_option_name() {
@@ -67,19 +65,19 @@ 
 		return 'https://api.vimeo.com';
 	}
 
-	public function get_video_ids( $video_urls ) {
+	public function get_video_ids($video_urls) {
 
 		$that = $this;
 
 		return array_filter(
 			array_map(
-				function ( $video_url ) use ( $that ) {
-					if ( ! $video_url ) {
+				function($video_url) use ($that) {
+					if ( ! $video_url) {
 						  return false;
 					}
-					preg_match( $that::VIMEO_URL_REGEX, $video_url, $matches );
+					preg_match($that::VIMEO_URL_REGEX, $video_url, $matches);
 
-					if ( ! array_key_exists( 3, $matches ) ) {
+					if ( ! array_key_exists(3, $matches)) {
 						return false;
 					}
 

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

Indentation +31 added lines, -31 removed lines

@@ -7,39 +7,39 @@
  * This acts as interface for api clients
  */
 interface Client {
-	/**
-	 *
-	 * @param $video_urls array<string> Array of urls.
-	 *
-	 * Response body or false if the request cant be made or failed.
-	 * @return string | false
-	 */
-	public function get_data( $video_urls );
+    /**
+     *
+     * @param $video_urls array<string> Array of urls.
+     *
+     * Response body or false if the request cant be made or failed.
+     * @return string | false
+     */
+    public function get_data( $video_urls );
 
-	/**
-	 * @return string
-	 */
-	public static function get_api_key();
-	/**
-	 * Returns the option where the api key is stored.
-	 *
-	 * @return string
-	 */
-	public static function get_api_key_option_name();
+    /**
+     * @return string
+     */
+    public static function get_api_key();
+    /**
+     * Returns the option where the api key is stored.
+     *
+     * @return string
+     */
+    public static function get_api_key_option_name();
 
-	/**
-	 * The api base url.
-	 *
-	 * @return string
-	 */
-	public function get_api_url();
+    /**
+     * The api base url.
+     *
+     * @return string
+     */
+    public function get_api_url();
 
-	/**
-	 * Get video ids from URLs.
-	 *
-	 * @param $video_urls array<string>
-	 * @return array<string>
-	 */
-	public function get_video_ids( $video_urls  );
+    /**
+     * Get video ids from URLs.
+     *
+     * @param $video_urls array<string>
+     * @return array<string>
+     */
+    public function get_video_ids( $video_urls  );
 
 }

Spacing +2 added lines, -2 removed lines

@@ -14,7 +14,7 @@ 
 	 * Response body or false if the request cant be made or failed.
 	 * @return string | false
 	 */
-	public function get_data( $video_urls );
+	public function get_data($video_urls);
 
 	/**
 	 * @return string
@@ -40,6 +40,6 @@ 
 	 * @param $video_urls array<string>
 	 * @return array<string>
 	 */
-	public function get_video_ids( $video_urls  );
+	public function get_video_ids($video_urls);
 
 }

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

Indentation +170 added lines, -170 removed lines

@@ -11,175 +11,175 @@
 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, 2 );
-		add_action( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 10, 2 );
-		$this->video_storage = $video_storage;
-	}
-
-	public function wl_after_get_jsonld( $jsonld, $post_id ) {
-		if ( 0 === count( $jsonld ) ) {
-			return $jsonld;
-		}
-		$current_item = $jsonld[0];
-
-		if ( ! is_array( $current_item ) || ! 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 ) {
-
-		$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, 2 );
+        add_action( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 10, 2 );
+        $this->video_storage = $video_storage;
+    }
+
+    public function wl_after_get_jsonld( $jsonld, $post_id ) {
+        if ( 0 === count( $jsonld ) ) {
+            return $jsonld;
+        }
+        $current_item = $jsonld[0];
+
+        if ( ! is_array( $current_item ) || ! 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 ) {
+
+        $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

@@ -21,53 +21,53 @@ 
 	 *
 	 * @param $video_storage Storage
 	 */
-	public function __construct( $video_storage ) {
-		add_action( 'wl_post_jsonld', array( $this, 'wl_post_jsonld' ), 10, 2 );
-		add_action( 'wl_after_get_jsonld', array( $this, 'wl_after_get_jsonld' ), 10, 2 );
+	public function __construct($video_storage) {
+		add_action('wl_post_jsonld', array($this, 'wl_post_jsonld'), 10, 2);
+		add_action('wl_after_get_jsonld', array($this, 'wl_after_get_jsonld'), 10, 2);
 		$this->video_storage = $video_storage;
 	}
 
-	public function wl_after_get_jsonld( $jsonld, $post_id ) {
-		if ( 0 === count( $jsonld ) ) {
+	public function wl_after_get_jsonld($jsonld, $post_id) {
+		if (0 === count($jsonld)) {
 			return $jsonld;
 		}
 		$current_item = $jsonld[0];
 
-		if ( ! is_array( $current_item ) || ! array_key_exists( '@type', $current_item ) ) {
+		if ( ! is_array($current_item) || ! 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);
 	}
 
 	/**
@@ -76,32 +76,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 ) {
+	public function wl_post_jsonld($jsonld, $post_id) {
 
-		$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;
@@ -109,7 +109,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;
 	}
@@ -119,18 +119,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;
 			}
@@ -145,11 +145,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(
@@ -159,7 +159,7 @@ 
 				);
 			}
 
-			if ( $video->is_live_video ) {
+			if ($video->is_live_video) {
 				$single_jsonld['publication'] = array(
 					'@type'           => 'BroadcastEvent',
 					'isLiveBroadcast' => true,
@@ -174,12 +174,12 @@ 
 		return $jsonld;
 	}
 
-	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/class-loader.php

Spacing +7 added lines, -7 removed lines

@@ -25,25 +25,25 @@
 
 	public function init_all_dependencies() {
 		$video_storage = Video_Storage_Factory::get_storage();
-		new Jsonld( $video_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 = 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 = 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 = new Rest_Controller($background_process);
 		$rest_controller->register_all_routes();
 
 		$post_edit_screen = new Post_Edit_Screen();

Indentation +56 added lines, -56 removed lines

@@ -24,60 +24,60 @@
  */
 class Loader extends Default_Loader {
 
-	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 );
-
-		$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();
-
-		$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();
-
-		$post_edit_screen = new Post_Edit_Screen();
-		$post_edit_screen->init();
-
-		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
-		 * Get videos from jw player.
-		 */
-		$jw_player_capture_videos = new Jw_Player_Capture();
-		$jw_player_capture_videos->init();
-
-		/**
-		 * Validate API Key for Youtube, Vimeo.
-		 */
-		new Video_Key_Validation_Service();
-
-	}
-
-	public function get_feature_slug() {
-		return 'videoobject';
-	}
-
-	public function get_feature_default_value() {
-		return false;
-	}
+    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 );
+
+        $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();
+
+        $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();
+
+        $post_edit_screen = new Post_Edit_Screen();
+        $post_edit_screen->init();
+
+        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
+         * Get videos from jw player.
+         */
+        $jw_player_capture_videos = new Jw_Player_Capture();
+        $jw_player_capture_videos->init();
+
+        /**
+         * Validate API Key for Youtube, Vimeo.
+         */
+        new Video_Key_Validation_Service();
+
+    }
+
+    public function get_feature_slug() {
+        return 'videoobject';
+    }
+
+    public function get_feature_default_value() {
+        return false;
+    }
 }

src/wordlift/videoobject/background-process/class-videos-data-source.php

Indentation +63 added lines, -63 removed lines

@@ -6,74 +6,74 @@
 
 class Videos_Data_Source extends Data_Source {
 
-	/**
-	 * @var \Wordlift_Log_Service
-	 */
-	private $log;
+    /**
+     * @var \Wordlift_Log_Service
+     */
+    private $log;
 
-	public function __construct( $state_storage_key ) {
-		parent::__construct( $state_storage_key );
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
-	}
+    public function __construct( $state_storage_key ) {
+        parent::__construct( $state_storage_key );
+        $this->log = \Wordlift_Log_Service::get_logger( get_class() );
+    }
 
-	public function next() {
-		$this->log->debug( 'Received video data source index as ' . $this->get_state()->index );
-		$this->log->debug( 'Count set to ' . $this->get_batch_size() );
+    public function next() {
+        $this->log->debug( 'Received video data source index as ' . $this->get_state()->index );
+        $this->log->debug( 'Count set to ' . $this->get_batch_size() );
 
-		return get_posts(
-			array(
-				'fields'      => 'ids',
-				'post_status' => 'any',
-				'numberposts' => $this->get_batch_size(),
-				'offset'      => $this->get_state()->index,
-				/**
-				* @return array<string> An array of supported post types for the video import procedure.
-				* @since 3.31.4
-				* Filter name : wl_videoobject_procedure_post_types
-				*/
-			'post_type'       => array_unique(
-				apply_filters(
-					'wl_videoobject_procedure_post_types',
-					array(
-						'post',
-						'page',
-					)
-				)
-			),
-			)
-		);
-	}
+        return get_posts(
+            array(
+                'fields'      => 'ids',
+                'post_status' => 'any',
+                'numberposts' => $this->get_batch_size(),
+                'offset'      => $this->get_state()->index,
+                /**
+                 * @return array<string> An array of supported post types for the video import procedure.
+                 * @since 3.31.4
+                 * Filter name : wl_videoobject_procedure_post_types
+                 */
+            'post_type'       => array_unique(
+                apply_filters(
+                    'wl_videoobject_procedure_post_types',
+                    array(
+                        'post',
+                        'page',
+                    )
+                )
+            ),
+            )
+        );
+    }
 
-	public function count() {
-		return count(
-			get_posts(
-				array(
-					'fields'      => 'ids',
-					'numberposts' => - 1,
-					'post_status' => 'any',
-					/**
-					* @return array<string> An array of supported post types for the video import procedure.
-					* @since 3.31.4
-					* Filter name : wl_videoobject_procedure_post_types
-					*/
-					'post_type'   => array_unique(
-						apply_filters(
-							'wl_videoobject_procedure_post_types',
-							array(
-								'post',
-								'page',
-							)
-						)
-					),
-				)
-			)
-		);
+    public function count() {
+        return count(
+            get_posts(
+                array(
+                    'fields'      => 'ids',
+                    'numberposts' => - 1,
+                    'post_status' => 'any',
+                    /**
+                     * @return array<string> An array of supported post types for the video import procedure.
+                     * @since 3.31.4
+                     * Filter name : wl_videoobject_procedure_post_types
+                     */
+                    'post_type'   => array_unique(
+                        apply_filters(
+                            'wl_videoobject_procedure_post_types',
+                            array(
+                                'post',
+                                'page',
+                            )
+                        )
+                    ),
+                )
+            )
+        );
 
-	}
+    }
 
-	public function get_batch_size() {
-		// For now use only 5 in order to prevent exceeding api limit.
-		return 5;
-	}
+    public function get_batch_size() {
+        // For now use only 5 in order to prevent exceeding api limit.
+        return 5;
+    }
 
 }

Spacing +6 added lines, -6 removed lines

@@ -11,14 +11,14 @@ 
 	 */
 	private $log;
 
-	public function __construct( $state_storage_key ) {
-		parent::__construct( $state_storage_key );
-		$this->log = \Wordlift_Log_Service::get_logger( get_class() );
+	public function __construct($state_storage_key) {
+		parent::__construct($state_storage_key);
+		$this->log = \Wordlift_Log_Service::get_logger(get_class());
 	}
 
 	public function next() {
-		$this->log->debug( 'Received video data source index as ' . $this->get_state()->index );
-		$this->log->debug( 'Count set to ' . $this->get_batch_size() );
+		$this->log->debug('Received video data source index as '.$this->get_state()->index);
+		$this->log->debug('Count set to '.$this->get_batch_size());
 
 		return get_posts(
 			array(
@@ -49,7 +49,7 @@ 
 			get_posts(
 				array(
 					'fields'      => 'ids',
-					'numberposts' => - 1,
+					'numberposts' => -1,
 					'post_status' => 'any',
 					/**
 					* @return array<string> An array of supported post types for the video import procedure.

src/wordlift/videoobject/filters/class-post-filter.php

Indentation +21 added lines, -21 removed lines

@@ -10,32 +10,32 @@
  */
 class Post_Filter {
 
-	/**
-	 * @var Video_Processor
-	 */
-	private $video_processor;
+    /**
+     * @var Video_Processor
+     */
+    private $video_processor;
 
-	public function __construct( $video_processor ) {
-		$this->video_processor = $video_processor;
-	}
+    public function __construct( $video_processor ) {
+        $this->video_processor = $video_processor;
+    }
 
-	public function init() {
-		add_action( 'save_post', array( $this, 'save_post' ), 10, 2 );
-	}
+    public function init() {
+        add_action( 'save_post', array( $this, 'save_post' ), 10, 2 );
+    }
 
-	/**
-	 * @param $post_id int
-	 * @param $post \WP_Post
-	 * @param $update bool
-	 */
-	public function save_post( $post_id, $post ) {
+    /**
+     * @param $post_id int
+     * @param $post \WP_Post
+     * @param $update bool
+     */
+    public function save_post( $post_id, $post ) {
 
-		if ( wp_is_post_revision( $post_id ) || wp_is_post_autosave( $post_id ) ) {
-			return;
-		}
+        if ( wp_is_post_revision( $post_id ) || wp_is_post_autosave( $post_id ) ) {
+            return;
+        }
 
-		$this->video_processor->process_video_urls( $post, $post_id );
+        $this->video_processor->process_video_urls( $post, $post_id );
 
-	}
+    }
 
 }

Spacing +5 added lines, -5 removed lines

@@ -15,12 +15,12 @@ 
 	 */
 	private $video_processor;
 
-	public function __construct( $video_processor ) {
+	public function __construct($video_processor) {
 		$this->video_processor = $video_processor;
 	}
 
 	public function init() {
-		add_action( 'save_post', array( $this, 'save_post' ), 10, 2 );
+		add_action('save_post', array($this, 'save_post'), 10, 2);
 	}
 
 	/**
@@ -28,13 +28,13 @@ 
 	 * @param $post \WP_Post
 	 * @param $update bool
 	 */
-	public function save_post( $post_id, $post ) {
+	public function save_post($post_id, $post) {
 
-		if ( wp_is_post_revision( $post_id ) || wp_is_post_autosave( $post_id ) ) {
+		if (wp_is_post_revision($post_id) || wp_is_post_autosave($post_id)) {
 			return;
 		}
 
-		$this->video_processor->process_video_urls( $post, $post_id );
+		$this->video_processor->process_video_urls($post, $post_id);
 
 	}