[TYPO3-core] RFC #17088: Cleanup: CSH display in new record wizard should use new API

Jeff Segars jsegars at alumni.rice.edu
Tue Jan 18 16:51:31 CET 2011


On 1/17/11 10:08 AM, François Suter wrote:
> This is an SVN patch request.
>
> Type: Code cleanup
>
> Bugtracker references:
> http://bugs.typo3.org/view.php?id=17088
>
> Branches:
> Trunk
>
> Problem:
> The new record wizard does not use the new CSH API.
>
> Solution:
> Use it, to increase uniformity within the Core.
>
> Notes:
> BTW should we deprecate t3lib_BEfunc::chsItem()? I haven't searched for
> other places where it's used in the Core, but I would if we agree on
> that. It seems to me that t3lib_BEfunc::wrapInHelp() can really replace
> it entirely.
>
> Cheers
>

+1 on reading and testing.

It looks like most of the usage of cshItem() is related to docheader 
buttons.  I would vote for deprecation because the only way it differs 
from wrapInHelp() is honoring options we've removed in 4.5 (hiding help 
text, icon mode vs text mode, etc).

Thanks,
Jeff


More information about the TYPO3-team-core mailing list