[TASK] Remove unimplemented function in FormEngine 76/32976/2
authorBenjamin Mack <benni@typo3.org>
Thu, 25 Sep 2014 18:41:14 +0000 (20:41 +0200)
committerGeorg Ringer <georg.ringer@gmail.com>
Thu, 25 Sep 2014 19:33:37 +0000 (21:33 +0200)
There is a function wrapLabels which does not do
anything for ages.
As it is empty, I suggest to remove it completely
instead of deprecating it.

Releases: master
Resolves: #61891
Change-Id: Ic0ebb366f663f43803321d823bb6d41ec579b052
Reviewed-on: http://review.typo3.org/32976
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/backend/Classes/Form/FormEngine.php

index 24ccbb0..6140edf 100644 (file)
@@ -1846,7 +1846,7 @@ TBE_EDITOR.customEvalFunctions[\'' . $evalData . '\'] = function(value) {
                                $cBID = $PA['itemFormElID'] . '_' . $c;
                                $item .= ($c > 0 ? '<br />' : '') . '<input type="checkbox"' . $this->insertDefStyle('check')
                                        . ' value="1" name="' . $cBName . '"' . $cBP . $PA['onFocus'] . $disabled . ' id="' . $cBID . '" />'
-                                       . $this->wrapLabels('<label for="' . $cBID . '">' . htmlspecialchars($p[0]) . '</label>');
+                                       . '<label for="' . $cBID . '">' . htmlspecialchars($p[0]) . '</label>';
                        }
                }
                if (!$disabled) {
@@ -4337,9 +4337,9 @@ TBE_EDITOR.customEvalFunctions[\'' . $evalData . '\'] = function(value) {
                }
                if ($imagesOnly) {
                        $rightbox = '';
-                       $thumbnails = '<div class="imagethumbs">' . $this->wrapLabels($params['thumbnails']) . '</div>';
+                       $thumbnails = '<div class="imagethumbs">' . $params['thumbnails'] . '</div>';
                } else {
-                       $rightbox = $this->wrapLabels($params['thumbnails']);
+                       $rightbox = $params['thumbnails'];
                        $thumbnails = '';
                }
                // Hook: dbFileIcons_postProcess (requested by FAL-team for use with the "fal" extension)
@@ -4365,15 +4365,15 @@ TBE_EDITOR.customEvalFunctions[\'' . $evalData . '\'] = function(value) {
                $str = '<table border="0" cellpadding="0" cellspacing="0" width="1" class="t3-form-field-group-file">
                        ' . ($params['headers'] ? '
                                <tr>
-                                       <td>' . $this->wrapLabels($params['headers']['selector']) . '</td>
+                                       <td>' . $params['headers']['selector'] . '</td>
                                        <td></td>
                                        <td></td>
-                                       <td>' . ($params['thumbnails'] ? $this->wrapLabels($params['headers']['items']) : '') . '</td>
+                                       <td>' . ($params['thumbnails'] ? $params['headers']['items'] : '') . '</td>
                                </tr>' : '') . '
                        <tr>
                                <td>' . $selector . $thumbnails;
                if (!$params['noList'] && $params['info'] !== '') {
-                       $str .= '<span class="filetypes">' . $this->wrapLabels($params['info']) . '</span>';
+                       $str .= '<span class="filetypes">' . $params['info'] . '</span>';
                }
                $str .= '</td>
                                        <td class="icons">' . implode('<br />', $icons['L']) . '</td>
@@ -5588,18 +5588,6 @@ TBE_EDITOR.customEvalFunctions[\'' . $evalData . '\'] = function(value) {
        }
 
        /**
-        * Wrapping labels
-        * Currently not implemented - just returns input value.
-        *
-        * @param string $str Input string.
-        * @return string Output string.
-        * @todo Define visibility
-        */
-       public function wrapLabels($str) {
-               return $str;
-       }
-
-       /**
         * Wraps all the table rows into a single table.
         * Used externally from scripts like alt_doc.php and db_layout.php (which uses TCEforms...)
         *