refactor: update lazyload data availability checks and adjust skip setting

- Renamed method for checking data availability in PageProfiler for clarity.
- Updated lazyload replacer and tag replacer to use the new method.
- Adjusted the 'skip_lazyload_images' setting from 3 to 2 for improved functionality.

Author: selul

inc/lazyload_replacer.php

@@ -461,7 +461,7 @@ public function can_lazyload_for( $url, $tag = '' ) {
 			$image_id                   = $this->get_id_by_url( $url );
 			$is_in_all_viewports        = Optml_Manager::instance()->page_profiler->is_in_all_viewports( $image_id );
 			$is_lcp_image               = Optml_Manager::instance()->page_profiler->is_lcp_image_in_all_viewports( $image_id );
-			$no_viewport_data_available = ! Optml_Manager::instance()->page_profiler->check_data_availability();
+			$no_viewport_data_available = ! Optml_Manager::instance()->page_profiler->is_data_available();
 
 			if ( OPTML_DEBUG ) {
 				if ( $is_in_all_viewports ) {
@@ -531,7 +531,7 @@ public function get_svg_for( $width, $height, $url = null ) {
 			list( $width, $height ) = $sizes;
 		}
 		// If the width is not found the url might be an offloaded attachment so we can get the width and height from the metadata.
-		if ( ! is_numeric( $width ) ) {
+		if ( ! is_numeric( $width ) && ! empty( $url ) ) {
 			$attachment_id = $this->attachment_url_to_post_id( $url );
 			$meta = wp_get_attachment_metadata( $attachment_id );
 			$width = $meta['width'] ?? false;

inc/settings.php

@@ -93,7 +93,7 @@ class Optml_Settings {
 		'transfer_status'            => 'disabled',
 		'cloud_images'               => 'enabled',
 		'strip_metadata'             => 'enabled',
-		'skip_lazyload_images'       => 3,
+		'skip_lazyload_images'       => 2,
 		'defined_image_sizes'        => [],
 		'banner_frontend'            => 'disabled',
 		'offloading_status'          => 'disabled',

inc/tag_replacer.php

@@ -433,9 +433,8 @@ public function regular_tag_replace( $new_tag, $original_url, $new_url, $optml_a
 
 		if ( $this->settings->is_lazyload_type_viewport() ) {
 			$image_id                   = $this->get_id_by_url( $original_url );
-			$is_lcp_image               = Optml_Manager::instance()->page_profiler->is_lcp_image_in_all_viewports( $image_id );
-			$no_viewport_data_available = ! Optml_Manager::instance()->page_profiler->check_data_availability();
-
+			$is_lcp_image               = Optml_Manager::instance()->page_profiler->is_lcp_image_in_all_viewports( $image_id ); 
+			$no_viewport_data_available = ! Optml_Manager::instance()->page_profiler->is_data_available();
 			if ( OPTML_DEBUG ) {
 					do_action( 'optml_log', 'Adding fetchpriority image is LCP ' . $original_url . '|' . $image_id );
 			}

inc/v2/PageProfiler/Profile.php

@@ -452,17 +452,14 @@ public static function get_active_devices(): array {
 	 *
 	 * @return bool
 	 */
-	public function check_data_availability(): bool {
-		$has_data = false;
-
+	public function is_data_available(): bool { 
 		foreach ( self::get_active_devices() as $device ) {
 			if ( empty( self::$current_profile_data[ $device ] ) ) {
-				$has_data = true;
-				break;
+				return false;
 			}
 		}
 
-		return $has_data;
+		return true;
 	}
 
 	/**