Feature #34283

Task #33745: Integrate File Abstraction Layer

IRRE - Possibility to render custom header controls

Added by Oliver Hader over 3 years ago. Updated over 3 years ago.

Status:Resolved Start date:2012-02-25
Priority:Should have Due date:
Assigned To:- % Done:

100%
Category:- Spent time: -
Target version:4.7.0-beta1
PHP Version:5.3 Sprint Focus:
Complexity:

Description

Render custom header controls by using a render function
defined in TCA field configuration customControls/userFunc

Associated revisions

Revision 2fe581e6
Added by TYPO3 Release Team over 3 years ago

[FEATURE] IRRE - Possibility to render custom header controls

Render custom header controls by using a render function
defined in TCA field configuration customControls/userFunc

Change-Id: I8a628c9434c540a1cc79156f3bcb4c35b1a3f1c3
Resolves: #34283
Releases: 4.7
Reviewed-on: http://review.typo3.org/9207
Reviewed-by: Ingmar Schlecht
Tested-by: Ingmar Schlecht
Reviewed-by: Steffen Ritter
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

History

#1 Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9207

#2 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9207

#3 Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9207

#4 Updated by Georg Ringer over 3 years ago

I would propose this change.

simplify the api by using $customControlConfig directly as input for t3lib_div::callUserFunction(). a call would then look like

'customControls' => array(
    'fo' => 'EXT:news/Classes/Hooks/Test.php:Tx_News_Hooks_Test->user_fo',
),

instead of 
'customControls' => array(
    'fo' => array(
        'userFunc' => 'EXT:news/Classes/Hooks/Test.php:Tx_News_Hooks_Test->user_fo'
    ),
),

you see this is much easier.

customControlConfig is IMO not needed inside the hook because $config is added which includes the customcontrols too.

#5 Updated by Ingmar Schlecht over 3 years ago

Good point, makes sense.

#6 Updated by TYPO3 Release Team over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF