Merge pull request #403 from cloudinary/fix/cloudinary-only-storage-transformations

Fix cloudinary only storage transformations

Author: pereirinha

php/class-media.php

@@ -1370,6 +1370,31 @@ public function is_cloudinary_url( $url ) {
 		return $test_parts['host'] === $cld_url;
 	}
 
+	/**
+	 * Check if a url is pointing to Cloudinary sync folder.
+	 *
+	 * @param string $url The tested URL.
+	 *
+	 * @return bool
+	 */
+	public function is_cloudinary_sync_folder( $url ) {
+		$path = wp_parse_url( $url, PHP_URL_PATH );
+		$parts = explode( '/', $path );
+
+		// Remove public id and file name.
+		array_splice( $parts, -2 );
+
+		foreach ( $parts as $part ) {
+			array_shift( $parts );
+			if ( 'v' === $part[0] && is_numeric( substr( $part, 1 ) ) ) {
+				break;
+			}
+		}
+
+		// Check for the Cloudinary folder.
+		return implode( '/', $parts ) === $this->get_cloudinary_folder( false );
+	}
+
 	/**
 	 * Add media tab template.
 	 */

php/media/class-filter.php

@@ -318,8 +318,8 @@ public function filter_out_local( $content, $tags = 'img' ) {
 			$url           = $this->get_url_from_tag( $asset );
 			$attachment_id = $this->get_id_from_tag( $asset );
 
-			// Check if this is not already a cloudinary url.
-			if ( $this->media->is_cloudinary_url( $url ) ) {
+			// Check if this is not already a cloudinary url and if is not in the sync folder, for Cloudinary only storage cases.
+			if ( $this->media->is_cloudinary_url( $url ) && ! $this->media->is_cloudinary_sync_folder( $url ) ) {
 				// Is a content based ID. If has a cloudinary ID, it's from an older plugin version.
 				// Check if has an ID, and push update to reset.
 				if ( ! empty( $attachment_id ) && ! $this->media->plugin->components['sync']->is_synced( $attachment_id ) ) {