Project

General

Profile

Actions

Bug #44797

closed

Rendering html5 video uses flvplayer for fallback instead of flowplayer

Added by Arjen Hoekema almost 12 years ago. Updated about 9 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Frontend
Target version:
-
Start date:
2013-01-25
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

It looks like the configuration is not properly merged when using the flowplayer as a fallback for rendering html5 video.

Traced this down to the following lines in 'class.tslib_content_media.php' (line 215 in TYPO3 4.7):

switch ($renderType) {
case 'flowplayer':
$conf[$conf['type'] . '.'] = array_merge((array) $conf['mimeConf.']['flowplayer.'][$conf['type'] . '.'], $typeConf);
$conf = array_merge((array) $conf['mimeConf.']['flowplayer.'], $conf);
unset($conf['mimeConf.']);

The configuration should be merged the other way around. This also applies to the rendering of "qt":
case 'qt' :
$conf[$conf['type'] . '.'] = array_merge($conf['mimeConf.']['swfobject.'][$conf['type'] . '.'], $typeConf);

Here the rendering of 'swfobject' is used instead of 'mimeConf.qtobject'.

Actions #1

Updated by Oliver Hader almost 12 years ago

Thanks for digging into this issue. Since you already have a possible solution, could you please push that to our Git review system?
If you're new to Git, find according step-by-step tutorial here
Don't hesitate to get back to me by mailto in case of further questions.

Actions #2

Updated by Oliver Hader almost 12 years ago

  • Status changed from New to Accepted
  • Target version set to 4.7.8
Actions #3

Updated by Mathias Schreiber almost 10 years ago

  • Target version changed from 4.7.8 to 7.2 (Frontend)
  • Is Regression set to No
Actions #4

Updated by Benni Mack over 9 years ago

  • Target version changed from 7.2 (Frontend) to 7.4 (Backend)
Actions #5

Updated by Wouter Wolters over 9 years ago

  • Assignee set to Marc Bastian Heinrichs
Actions #6

Updated by Susanne Moog over 9 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions #7

Updated by Benni Mack about 9 years ago

  • Category changed from Frontend to 1606
  • Assignee deleted (Marc Bastian Heinrichs)
  • Target version deleted (7.5)
Actions #8

Updated by Georg Ringer about 9 years ago

mediace has been extracted from the core and is not needed anymore using CMS7. however it can be now found at github ant the issue has been moved to https://github.com/FriendsOfTYPO3/mediace/issues/10

Actions #9

Updated by Georg Ringer about 9 years ago

  • Status changed from Accepted to Closed
Actions

Also available in: Atom PDF