mahara-contributors team mailing list archive

[Robert Lyon <robertl@xxxxxxxxxxxxxxx>]

Ok there was a bug in my patch that slipped through.

However there still is a problem with embedding code into the
externalmedia block.

If I go to a YouTube video and select the 'Embed' option I get to copy code that looks like this:
<iframe width="420" height="315" src="//www.youtube.com/embed/WM5Gwzk3Vfc" frameborder="0" allowfullscreen></iframe>

Adding that to the externalmedia block gives me no video.

However if I go to a YouTube video and select the 'Embed' option and also tick the 'use old embed code' I get to copy code that looks like this:
<object width="420" height="315"><param name="movie" value="//www.youtube.com/v/WM5Gwzk3Vfc?version=3&amp;hl=en_US"></param><param name="allowFullScreen" value="true"></param><param name="allowscriptaccess" value="always"></param><embed src="//www.youtube.com/v/WM5Gwzk3Vfc?version=3&amp;hl=en_US" type="application/x-shockwave-flash" width="420" height="315" allowscriptaccess="always" allowfullscreen="true"></embed></object>

That code does work for me.

So the question is - is it easier to fix the problem where the iframe
code doesn't work or to tell everyone to click the 'use old embed code'
option. I think having both work is a better choice.

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contrib members
https://bugs.launchpad.net/bugs/1207140

Title:
  YouTube filter requires updates

Status in Mahara ePortfolio:
  Incomplete
Status in Mahara 1.6 series:
  Incomplete
Status in Mahara 1.7 series:
  Incomplete
Status in Mahara 1.8 series:
  Incomplete

Bug description:
  1. The iframe embed code for YouTube videos does not have the protocol
  included anymore thus causing problems when trying to embed the code
  into a site.

  The code reads:
  <iframe width="560" height="315" src="//www.youtube.com/embed/XXXXX?rel=0" frameborder="0" allowfullscreen></iframe>

  YouTube says:

  ''Just using //: instead of http:// or https:// means that you would
  like to use whichever protocol the parent page is using.

  So a page called http://example.com with include http://youtube.com
  and a page called https://example.com with include https://youtube.com

  It's intentional.''

  https://groups.google.com/a/googleproductforums.com/d/msgid/youtube
  /bce484af-b2e6-4f6e-9bc0-6c7617e08406%40googleproductforums.com

  2. When someone wants to use the YouTube URL from a mobile device, it
  is shortened to youtu.be. However, these videos can't be embedded
  because the filter doesn't understand that URL.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1207140/+subscriptions