Project

General

Profile

Actions

Bug #58569

closed

Content element "media" does not render Youtube videos

Added by Thomas Off about 10 years ago. Updated over 5 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 4 (0 open4 closed)

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
Actions #1

Updated by Marco Seiler about 10 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

Actions #2

Updated by Gerrit Code Review about 10 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

Actions #3

Updated by Gerrit Code Review about 10 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

Actions #4

Updated by Markus Klein about 10 years ago

  • Is Regression changed from No to Yes
Actions #5

Updated by Thomas Löffler about 10 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF