PHP error on embedding Youtube Shorts

Attaching the patch I have written to address this issue.

@Luke.Leber, added new condition for fix this issue "PHP error on embedding Youtube Shorts". please kindly review the MR

Hello @abhishek_virasat,

There was already a patch added in #2. Your merge request does not add any additional value and will not be considered for contribution credit. Please collaborate with community peers versus starting new work.

Thanks for the bug report, Tom. I'll take some time this morning to update the tests and add additional coverage with this.

These new test cases should prove that there's a bug to fix here.

The last submitted patch, 8: 3392082-8-test-only.patch, failed testing. View results →

Luke.Leber → closed merge request !20

...and this should prove that it's fixed!

I slightly extended #2 in order to also support playlists...not sure how I missed that originally. I'm going to move onto manual testing if the test bot is green with this one :-).

Manual testing all passed on www.worldcampus.psu.edu on Friday, so I will be merging and cutting a new patch release Monday morning, following internal peer review, unless otherwise noted.

Cheers

Committed and pushed to 2.0.x, thanks Tom!

I'll cut 2.0.4 stable this week to ship the fixes.

Automatically closed - issue fixed for 2 weeks with no activity.