Feature #21062

Content Element Media needs stdWrap to change path (e.g. for DAM)

Added by Georg Ringer over 11 years ago. Updated over 10 years ago.

Status:
Closed
Priority:
Should have
Category:
Communication
Target version:
-
Start date:
2009-09-15
Due date:
% Done:

0%

Estimated time:
PHP Version:
5.2
Tags:
Complexity:
Sprint Focus:

Description

currently there is no way to change the path of a file if the plugin is used because code like this is used:
-----------
// it is a content element
$this->readFlexformIntoConf($flexParams, $conf['parameter.']);
$url = $conf['parameter.']['mmFile'];
} else {
// it is a TS object
$url = $this->stdWrap($conf['file'], $conf['file.']);
----------

so a $url = $this->stdWrap($url, $conf['file.']); is missing in the if clause.

the attached patch makes it easy to change the path if e.g. DAM is used. Furthermore this could be really nice if cloud hosting is used

(issue imported from #M11965)


Files

patch-cemedia-stdwrap.patch (566 Bytes) patch-cemedia-stdwrap.patch Administrator Admin, 2009-09-15 22:20
#1

Updated by Steffen Kamper over 11 years ago

this is already done:

$filename = $this->stdWrap($conf['file'], $conf['file.']);

in SWFOBJECT

#2

Updated by Georg Ringer over 11 years ago

sorry I dont get it working with that because several things change without having the stdWrap

e.g. $mode = is_file(PATH_site . $url) ? 'file' : 'url'; will change and this changes some other things again.

furthermore it makes not so much sense by having stdwrap if this function called with TS! IMO the handling should be the same no matter if the whole thing is called by ts or not!

thanks for correcting me if I did something wrong

#3

Updated by Georg Ringer over 11 years ago

another investigation:

if $mode = file there can be no $renderType defined through the fileending because t3lib_div::split_fileref($filename); is empty? wouldnt make it sense.

so it is not possible to get to the swfobject because in the 1st place there is not a real file (because of DAM), so there is no file type hinting and so i would need to do all by myself ...
and everything could be changed by either setting stdwrap at the beginning.

i dont see another way

#4

Updated by Steffen Kamper over 11 years ago

I see, yes that's a good addition. I'll test it.

#5

Updated by Steffen Kamper over 11 years ago

committed to trunk, rev 5940

Also available in: Atom PDF