fix issue when the offloaded images were marked as done on the legacy system but not on the new system.

Improve the logic the attachment cache logic

Author: selul

inc/media_offload.php

@@ -813,7 +813,8 @@ public function upload_and_update_existing_images( $image_ids ) {
 		foreach ( $image_ids as $id ) {
 			if ( self::is_uploaded_image( wp_get_attachment_metadata( $id )['file'] ) ) {
 				// if this meta flag below failed at the initial update but the file meta above is updated it will cause an infinite query loop
-				update_post_meta( $id, 'optimole_offload', 'true' );
+				update_post_meta( $id, self::META_KEYS['offloaded'], 'true' );
+				update_post_meta( $id, self::OM_OFFLOADED_FLAG, true );
 				$success_up ++;
 				continue;
 			}

inc/traits/dam_offload_utils.php

@@ -223,21 +223,46 @@ private function get_scaled_url( $url ) {
 	private function is_scaled_url( $url ) {
 		return strpos( $url, '-scaled.' ) !== false;
 	}
+
+	/**
+	 * Check if the image has been offloaded completely.
+	 *
+	 * @param int $id The attachment ID.
+	 *
+	 * @return bool
+	 */
+	private function is_completed_offload( $id ) {
+		// This is the flag that is used to mark the image as offloaded at the end.
+		$completed = ! empty( get_post_meta( $id, Optml_Media_Offload::OM_OFFLOADED_FLAG, true ) );
+		if ( $completed ) {
+			return true;
+		}
+		// In some rare cases the image is offloaded but the flag is not set so we can alternatively check this using the file path.
+		$meta = wp_get_attachment_metadata( $id );
+		if ( ! isset( $meta['file'] ) ) {
+			return false;
+		}
+		if ( Optml_Media_Offload::is_uploaded_image( $meta['file'] ) ) {
+			return true;
+		}
+
+		return false;
+	}
 	/**
 	 * Get the attachment ID from URL.
 	 *
-	 * @param string $url The attachment URL.
+	 * @param string $input_url The attachment URL.
 	 *
 	 * @return int
 	 */
-	private function attachment_url_to_post_id( $url ) {
-		$cached = Optml_Attachment_Cache::get_cached_attachment_id( $url );
+	private function attachment_url_to_post_id( $input_url ) {
+		$cached = Optml_Attachment_Cache::get_cached_attachment_id( $input_url );
 
 		if ( $cached !== false ) {
 			return $cached;
 		}
 
-		$url = $this->strip_image_size( $url );
+		$url = $this->strip_image_size( $input_url );
 
 		$attachment_id = attachment_url_to_postid( $url );
 
@@ -265,7 +290,7 @@ private function attachment_url_to_post_id( $url ) {
 				$attachment_id = attachment_url_to_postid( $scaled_url );
 			}
 		}
-		Optml_Attachment_Cache::set_cached_attachment_id( $url, $attachment_id );
+		Optml_Attachment_Cache::set_cached_attachment_id( $input_url, $attachment_id );
 
 		return $attachment_id;
 	}