Abstract is syncable delivery types

Author: Marco Pereirinha

php/class-media.php

@@ -1688,14 +1688,7 @@ public function media_column( $cols ) {
 	 */
 	public function media_column_value( $column_name, $attachment_id ) {
 		if ( 'cld_status' === $column_name ) {
-			if (
-				$this->is_media( $attachment_id ) &&
-				in_array(
-					$this->get_media_delivery( $attachment_id ),
-					$this->get_syncable_delivery_types(),
-					true
-				)
-			) :
+			if ( $this->sync->is_syncable( $attachment_id ) ) :
 				$status = array(
 					'state' => 'inactive',
 					'note'  => esc_html__( 'Not Synced', 'cloudinary' ),

php/class-sync.php

@@ -244,13 +244,7 @@ public function can_sync( $attachment_id, $type = 'file' ) {
 		}
 
 		// Can sync only syncable delivery types.
-		if (
-			! in_array(
-				$this->managers['media']->get_media_delivery( $attachment_id ),
-				$this->managers['media']->get_syncable_delivery_types(),
-				true
-			)
-		) {
+		if ( ! $this->is_syncable( $attachment_id ) ) {
 			$can = false;
 		}
 
@@ -328,6 +322,29 @@ public function generate_public_id( $attachment_id ) {
 		return ltrim( $public_id, '/' );
 	}
 
+	/**
+	 * Is syncable asset.
+	 *
+	 * @param int $attachment_id The attachment ID.
+	 *
+	 * @return bool
+	 */
+	public function is_syncable( $attachment_id ) {
+		$syncable = false;
+		if (
+			$this->managers['media']->is_media( $attachment_id )
+			&& in_array(
+				$this->managers['media']->get_media_delivery( $attachment_id ),
+				$this->managers['media']->get_syncable_delivery_types(),
+				true
+			)
+		) {
+			$syncable = true;
+		}
+
+		return $syncable;
+	}
+
 	/**
 	 * Register a new sync type.
 	 *

php/sync/class-push-sync.php

@@ -202,13 +202,7 @@ public function process_assets( $attachments = array() ) {
 				continue;
 			}
 			// Skip unsyncable delivery types.
-			if (
-				! in_array(
-					$this->media->get_media_delivery( $attachment_id ),
-					$this->media->get_syncable_delivery_types(),
-					true
-				)
-			) {
+			if ( ! $this->sync->is_syncable( $attachment_id ) ) {
 				continue;
 			}
 			// Flag attachment as being processed.

php/sync/class-upload-sync.php

@@ -118,13 +118,7 @@ public function add_inline_action( $actions, $post ) {
 			if ( ! $this->media->is_local_media( $post->ID ) ) {
 				return $actions;
 			}
-			if (
-				! in_array(
-					$this->media->get_media_delivery( $post->ID ),
-					$this->media->get_syncable_delivery_types(),
-					true
-				)
-			) {
+			if ( ! $this->sync->is_syncable( $post->ID ) ) {
 				return $actions;
 			}
 			if ( ! $this->plugin->components['sync']->is_synced( $post->ID ) ) {
@@ -168,13 +162,7 @@ public function handle_bulk_actions( $location, $action, $post_ids ) {
 						$this->sync->delete_cloudinary_meta( $post_id );
 						continue;
 					}
-					if (
-						! in_array(
-							$this->media->get_media_delivery( $post_id ),
-							$this->media->get_syncable_delivery_types(),
-							true
-						)
-					) {
+					if ( ! $this->sync->is_syncable( $post_id ) ) {
 						continue;
 					}
 					$this->sync->set_signature_item( $post_id, 'file', '' );

php/traits/trait-cli.php

@@ -201,11 +201,7 @@ protected function process_sync( $posts, $total ) {
 			if (
 				! $this->plugin->get_component( 'sync' )->is_synced( $asset )
 				&& $this->plugin->get_component( 'media' )->is_local_media( $asset )
-				&& in_array(
-					$this->plugin->get_component( 'media' )->get_media_delivery( $asset ),
-					$this->plugin->get_component( 'media' )->get_syncable_delivery_types(),
-					true
-				)
+				&& $this->plugin->get_component( 'sync' )->is_syncable( $asset )
 			) {
 				$this->plugin->get_component( 'sync' )->managers['push']->process_assets( $asset, $bar );
 			}
@@ -248,13 +244,8 @@ protected function process_analyze( $posts, $total ) {
 			$done ++;
 			$key = '_cld_unsupported';
 			if (
-				$this->plugin->get_component( 'media' )->is_media( $asset )
-				&& $this->plugin->get_component( 'media' )->is_local_media( $asset )
-				&& in_array(
-					$this->plugin->get_component( 'media' )->get_media_delivery( $asset ),
-					$this->plugin->get_component( 'media' )->get_syncable_delivery_types(),
-					true
-				)
+				$this->plugin->get_component( 'media' )->is_local_media( $asset )
+				&& $this->plugin->get_component( 'sync' )->is_syncable( $asset )
 			) {
 				// Add a key.
 				$key = '_cld_synced';