[TYPO3-dam-devel] RFC #16349: Adapt dam_tt_content to the TCA changes in TYPO3 4.5

[Lorenz Ulrich]

Hi Chris

Thanks.

 > some CGL related remarks:
 >
 > instead of using
 > ## CTypes "text w/image" and "image"
 > you should use
 > // CTypes "text w/image" and "image"
 >
 > You are mixing up tabs and spaces, spaces are in the lines:
 > 
t3lib_extMgm::addToAllTCAtypes('tt_content','tx_damttcontent_files','textpic','after:image'); 

 > - twice
 > $TCA['tt_content']['types']['textpic']['showitem'] = str_replace(',
 > image;', ', tx_damttcontent_files;',
 > $TCA['tt_content']['types']['textpic']['showitem']);

Sorry, didn't setup my new IDE properly. I will clean these things up 
when commiting.

Best regards,


Lorenz


Am 10.01.2011 12:09, schrieb Chris Müller:
> +1 by reading and testing on TYPO3 4.5 trunk
>

>
>
> Am 10.01.2011 08:51, schrieb Bjoern Pedersen:
>> Am 06.01.2011 18:45, schrieb Lorenz Ulrich:
>>> Hi DAM people
>>>
>>> This is an SVN patch request.
>>>
>>> Type:
>>> Bugfix
>>>
>>> Bugtracker reference:
>>> http://bugs.typo3.org/view.php?id=16349
>>>
>>> Branches:
>>> trunk
>>>
>>> Problem:
>>> The TCA for the image fields changed considerably for 4.5 (with the
>>> tt_content overhaul). Every field was moved into a palette so the
>>> current hacky implementation of replacing the original image field with
>>> the DAM media field fails.
>>>
>>> Solution:
>>> t3lib_extMgm::addToAllTCAtypes was improved with the ability to replace
>>> fields. These modifications are now used to replace the field instead of
>>> the old str_replace hack.
>>>
>>> On this occasion (more details about the why in the bugtracker) the
>>> possibility to add/replace the DAM field separately for Text w/image and
>>> Image was removed.
>>>
>>> Notes:
>>> You need to update your extension configuration in the EM. Updating with
>>> the new EM (beta3) did not work for me, maybe someone can double-check
>>> that.
>>>
>>> Best regards,
>>>
>>> Lorenz
>>
>> +1 by testing on 4.5 trunk (EM RC1 is working correctly)
>