Bug #58569

Content element "media" does not render Youtube videos

Added by Thomas Off over 8 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Frontend
Target version:
Start date:
2014-05-06
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

In TYPO3 6.2 the "media" content element does not handle Youtube videos correctly: it renders a wrong URL into the object tag.

I compared 6.1 and 6.2, added a "media" element in both and used the same Youtube URL (https://www.youtube.com/watch?v=e0AW-wtPl1o); in 6.1 I get the following URL in the object tag:

http://www.youtube.com/v/e0AW-wtPl1o?fs=1

In 6.2 I get the following:

http://www.youtube.com/embed/e0AW-wtPl1o?fs=1

When examining the URLs, I found out that the one from 6.1 loads a Flash file, while the URL generated in 6.2 fetches an HTML page (which in turn does not play in the Flash object). So in my opinion there are two options:

1. Revert the URL rendering to what it was in 6.1.
2. Change the media element so that Youtube videos are not rendered using Flash, but an Iframe (which is what Youtube recommends anyway).


Related issues

Related to TYPO3 Core - Bug #51728: MediaWizardProvider::process_youtube does not handle all url variantsClosedSebastian Fischer2013-09-04

Actions
Related to TYPO3 Core - Bug #58986: Content element Media: 2 bug with External URL in 6.2.2: wrong http:// handlingClosed2014-05-21

Actions
Related to TYPO3 Core - Bug #33090: New way to embed youtube videosClosedBenni Mack2012-01-10

Actions
Has duplicate TYPO3 Core - Bug #58985: Content element Media: 1 bug with External URL in 6.2.2: wrong data-valueClosed2014-05-21

Actions
#1

Updated by Marco Seiler over 8 years ago

I can confirm this issue.

The responsible code is in typo3/sysext/frontend/Classes/MediaWizard/MediaWizardProvider.php on line 141:

$url = $this->getUrlSchema() . 'www.youtube.com/embed/' . $videoId . '?fs=1';

I think it should be
$url = $this->getUrlSchema() . 'www.youtube.com/v/' . $videoId . '?fs=1';

Was introduced while resolving this issue: http://forge.typo3.org/issues/51728

#2

Updated by Gerrit Code Review over 8 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29952

#3

Updated by Gerrit Code Review about 8 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/29952

#4

Updated by Markus Klein about 8 years ago

  • Is Regression changed from No to Yes
#5

Updated by Thomas Löffler about 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#6

Updated by Benni Mack almost 4 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF