CoreCommunity ExtensionsIncubatorDistributionsTYPO3 4.5 ProjectsTYPO3 4.7 ProjectsTYPO3 6.0 ProjectsTYPO3 6.1 ProjectsTYPO3 6.2 Projects (+)

Task #13223

Refactore so called "interfaces"

Added by Nikolas Hagelstein over 2 years ago. Updated 4 months ago.

Status:Resolved Start date:2011-02-21
Priority:Must have Due date:
Assignee:Nikolas Hagelstein % Done:

0%

Category:Core
Target version:2.0.1
TYPO3 Version:6.1 Has patch:No
PHP Version: Complexity:
Votes: 0

Description

Not sure what the exact purpose is right now. But at least the naming sounds strange to me.
Georg, could you point out? I would like to move them to a different location and restructure them a little.

Associated revisions

Revision 237a4be9
Added by Georg Ringer 4 months ago

[TASK] Rename interfaces to mediarenderer

Change-Id: Ie04252f3975282db55fbce87cf82c92be8929d9c
Resolves: #13223
Releases:
Reviewed-on: https://review.typo3.org/18363
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

History

Updated by Georg Ringer over 2 years ago

Hey,

don't you remember our mail/skype discussion?

it was about rendering media elements and changing the rendering process without much magic (just set the order and name of classes in TS). Those classes are then called in the MediaFactoryViewHelper.

Function enabled() gets the media element and checks if the current class is capable of rendering the element (e.g. if file type is flv.
Function render() renders the media element: e.g. flv with flowplayer, quicktime with its ugly embedd html and videosites uses the t3lib_media thingy to get correct url and includes it using a swfobject.

I believe that the reason behind all this is awesome and the correct way to go, renaming and restructuring is fine for me.

Updated by Gerrit Code Review 4 months ago

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/18363

Updated by Georg Ringer 4 months ago

  • Status changed from Needs Feedback to Resolved
  • Target version changed from 1.0.0 to 2.0.1
  • Has patch set to No
  • TYPO3 Version set to 6.1

Also available in: Atom PDF