[TYPO3-core] RFC: Feature Request #8487: Enable configuration of filesize labels for CType uploads

[Martin Kutschker]

Oliver Hader schrieb:
> Hi again,
> 
> Oliver Hader schrieb:
>> Hi Ernesto,
>>
>> Ernesto Baschny [cron IT] schrieb:
>>> I am ok with that, but it "breaks compatibility". I would tend to
>>> agree that the old default " | K| M| G" is not really good, but this
>>> is how it was displayed until now and maybe some web-design "breaks"
>>> if this string gets longer.
>>>
>>> Even better would be to pass $fileD['filesize'] through stdWrap,
>>> something like:
>>>
>>>     $this->cObj->stdWrap($fileD['filesize'], $conf['filesize.']);
>>>
>>> and have a default in TypoScript:
>>>
>>>     filesize.bytes = 1
>>>     filesize.bytes.label = {$styles.content.uploads.filesize.labels}
>>>
>>> (note: bytes is a stdWrap function that calls t3lib_div::formatSize)
>>> This should provides enough flexibility to format the sizes of the
>>> files the way the user wants it.
>>>
>>> So I +1 on the change (either "new property" or my stdWrap
>>> suggestion), but only if that part is included in TypoScript in a
>>> compatVersion 4.3 section.
>>
>> I changed the labels back to the old default " | K| M| G". Thus, I
>> think the compat version is not necessary anymore, is it? Furthermore
>> I took your sdtWrap suggestion. New patch is attached...
> 
> Just saw that it doesn't make sense to have "wrap" and "wrap2" in the
> patch, when the other markup stuff is not configurable. Will move it
> back to the user function in final commit.

Yes, please do so. Also my +1 for this patch.

Masi