[TASK] EXT:form - Optimize deprecation messages 39/43539/2
authorRalf Zimmermann <ralf.zimmermann@tritum.de>
Thu, 24 Sep 2015 19:33:06 +0000 (21:33 +0200)
committerGeorg Ringer <georg.ringer@gmail.com>
Fri, 25 Sep 2015 08:40:06 +0000 (10:40 +0200)
Changes the deprecation log messages to be more understandable.

Resolves: #70011
Releases: master
Change-Id: I0ae03e7d3ed297e20fe2a226f4a4e5bea5ab2dc2
Reviewed-on: http://review.typo3.org/43539
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/form/Classes/Domain/Builder/FormBuilder.php
typo3/sysext/form/Classes/Utility/CompatibilityLayerUtility.php

index d162a15..1c6533e 100644 (file)
@@ -274,6 +274,10 @@ class FormBuilder {
        protected function reviveElement(Element $element, array $userConfiguredElementTypoScript, $elementType = '') {
                // @todo Check $userConfiguredElementTypoScript
 
+               if ($elementType === 'IMAGEBUTTON') {
+                       GeneralUtility::deprecationLog('EXT:form: The element IMAGEBUTTON is deprecated since TYPO3 CMS 7, will be removed with TYPO3 CMS 8.');
+               }
+
                $element->setElementType($elementType);
                $element->setElementCounter($this->elementCounter);
 
index b26337f..a368079 100644 (file)
@@ -107,10 +107,10 @@ class CompatibilityLayerUtility {
         */
        public function setGlobalLayoutConfiguration($typoscript = array()) {
                if (!empty($typoscript['layout.'])) {
-                       GeneralUtility::deprecationLog('Deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8, as the functionality is now done via fluid');
+                       GeneralUtility::deprecationLog('EXT:form: Do not use "layout." anymore. Deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8.');
                        $layout = $typoscript['layout.'];
                } else if (!empty($typoscript['layout'])) {
-                       GeneralUtility::deprecationLog('Deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8, as the functionality is now done via fluid');
+                       GeneralUtility::deprecationLog('EXT:form: Do not use "layout." anymore. Deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8.');
                        $layout = $typoscript['layout'];
                }
                if (is_array($layout)) {
@@ -131,7 +131,7 @@ class CompatibilityLayerUtility {
         * @deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8, as the functionality is now done via fluid
         */
        public function getGlobalLayoutByElementType($elementType) {
-               GeneralUtility::deprecationLog('Deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8, as the functionality is now done via fluid');
+               GeneralUtility::deprecationLog('EXT:form: Do not use "layout." anymore. Deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8.');
                $layout = '';
                if (!empty($this->layout[$elementType])) {
                        $layout = $this->layout[$elementType];
@@ -193,7 +193,7 @@ class CompatibilityLayerUtility {
         * @deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8, as the functionality is now done via fluid
         */
        public function setElementLayouts(Element $element, array $userConfiguredElementTyposcript = array()) {
-               GeneralUtility::deprecationLog('Deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8, as the functionality is now done via fluid');
+               GeneralUtility::deprecationLog('EXT:form: Do not use "layout." anymore. Deprecated since TYPO3 CMS 7, this function will be removed in TYPO3 CMS 8.');
                if ($element->getElementType() === 'FORM') {
                        $containerWrapReturn = $this->replaceTagWithMarker('elements', 'body', $this->getGlobalLayoutByElementType('CONTAINERWRAP'));
                        $formWrapReturn = $this->replaceTagWithMarker('containerwrap', 'form', $this->getGlobalLayoutByElementType('FORM'));
@@ -509,14 +509,14 @@ class CompatibilityLayerUtility {
        public function remapOldAttributes($elementType, $attributeName, array $additionalArguments, $userConfiguredElementTyposcript = array()) {
                if ($elementType === 'OPTION') {
                        if ($attributeName === 'data') {
-                               GeneralUtility::deprecationLog('Deprecated since TYPO3 CMS 7, use text instead of data to configure the OPTION text');
+                               GeneralUtility::deprecationLog('EXT:form: Deprecated since TYPO3 CMS 7, use text instead of data to configure the OPTION text');
                                $userConfiguredElementTyposcript['text'] = $userConfiguredElementTyposcript['data'];
                                unset($userConfiguredElementTyposcript[$attributeName]);
                                $attributeName = 'text';
                        }
                } elseif ($elementType === 'TEXTAREA') {
                        if ($attributeName === 'data') {
-                               GeneralUtility::deprecationLog('Deprecated since TYPO3 CMS 7, use value instead of data to configure the TEXTAREA value');
+                               GeneralUtility::deprecationLog('EXT:form: Deprecated since TYPO3 CMS 7, use text instead of data to configure the TEXTAREA value');
                                $userConfiguredElementTyposcript['value'] = $userConfiguredElementTyposcript['data'];
                                unset($userConfiguredElementTyposcript[$attributeName]);
                                $attributeName = 'value';