[BUGFIX] Readonly fields must not render links to allowed tables 88/39388/3
authorGeorg Ringer <georg.ringer@gmail.com>
Sat, 9 May 2015 20:16:16 +0000 (22:16 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Mon, 11 May 2015 10:39:11 +0000 (12:39 +0200)
If a field is configured as readOnly, the allowed tables below
should not be linked to the Element Browser

Change-Id: Ibae0056cd681e0112c715488ea0ff3b13d11ab42
Resolves: #66870
Releases: master, 6.2
Reviewed-on: http://review.typo3.org/39388
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/backend/Classes/Form/Element/AbstractFormElement.php

index 2465872..8934c8a 100644 (file)
@@ -726,7 +726,11 @@ abstract class AbstractFormElement extends AbstractNode {
                        $allowedTables .= '<div class="help-block">';
                        foreach ($params['allowedTables'] as $key => $item) {
                                if (is_array($item)) {
-                                       $allowedTables .= '<a href="#" onClick="' . htmlspecialchars($item['onClick']) . '" class="btn btn-default">' . $item['icon'] . ' ' . htmlspecialchars($item['name']) . '</a> ';
+                                       if (empty($params['readOnly'])) {
+                                               $allowedTables .= '<a href="#" onClick="' . htmlspecialchars($item['onClick']) . '" class="btn btn-default">' . $item['icon'] . ' ' . htmlspecialchars($item['name']) . '</a> ';
+                                       } else {
+                                               $allowedTables .= '<span>' . htmlspecialchars($item['name']) . '</span> ';
+                                       }
                                } elseif($key === 'name') {
                                        $allowedTables .= '<span>' . htmlspecialchars($item) . '</span> ';
                                }