http://forge.typo3.org/http://forge.typo3.org/themes/typo3_forge/favicon/favicon.png?17058661692012-02-25T22:50:45ZTYPO3 ForgeTYPO3 Core - Feature #34283: IRRE - Possibility to render custom header controlshttp://forge.typo3.org/issues/34283?journal_id=1039622012-02-25T22:50:45ZGerrit Code Review
<ul><li><strong>Status</strong> changed from <i>New</i> to <i>Under Review</i></li></ul><p>Patch set 1 for branch <strong>master</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/9207">http://review.typo3.org/9207</a></p> TYPO3 Core - Feature #34283: IRRE - Possibility to render custom header controlshttp://forge.typo3.org/issues/34283?journal_id=1039712012-02-25T23:17:38ZGerrit Code Review
<ul></ul><p>Patch set 2 for branch <strong>master</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/9207">http://review.typo3.org/9207</a></p> TYPO3 Core - Feature #34283: IRRE - Possibility to render custom header controlshttp://forge.typo3.org/issues/34283?journal_id=1039922012-02-26T15:12:01ZGerrit Code Review
<ul></ul><p>Patch set 3 for branch <strong>master</strong> has been pushed to the review server.<br />It is available at <a class="external" href="http://review.typo3.org/9207">http://review.typo3.org/9207</a></p> TYPO3 Core - Feature #34283: IRRE - Possibility to render custom header controlshttp://forge.typo3.org/issues/34283?journal_id=1039932012-02-26T15:17:12ZGeorg Ringer
<ul></ul><p>I would propose this change.</p>
<p>simplify the api by using $customControlConfig directly as input for t3lib_div::callUserFunction(). a call would then look like</p>
<pre>
'customControls' => array(
'fo' => 'EXT:news/Classes/Hooks/Test.php:Tx_News_Hooks_Test->user_fo',
),
</pre><br />instead of <br /><pre>
'customControls' => array(
'fo' => array(
'userFunc' => 'EXT:news/Classes/Hooks/Test.php:Tx_News_Hooks_Test->user_fo'
),
),
</pre>
<p>you see this is much easier.</p>
<p>customControlConfig is IMO not needed inside the hook because $config is added which includes the customcontrols too.</p> TYPO3 Core - Feature #34283: IRRE - Possibility to render custom header controlshttp://forge.typo3.org/issues/34283?journal_id=1039942012-02-26T15:21:54ZIngmar Schlechtingmar@typo3.org
<ul></ul><p>Good point, makes sense.</p> TYPO3 Core - Feature #34283: IRRE - Possibility to render custom header controlshttp://forge.typo3.org/issues/34283?journal_id=1040912012-02-27T02:31:37ZTYPO3 Release Teamtypo3v4@typo3.org
<ul><li><strong>Status</strong> changed from <i>Under Review</i> to <i>Resolved</i></li><li><strong>% Done</strong> changed from <i>0</i> to <i>100</i></li></ul><p>Applied in changeset <a class="changeset" title="[FEATURE] IRRE - Possibility to render custom header controls Render custom header controls by u..." href="http://forge.typo3.org/projects/typo3cms-core/repository/1749/revisions/2fe581e6011c4fe80f5d567676632fbb0ab9f1bd">2fe581e6011c4fe80f5d567676632fbb0ab9f1bd</a>.</p> TYPO3 Core - Feature #34283: IRRE - Possibility to render custom header controlshttp://forge.typo3.org/issues/34283?journal_id=3486592017-10-23T22:20:25ZRiccardo De Contardierredeco@gmail.com
<ul><li><strong>Status</strong> changed from <i>Resolved</i> to <i>Closed</i></li></ul>