Merge pull request #1026 from cloudinary/fix/division-by-zero-error

Fix DivisionByZeroError in Responsive_Breakpoints

Author: aatanasov-cloudinary

php/delivery/class-responsive-breakpoints.php

@@ -127,12 +127,22 @@ public function add_features( $tag_element ) {
 	 * @return array
 	 */
 	protected function apply_breakpoints( $tag_element ) {
+		if ( empty( $tag_element['id'] ) ) {
+			return $tag_element;
+		}
+
+		$settings  = $this->settings->get_value( 'media_display' );
+		$max       = $settings['max_width'];
+		$min       = $settings['min_width'];
+		$width     = (int) $tag_element['width'];
+		$height    = (int) $tag_element['height'];
+		$debug_key = 'responsive_breakpoints_' . $tag_element['id'];
+
+		if ( ! $width || ! $height ) {
+			Utils::log( 'Missing width or height for image with ID: ' . $tag_element['id'], $debug_key );
+			return $tag_element;
+		}
 
-		$settings            = $this->settings->get_value( 'media_display' );
-		$max                 = $settings['max_width'];
-		$min                 = $settings['min_width'];
-		$width               = $tag_element['width'];
-		$height              = $tag_element['height'];
 		$size_tag            = '-' . $width . 'x' . $height . '.';
 		$step                = $settings['pixel_step'];
 		$ratio               = $width / $height;