From 9c1bd4e09c9d57011b61355ea4a6d086b7caa8ce Mon Sep 17 00:00:00 2001
From: Christian Kuhn <lolli@schwarzbu.ch>
Date: Sat, 22 Oct 2016 17:46:09 +0200
Subject: [PATCH] [BUGFIX] Custom select renderTypes use TcaSelectItems
There are two FormEngine data provider for type=select: TcaSelectItems
for "normal" renderTypes and TcaSelectTreeItems for renderType="selectTree".
While this is hacky in the first place - the data provider should not
depend on renderType at all but only on type, this can not be changed
easily and a real solution is to make "trees" an own type instead, later.
However, if extensions add a new renderType for select at the moment, the
normal TcaSelectItems data provider does not kick in since it is restricted
to the known renderTypes. This is usually not what an extension wants,
typically just the rendering is different while the data providing from
TcaSelectItems is used.
The patch allows this by now blacklisting "selectTree" in TcaSelectItems
instead of whitelisting the known types.
Change-Id: Ic71177e22b191a5abee90a7cfb9871167fe4611e
Resolves: #78381
Releases: master, 7.6
Reviewed-on: https://review.typo3.org/50325
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Reviewed-by: Mamadou Mbow <mmbow@dfau.de>
Tested-by: Mamadou Mbow <mmbow@dfau.de>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
---
.../Classes/Form/FormDataProvider/TcaSelectItems.php | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/typo3/sysext/backend/Classes/Form/FormDataProvider/TcaSelectItems.php b/typo3/sysext/backend/Classes/Form/FormDataProvider/TcaSelectItems.php
index c37fff3cede1..79fc49841341 100644
--- a/typo3/sysext/backend/Classes/Form/FormDataProvider/TcaSelectItems.php
+++ b/typo3/sysext/backend/Classes/Form/FormDataProvider/TcaSelectItems.php
@@ -152,10 +152,6 @@ class TcaSelectItems extends AbstractItemProvider implements FormDataProviderInt
*/
protected function isTargetRenderType(array $fieldConfig)
{
- return in_array(
- $fieldConfig['config']['renderType'],
- ['selectSingle', 'selectSingleBox', 'selectCheckBox', 'selectMultipleSideBySide'],
- true
- );
+ return $fieldConfig['config']['renderType'] !== 'selectTree';
}
}
--
GitLab