insideout10 / wordlift-plugin

src/wordlift/videoobject/parser/class-classic-editor-parser.php

Indentation +40 added lines, -40 removed lines

@@ -7,45 +7,45 @@
 
 class Classic_Editor_Parser implements Parser {
 
-	public function get_videos( $post_id ) {
-
-		$post = get_post( $post_id );
-
-		$content = $post->post_content;
-
-		$line_matches      = array();
-		$paragraph_matches = array();
-
-		/**
-		 * This replicates the wp autoembed function, instead of replacing we capture
-		 * all the urls.
-		 */
-		if ( preg_match( '#(^|\s|>)https?://#i', $content ) ) {
-			// Find URLs on their own line.
-			preg_match_all( '|^(\s*)(https?://[^\s<>"]+)(\s*)$|im', $content, $line_matches, PREG_SET_ORDER );
-			// Find URLs in their own paragraph.
-			preg_match_all( '|(<p(?: [^>]*)?>\s*)(https?://[^\s<>"]+)(\s*<\/p>)|i', $content, $paragraph_matches, PREG_SET_ORDER );
-		}
-
-		$regex_matches = array_merge( $line_matches, $paragraph_matches );
-		$matches       = array_map( array( $this, 'get_url_from_match' ), $regex_matches );
-
-		$matches = array_values( array_unique( array_filter( $matches ) ) );
-
-		return array_map( array( $this, 'url_to_embedded_video_object' ), $matches );
-	}
-
-	/**
-	 * @param $url
-	 *
-	 * @return Embedded_Video
-	 */
-	private static function url_to_embedded_video_object( $url ) {
-		return Embedded_Video_Factory::get_embedded_video( trim( $url ) );
-	}
-
-	public static function get_url_from_match( $match ) {
-		return array_key_exists( 0, $match ) ? $match[0] : false;
-	}
+    public function get_videos( $post_id ) {
+
+        $post = get_post( $post_id );
+
+        $content = $post->post_content;
+
+        $line_matches      = array();
+        $paragraph_matches = array();
+
+        /**
+         * This replicates the wp autoembed function, instead of replacing we capture
+         * all the urls.
+         */
+        if ( preg_match( '#(^|\s|>)https?://#i', $content ) ) {
+            // Find URLs on their own line.
+            preg_match_all( '|^(\s*)(https?://[^\s<>"]+)(\s*)$|im', $content, $line_matches, PREG_SET_ORDER );
+            // Find URLs in their own paragraph.
+            preg_match_all( '|(<p(?: [^>]*)?>\s*)(https?://[^\s<>"]+)(\s*<\/p>)|i', $content, $paragraph_matches, PREG_SET_ORDER );
+        }
+
+        $regex_matches = array_merge( $line_matches, $paragraph_matches );
+        $matches       = array_map( array( $this, 'get_url_from_match' ), $regex_matches );
+
+        $matches = array_values( array_unique( array_filter( $matches ) ) );
+
+        return array_map( array( $this, 'url_to_embedded_video_object' ), $matches );
+    }
+
+    /**
+     * @param $url
+     *
+     * @return Embedded_Video
+     */
+    private static function url_to_embedded_video_object( $url ) {
+        return Embedded_Video_Factory::get_embedded_video( trim( $url ) );
+    }
+
+    public static function get_url_from_match( $match ) {
+        return array_key_exists( 0, $match ) ? $match[0] : false;
+    }
 
 }

Spacing +13 added lines, -13 removed lines

@@ -7,9 +7,9 @@ 
 
 class Classic_Editor_Parser implements Parser {
 
-	public function get_videos( $post_id ) {
+	public function get_videos($post_id) {
 
-		$post = get_post( $post_id );
+		$post = get_post($post_id);
 
 		$content = $post->post_content;
 
@@ -20,19 +20,19 @@ 
 		 * This replicates the wp autoembed function, instead of replacing we capture
 		 * all the urls.
 		 */
-		if ( preg_match( '#(^|\s|>)https?://#i', $content ) ) {
+		if (preg_match('#(^|\s|>)https?://#i', $content)) {
 			// Find URLs on their own line.
-			preg_match_all( '|^(\s*)(https?://[^\s<>"]+)(\s*)$|im', $content, $line_matches, PREG_SET_ORDER );
+			preg_match_all('|^(\s*)(https?://[^\s<>"]+)(\s*)$|im', $content, $line_matches, PREG_SET_ORDER);
 			// Find URLs in their own paragraph.
-			preg_match_all( '|(<p(?: [^>]*)?>\s*)(https?://[^\s<>"]+)(\s*<\/p>)|i', $content, $paragraph_matches, PREG_SET_ORDER );
+			preg_match_all('|(<p(?: [^>]*)?>\s*)(https?://[^\s<>"]+)(\s*<\/p>)|i', $content, $paragraph_matches, PREG_SET_ORDER);
 		}
 
-		$regex_matches = array_merge( $line_matches, $paragraph_matches );
-		$matches       = array_map( array( $this, 'get_url_from_match' ), $regex_matches );
+		$regex_matches = array_merge($line_matches, $paragraph_matches);
+		$matches       = array_map(array($this, 'get_url_from_match'), $regex_matches);
 
-		$matches = array_values( array_unique( array_filter( $matches ) ) );
+		$matches = array_values(array_unique(array_filter($matches)));
 
-		return array_map( array( $this, 'url_to_embedded_video_object' ), $matches );
+		return array_map(array($this, 'url_to_embedded_video_object'), $matches);
 	}
 
 	/**
@@ -40,12 +40,12 @@ 
 	 *
 	 * @return Embedded_Video
 	 */
-	private static function url_to_embedded_video_object( $url ) {
-		return Embedded_Video_Factory::get_embedded_video( trim( $url ) );
+	private static function url_to_embedded_video_object($url) {
+		return Embedded_Video_Factory::get_embedded_video(trim($url));
 	}
 
-	public static function get_url_from_match( $match ) {
-		return array_key_exists( 0, $match ) ? $match[0] : false;
+	public static function get_url_from_match($match) {
+		return array_key_exists(0, $match) ? $match[0] : false;
 	}
 
 }

src/wordlift/videoobject/data/embedded-video/class-embedded-video-factory.php

Indentation +3 added lines, -3 removed lines

@@ -8,8 +8,8 @@
  */
 class Embedded_Video_Factory {
 
-	public static function get_embedded_video( $video_url ) {
-		return new Default_Embedded_Video( $video_url );
-	}
+    public static function get_embedded_video( $video_url ) {
+        return new Default_Embedded_Video( $video_url );
+    }
 
 }

Spacing +2 added lines, -2 removed lines

@@ -8,8 +8,8 @@
  */
 class Embedded_Video_Factory {
 
-	public static function get_embedded_video( $video_url ) {
-		return new Default_Embedded_Video( $video_url );
+	public static function get_embedded_video($video_url) {
+		return new Default_Embedded_Video($video_url);
 	}
 
 }

src/wordlift/videoobject/data/embedded-video/class-embedded-video.php

Indentation +2 added lines, -2 removed lines

@@ -12,8 +12,8 @@
  */
 interface Embedded_Video {
 
-	public function get_api_provider();
+    public function get_api_provider();
 
-	public function get_url();
+    public function get_url();
 
 }

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

Indentation +3 added lines, -3 removed lines

@@ -10,8 +10,8 @@
 
 class Video_Storage_Factory {
 
-	public static function get_storage() {
-		return new Meta_Storage();
-	}
+    public static function get_storage() {
+        return new Meta_Storage();
+    }
 
 }

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

Indentation +36 added lines, -36 removed lines

@@ -10,41 +10,41 @@
  */
 interface Storage {
 
-	/**
-	 * @param $post_id int Post id to add video.
-	 * @param $video Video The video object which needs to be saved.
-	 *
-	 * @return bool true if video is added, false otherwise.
-	 */
-	public function add_video( $post_id, $video );
-
-	/**
-	 * Return all video objects for the post
-	 *
-	 * @param $post_id int Post id to get all videos.
-	 *
-	 * @return array<Video>
-	 */
-	public function get_all_videos( $post_id );
-
-	/**
-	 * Remove the Video Object.
-	 *
-	 * @param $videos_to_be_removed array<Video>
-	 *
-	 * @param $post_id int
-	 *
-	 * @return bool return true if removed or false
-	 */
-	public function remove_videos( $videos_to_be_removed, $post_id );
-
-	/**
-	 * Remove all videos present for post id
-	 *
-	 * @param $post_id
-	 *
-	 * @return true if removed else false.
-	 */
-	public function remove_all_videos( $post_id );
+    /**
+     * @param $post_id int Post id to add video.
+     * @param $video Video The video object which needs to be saved.
+     *
+     * @return bool true if video is added, false otherwise.
+     */
+    public function add_video( $post_id, $video );
+
+    /**
+     * Return all video objects for the post
+     *
+     * @param $post_id int Post id to get all videos.
+     *
+     * @return array<Video>
+     */
+    public function get_all_videos( $post_id );
+
+    /**
+     * Remove the Video Object.
+     *
+     * @param $videos_to_be_removed array<Video>
+     *
+     * @param $post_id int
+     *
+     * @return bool return true if removed or false
+     */
+    public function remove_videos( $videos_to_be_removed, $post_id );
+
+    /**
+     * Remove all videos present for post id
+     *
+     * @param $post_id
+     *
+     * @return true if removed else false.
+     */
+    public function remove_all_videos( $post_id );
 
 }

Spacing +4 added lines, -4 removed lines

@@ -16,7 +16,7 @@ 
 	 *
 	 * @return bool true if video is added, false otherwise.
 	 */
-	public function add_video( $post_id, $video );
+	public function add_video($post_id, $video);
 
 	/**
 	 * Return all video objects for the post
@@ -25,7 +25,7 @@ 
 	 *
 	 * @return array<Video>
 	 */
-	public function get_all_videos( $post_id );
+	public function get_all_videos($post_id);
 
 	/**
 	 * Remove the Video Object.
@@ -36,7 +36,7 @@ 
 	 *
 	 * @return bool return true if removed or false
 	 */
-	public function remove_videos( $videos_to_be_removed, $post_id );
+	public function remove_videos($videos_to_be_removed, $post_id);
 
 	/**
 	 * Remove all videos present for post id
@@ -45,6 +45,6 @@ 
 	 *
 	 * @return true if removed else false.
 	 */
-	public function remove_all_videos( $post_id );
+	public function remove_all_videos($post_id);
 
 }

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(),
 			)
 		);