Merge pull request #184 from jrfnl/feature/fix-undefined-notice

Sanity check the `preg_match()` values for width/height before using. Props @jrfnl
ResponsiveImagesCG · Sep 25, 2015 · 20301c7 · 20301c7
2 parents 4bc3295 + ae2bcbf
commit 20301c7
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/wp-tevko-responsive-images.php b/wp-tevko-responsive-images.php
@@ -342,13 +342,12 @@ function _tevkori_filter_content_images_callback( $image ) {
 	}
 
 	if ( $id && false === $size ) {
-		preg_match( '/width="([0-9]+)"/', $atts, $width );
-		preg_match( '/height="([0-9]+)"/', $atts, $height );
-
-		$size = array(
-			(int) $width[1],
-			(int) $height[1]
-		);
+		if ( preg_match( '/ width="([0-9]+)"/', $atts, $width ) && preg_match( '/ height="([0-9]+)"/', $atts, $height ) ) {
+			$size = array(
+				(int) $width[1],
+				(int) $height[1]
+			);
+		}
 	}
 
 	/*