[TASK] Purge onFocus occurrences from FormEngine 99/47699/3
authorOliver Hader <oliver@typo3.org>
Fri, 15 Apr 2016 10:18:54 +0000 (12:18 +0200)
committerBenni Mack <benni@typo3.org>
Sun, 17 Apr 2016 20:03:02 +0000 (22:03 +0200)
In TYPO3 6.2 palette fields have been handled by an individual on-focus
DOM events, depending on a particular user setting. During the FormEngine
rewrite with TYPO3 CMS 7, these parts have been kept and transported
through the system. Since this information is not used anymore, it can
be removed without any substitution.

Resolves: #75657
Releases: master
Change-Id: I7ab55d3f38b8f34e85027cdbbe8d5850f91dddb5
Reviewed-on: https://review.typo3.org/47699
Reviewed-by: Susanne Moog <typo3@susannemoog.de>
Tested-by: Susanne Moog <typo3@susannemoog.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
12 files changed:
typo3/sysext/backend/Classes/Form/Container/FlexFormElementContainer.php
typo3/sysext/backend/Classes/Form/Element/AbstractFormElement.php
typo3/sysext/backend/Classes/Form/Element/CheckboxElement.php
typo3/sysext/backend/Classes/Form/Element/GroupElement.php
typo3/sysext/backend/Classes/Form/Element/InputTextElement.php
typo3/sysext/backend/Classes/Form/Element/RadioElement.php
typo3/sysext/backend/Classes/Form/Element/SelectCheckBoxElement.php
typo3/sysext/backend/Classes/Form/Element/SelectMultipleSideBySideElement.php
typo3/sysext/backend/Classes/Form/Element/SelectSingleBoxElement.php
typo3/sysext/backend/Classes/Form/Element/SelectSingleElement.php
typo3/sysext/backend/Classes/Form/Element/TextElement.php
typo3/sysext/rsaauth/Classes/Form/Element/RsaInputElement.php

index 41d1359..c6df908 100644 (file)
@@ -86,7 +86,6 @@ class FlexFormElementContainer extends AbstractContainer
                         'onChange' => $flexFormFieldArray['onChange'],
                     ],
                     'fieldChangeFunc' => $parameterArray['fieldChangeFunc'],
-                    'onFocus' => $parameterArray['onFocus'],
                     'label' => $parameterArray['label'],
                 ];
 
index 940c3f4..a8e21a6 100644 (file)
@@ -435,7 +435,7 @@ abstract class AbstractFormElement extends AbstractNode
      * @param array $itemArray The array of items. For "select" and "group"/"file" this is just a set of value. For "db" its an array of arrays with table/uid pairs.
      * @param string $selector Alternative selector box.
      * @param array $params An array of additional parameters, eg: "size", "info", "headers" (array with "selector" and "items"), "noBrowser", "thumbnails
-     * @param string $onFocus On focus attribute string
+     * @param null $_ unused (onFocus in the past), will be removed in TYPO3 CMS 9
      * @param string $table (optional) Table name processing for
      * @param string $field (optional) Field of table name processing for
      * @param string $uid (optional) uid of table record processing for
@@ -444,7 +444,7 @@ abstract class AbstractFormElement extends AbstractNode
      * @throws \UnexpectedValueException
      * @todo: Hack this mess into pieces and inline to group / select element depending on what they need
      */
-    protected function dbFileIcons($fName, $mode, $allowed, $itemArray, $selector = '', $params = array(), $onFocus = '', $table = '', $field = '', $uid = '', $config = array())
+    protected function dbFileIcons($fName, $mode, $allowed, $itemArray, $selector = '', $params = array(), $_ = null, $table = '', $field = '', $uid = '', $config = array())
     {
         $languageService = $this->getLanguageService();
         $disabled = '';
@@ -508,7 +508,7 @@ abstract class AbstractFormElement extends AbstractNode
             $selector = '<select id="' . StringUtility::getUniqueId('tceforms-multiselect-') . '" '
                 . ($params['noList'] ? 'style="display: none"' : 'size="' . $sSize . '" class="form-control tceforms-multiselect"')
                 . ($isMultiple ? ' multiple="multiple"' : '')
-                . ' data-formengine-input-name="' . htmlspecialchars($fName) . '" ' . $this->getValidationDataAsDataAttribute($config) . $onFocus . $params['style'] . $disabled . '>' . implode('', $opt)
+                . ' data-formengine-input-name="' . htmlspecialchars($fName) . '" ' . $this->getValidationDataAsDataAttribute($config) . $params['style'] . $disabled . '>' . implode('', $opt)
                 . '</select>';
         }
         $icons = array(
@@ -714,7 +714,6 @@ abstract class AbstractFormElement extends AbstractNode
                     'mode' => $mode,
                     'allowed' => $allowed,
                     'itemArray' => $itemArray,
-                    'onFocus' => $onFocus,
                     'table' => $table,
                     'field' => $field,
                     'uid' => $uid,
index 21a0967..618c2c9 100644 (file)
@@ -136,7 +136,6 @@ class CheckboxElement extends AbstractFormElement
                                                value="1"
                                                data-formengine-input-name="' . htmlspecialchars($additionalInformation['itemFormElName']) . '"
                                                ' . $checkboxParameters . '
-                                               ' . $additionalInformation['onFocus'] . '
                                                ' . (!$disabled ?: ' disabled="disabled"') . '
                                                id="' . $checkboxId . '" />
                                        ' . ($label ? htmlspecialchars($label) : '&nbsp;') . '
index 206965e..41bf479 100644 (file)
@@ -202,11 +202,12 @@ class GroupElement extends AbstractFormElement
                     $itemArray,
                     '',
                     $params,
-                    $parameterArray['onFocus'],
+                    null,
                     '',
                     '',
                     '',
-                    $config);
+                    $config
+                );
                 if (!$disabled && !(isset($config['disable_controls']) && GeneralUtility::inList($config['disable_controls'], 'upload'))) {
                     // Adding the upload field:
                     $isDirectFileUploadEnabled = (bool)$this->getBackendUserAuthentication()->uc['edit_docModuleUpload'];
@@ -240,8 +241,7 @@ class GroupElement extends AbstractFormElement
                     '',
                     $itemArray,
                     '',
-                    $params,
-                    $parameterArray['onFocus']
+                    $params
                 );
                 break;
             case 'db':
@@ -298,7 +298,7 @@ class GroupElement extends AbstractFormElement
                     $itemArray,
                     '',
                     $params,
-                    $parameterArray['onFocus'],
+                    null,
                     $table,
                     $fieldName,
                     $row['uid'],
index 3744941..55b21c9 100644 (file)
@@ -159,10 +159,7 @@ class InputTextElement extends AbstractFormElement
             $attributeString .= ' ' . $attributeName . '="' . htmlspecialchars($attributeValue) . '"';
         }
 
-        $html = '
-                       <input type="text"'
-                . $attributeString
-                . $parameterArray['onFocus'] . ' />';
+        $html = '<input type="text"' . $attributeString . ' />';
 
         // This is the ACTUAL form field - values from the EDITABLE field must be transferred to this field which is the one that is written to the database.
         $html .= '<input type="hidden" name="' . $parameterArray['itemFormElName'] . '" value="' . htmlspecialchars($parameterArray['itemFormElValue']) . '" />';
index af34cb9..96b6d08 100644 (file)
@@ -44,7 +44,6 @@ class RadioElement extends AbstractFormElement
                                 . ' id="' . $radioId . '"'
                                 . ' value="' . htmlspecialchars($value) . '"'
                                 . $radioChecked
-                                . $this->data['parameterArray']['onFocus']
                                 . $disabled
                                 . ' onclick="' . htmlspecialchars(implode('', $this->data['parameterArray']['fieldChangeFunc'])) . '"'
                             . '/>'
index 287c1a2..2f20727 100644 (file)
@@ -139,8 +139,7 @@ class SelectCheckBoxElement extends AbstractFormElement
                                             . 'value="' . htmlspecialchars($item['value']) . '" '
                                             . 'onclick="' . htmlspecialchars($sOnChange) . '" '
                                             . ($item['checked'] ? 'checked=checked ' : '')
-                                            . ($item['disabled'] ? 'disabled=disabled ' : '')
-                                            . $parameterArray['onFocus'] . '>';
+                                            . ($item['disabled'] ? 'disabled=disabled ' : '') . '>';
                         $tableRows[] =    '</td>';
                         $tableRows[] =    '<td class="col-icon">';
                         $tableRows[] =        '<label class="label-block" for="' . $item['id'] . '">' . $item['icon'] . '</label>';
index cd3c538..8beb33b 100644 (file)
@@ -113,7 +113,6 @@ class SelectMultipleSideBySideElement extends AbstractFormElement
                 . 'class="form-control t3js-formengine-select-itemstoselect" '
                 . ($size ? ' size="' . $size . '" ' : '')
                 . 'onchange="' . htmlspecialchars($sOnChange) . '" '
-                . $parameterArray['onFocus']
                 . $this->getValidationDataAsDataAttribute($config)
                 . $selector_itemListStyle
                 . '>';
@@ -171,7 +170,7 @@ class SelectMultipleSideBySideElement extends AbstractFormElement
             'rightbox' => implode(LF, $itemsToSelect),
             'readOnly' => $disabled
         );
-        $html .= $this->dbFileIcons($parameterArray['itemFormElName'], '', '', $itemsArray, '', $params, $parameterArray['onFocus']);
+        $html .= $this->dbFileIcons($parameterArray['itemFormElName'], '', '', $itemsArray, '', $params);
 
         // Wizards:
         if (!$disabled) {
index 664f14f..12ca652 100644 (file)
@@ -155,7 +155,7 @@ class SelectSingleBoxElement extends AbstractFormElement
         }
 
         $html = [
-            '<select ' . $this->implodeAttributes($attributes) . ' ' . $parameterArray['onFocus'] . ' ' . $this->getValidationDataAsDataAttribute($config) . '>',
+            '<select ' . $this->implodeAttributes($attributes) . ' ' . $this->getValidationDataAsDataAttribute($config) . '>',
                 implode(LF, $optionElements),
             '</select>',
         ];
index 60a9e7e..5ae43dc 100644 (file)
@@ -255,10 +255,7 @@ class SelectSingleElement extends AbstractFormElement
                     '{',
                         'onChange: function() {',
                             implode('', $parameterArray['fieldChangeFunc']),
-                        '},',
-                        'onFocus: function() {',
-                            $parameterArray['onFocus'],
-                        '},',
+                        '}',
                     '}',
                 ');',
             '}',
index 339e500..0944e9b 100644 (file)
@@ -171,7 +171,6 @@ class TextElement extends AbstractFormElement
             $html .= '<textarea'
                 . $attributeString
                 . $placeholderAttribute
-                . $parameterArray['onFocus']
                 . '>' . htmlspecialchars($parameterArray['itemFormElValue']) . '</textarea>';
 
             // Wrap a wizard around the item?
index c697dca..2d1ca13 100644 (file)
@@ -134,10 +134,7 @@ class RsaInputElement extends AbstractFormElement
             $attributeString .= ' ' . $attributeName . '="' . htmlspecialchars($attributeValue) . '"';
         }
 
-        $html = '
-                       <input'
-            . $attributeString
-            . $parameterArray['onFocus'] . ' />';
+        $html = '<input' . $attributeString . ' />';
 
         // This is the ACTUAL form field - values from the EDITABLE field must be transferred to this field which is the one that is written to the database.
         $html .= '<input type="hidden" data-rsa-encryption="" id="' . $parameterArray['itemFormElID'] . '_hidden" name="' . $parameterArray['itemFormElName'] . '" value="' . htmlspecialchars($parameterArray['itemFormElValue']) . '" />';