From 1251ac11dcdc1c761cd1a5e7a6a921b1916d4401 Mon Sep 17 00:00:00 2001 From: Jeremy Herve Date: Thu, 6 Sep 2018 16:33:33 +0200 Subject: [PATCH] Responsive Videos: avoid notice when param is not given. Fixes #3048 This was already fixed in #3243, but got reverted by mistake in ab7f2e0d3619bfc483723727bb427656e1581898 --- modules/theme-tools/responsive-videos.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/modules/theme-tools/responsive-videos.php b/modules/theme-tools/responsive-videos.php index eb97130edddd6..a5dba5ac3818e 100644 --- a/modules/theme-tools/responsive-videos.php +++ b/modules/theme-tools/responsive-videos.php @@ -61,11 +61,11 @@ function jetpack_responsive_videos_embed_html( $html ) { } /** - * Check if oEmbed is YouTube or Vimeo before wrapping. + * Check if oEmbed is a `$video_patterns` provider video before wrapping. * * @return string */ -function jetpack_responsive_videos_maybe_wrap_oembed( $html, $url ) { +function jetpack_responsive_videos_maybe_wrap_oembed( $html, $url = null ) { if ( empty( $html ) || ! is_string( $html ) || ! $url ) { return $html; }