Skip to content
Snippets Groups Projects
Commit 779a6ff4 authored by Tim Rücker's avatar Tim Rücker Committed by Christian Kuhn
Browse files

[BUGFIX] Add missing functionality for YouTube "related" parameter

This parameter indicates whether the player should show
related videos when playback of the initial video ends.
Supported values are 0 and 1. The default value is 1.

Releases: master,7.6
Resolves: #77710
Change-Id: I95576369b8dae579b39298095024bdbddc75ebc6
Reviewed-on: https://review.typo3.org/50031


Tested-by: default avatarTYPO3com <no-reply@typo3.com>
Reviewed-by: default avatarMarkus Klein <markus.klein@typo3.org>
Tested-by: default avatarMarkus Klein <markus.klein@typo3.org>
Reviewed-by: default avatarFrank Naegler <frank.naegler@typo3.org>
Reviewed-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
Tested-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
parent 97afb12b
Branches
Tags
No related merge requests found
......@@ -115,6 +115,9 @@ class YouTubeRenderer implements FileRendererInterface
if (!empty($options['loop'])) {
$urlParams[] = 'loop=1&amp;playlist=' . $videoId;
}
if (isset($options['relatedVideos'])) {
$urlParams[] = 'rel=' . (int)(bool)$options['relatedVideos'];
}
if (!isset($options['enablejsapi']) || !empty($options['enablejsapi'])) {
$urlParams[] = 'enablejsapi=1&amp;origin=' . GeneralUtility::getIndpEnv('TYPO3_REQUEST_HOST');
}
......
......@@ -149,7 +149,7 @@ class YouTubeRendererTest extends UnitTestCase
/**
* @test
*/
public function renderOutputWithAutoplayAndWithoutControllsIsCorrect()
public function renderOutputWithAutoplayAndWithoutControlsIsCorrect()
{
/** @var File|\PHPUnit_Framework_MockObject_MockObject $fileResourceMock */
$fileResourceMock = $this->createMock(File::class);
......@@ -159,4 +159,18 @@ class YouTubeRendererTest extends UnitTestCase
$this->subject->render($fileResourceMock, '300m', '200', ['controls' => 0, 'autoplay' => 1])
);
}
/**
* @test
*/
public function renderOutputWithRelatedVideosTurnedOffIsCorrect()
{
/** @var File|\PHPUnit_Framework_MockObject_MockObject $fileResourceMock */
$fileResourceMock = $this->createMock(File::class);
$this->assertSame(
'<iframe src="https://www.youtube.com/embed/7331?autohide=1&amp;controls=2&amp;rel=0&amp;enablejsapi=1&amp;origin=http://test.server.org&amp;showinfo=0" allowfullscreen width="300" height="200"></iframe>',
$this->subject->render($fileResourceMock, '300m', '200', ['relatedVideos' => 0])
);
}
}
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment