[TASK] FormEngine: Remove references to deprecated renderMode property 61/44361/2
authorMorton Jonuschat <m.jonuschat@mojocode.de>
Thu, 29 Oct 2015 16:27:45 +0000 (17:27 +0100)
committerChristian Kuhn <lolli@schwarzbu.ch>
Thu, 29 Oct 2015 17:19:56 +0000 (18:19 +0100)
Remove references to the deprecated renderMode property in comments.
Update AbstractNode to check renderType instead of renderMode when
converting the validation rules into a JSON object.

Resolves: #70589
Releases: master
Change-Id: Ia34bab0d74712051f0a728fc383ca57a753d267b
Reviewed-on: https://review.typo3.org/44361
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/backend/Classes/Form/AbstractNode.php
typo3/sysext/backend/Classes/Form/Element/AbstractFormElement.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/SelectTreeElement.php

index f2db428..ae9d255 100644 (file)
@@ -157,7 +157,7 @@ abstract class AbstractNode implements NodeInterface
             $minItems = (isset($config['minitems'])) ? (int)$config['minitems'] : 0;
             $maxItems = (isset($config['maxitems'])) ? (int)$config['maxitems'] : 10000;
             $type = ($config['type']) ?: 'range';
-            if ($config['renderMode'] !== 'tree' && $maxItems <= 1 && $minItems > 0) {
+            if ($config['renderType'] !== 'selectTree' && $maxItems <= 1 && $minItems > 0) {
                 $validationRules[] = array(
                     'type' => $type,
                     'minItems' => 1,
index a0fdeca..94484ff 100644 (file)
@@ -142,8 +142,8 @@ abstract class AbstractFormElement extends AbstractNode
             $flexFormPath = str_replace('][', '/', substr($PA['itemFormElName'], strlen($prefixOfFormElName) + 1, -1));
         }
 
-        // Add a suffix-value if the item is a selector box with renderMode "singlebox":
-        if ($PA['fieldConf']['config']['type'] === 'select' && (int)$PA['fieldConf']['config']['maxitems'] > 1 && $PA['fieldConf']['config']['renderMode'] === 'singlebox') {
+        // Add a suffix-value if the item is a selector box with renderType "selectSingleBox":
+        if ($PA['fieldConf']['config']['type'] === 'select' && (int)$PA['fieldConf']['config']['maxitems'] > 1 && $PA['fieldConf']['config']['renderType'] === 'selectSingleBox') {
             $itemName .= '[]';
         }
 
index 977332b..8d1f221 100644 (file)
@@ -23,7 +23,7 @@ use TYPO3\CMS\Backend\Form\Utility\FormEngineUtility;
 /**
  * Creates a widget with check box elements.
  *
- * This is rendered for config type=select, renderMode=checkbox, maxitems > 1
+ * This is rendered for config type=select, renderType=selectCheckBox, maxitems > 1
  */
 class SelectCheckBoxElement extends AbstractFormElement
 {
index c7b628a..9978e7e 100644 (file)
@@ -23,7 +23,7 @@ use TYPO3\CMS\Core\Utility\StringUtility;
 /**
  * Render a widget with two boxes side by side.
  *
- * This is rendered for config type=select, maxitems > 1, no other renderMode set
+ * This is rendered for config type=select, maxitems > 1, renderType=selectMultipleSideBySide set
  */
 class SelectMultipleSideBySideElement extends AbstractFormElement
 {
index e3c371f..7167404 100644 (file)
@@ -23,7 +23,7 @@ use TYPO3\CMS\Core\Utility\StringUtility;
 /**
  * Create a widget with a select box where multiple items can be selected
  *
- * This is rendered for config type=select, maxitems > 1, renderMode=singlebox
+ * This is rendered for config type=select, maxitems > 1, renderType=selectSingleBox
  */
 class SelectSingleBoxElement extends AbstractFormElement
 {
index 9594043..ad38472 100644 (file)
@@ -22,7 +22,7 @@ use TYPO3\CMS\Core\Utility\GeneralUtility;
 /**
  * Render data as a tree.
  *
- * Typically rendered for config [type=select, renderMode=tree
+ * Typically rendered for config [type=select, renderType=selectTree
  */
 class SelectTreeElement extends AbstractFormElement
 {