Feature #34283

Task #33745: Integrate File Abstraction Layer

IRRE - Possibility to render custom header controls

Added by Oliver Hader over 7 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
Start date:
2012-02-25
Due date:
% Done:

100%

PHP Version:
5.3
Tags:
Complexity:
Sprint Focus:

Description

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

Associated revisions

Revision 2fe581e6 (diff)
Added by TYPO3 Release Team over 7 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 7 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 7 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 7 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 7 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 7 years ago

Good point, makes sense.

#6 Updated by TYPO3 Release Team over 7 years ago

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

#7 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF