[TYPO3-core] RFC: Feature Request #9625: Integrate possibility to extend TCEforms palettes easily

Martin Kutschker masi-no at spam-typo3.org
Tue Oct 28 23:28:14 CET 2008


Martin Kutschker schrieb:
> Oliver Hader schrieb:
>> Hi Ingo,
>>
>> Ingo Renner schrieb:
>>> Oliver Hader wrote:
>>>
>>> Hey Olly,
>>>
>>>> t3lib_extMgm::addNewFieldsToAllPalettesOfField
>>> it's quite obvious that the fields you're going to add are "New", don't
>>> you think? Can we therefore name it just addFieldsToAllPalettesOfField? ;)
>>>
>>>> Comments:
>>>> This patch will be used also for the OpenId integration (RFC #9474).
>>>> Furthermore this patch extends t3lib_div::trimExplode by the possibility
>>>> to limit the result items like it's possible with PHP's explode.
>>> I'd argue this should have been a different patch to be really clean,
>>> but so what...
>>>
>>> will do a review later, just wanted to do some nitpicking for now ;)
>> I attached the new patch. The t3lib_div::trimExplode issue is extracted
>> to RFC #9656 - that patch is required for this RFC!
> 
> Committed to trunk. It lacks a formal review but worked for me when
> testing #9474 (Integrate OpenID authentication support). And as the
> release manager wanted the latter in I had no choice ;-)

I forgot to mention that I followed Ingo's suggestion and renamed the
function to addFieldsToAllPalettesOfField().

Masi


More information about the TYPO3-team-core mailing list