[!!!][BUGFIX] Rename configuration for confirmation view 05/49305/4
authorRalf Zimmermann <ralf.zimmermann@tritum.de>
Tue, 2 Aug 2016 12:33:09 +0000 (14:33 +0200)
committerAnja Leichsenring <aleichsenring@ab-softlab.de>
Tue, 2 Aug 2016 12:40:04 +0000 (14:40 +0200)
The configuration key for the confirmation layout has been renamed
from confirmation.layout to confirmationView.layout.

Resolves: #77344
Releases: 7.6
Change-Id: I6dad180d58143abd1da4a9930ca36cb0dbfbc9c7
Reviewed-on: https://review.typo3.org/49305
Reviewed-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Tested-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.de>
Tested-by: Markus Hölzle <typo3@markus-hoelzle.de>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
typo3/sysext/core/Documentation/Changelog/7.6/Breaking-77344-EXTform-RenameConfigurationForConfirmationView.rst [new file with mode: 0644]
typo3/sysext/form/Classes/Domain/Builder/FormBuilder.php
typo3/sysext/form/Documentation/Configuration/Layout/LayoutViewSpecific/Index.rst

diff --git a/typo3/sysext/core/Documentation/Changelog/7.6/Breaking-77344-EXTform-RenameConfigurationForConfirmationView.rst b/typo3/sysext/core/Documentation/Changelog/7.6/Breaking-77344-EXTform-RenameConfigurationForConfirmationView.rst
new file mode 100644 (file)
index 0000000..ecaaa69
--- /dev/null
@@ -0,0 +1,33 @@
+========================================================================
+Breaking: #77344 - EXT:form - Rename configuration for confirmation view
+========================================================================
+
+Description
+===========
+
+The typoscript key :ts:`configuration` for the confirmation configuration has been renamed.
+
+Up until now the layout settings of the confirmation view could be addressed via :ts:`tt_content.mailform.20.confirmation.layout`. This setting was introduced with patch 28526 but never documented.
+
+Besides this, the confirmation view enable setting can be set via :ts:`tt_content.mailform.20.confirmation = 1`.
+
+To keep the meaning of the settings clear, it was decided to rename the configuration of the confirmation view.
+
+
+Impact
+======
+
+Having the confirmation view enabled and the confirmation configuration customized, a naming collision occurs. As a result, the confirmation step has been disabled.
+Since the configuration was never documented, only few people know about this setting.
+
+
+Affected Installations
+======================
+
+All installations enabling the confirmation view and customizing the layout of this view.
+
+
+Migration
+=========
+
+All occurences of :ts:`tt_content.mailform.20.confirmation.layout` have to be migrated to :ts:`tt_content.mailform.20.confirmationView.layout`.
index e4daaa7..1e4a122 100644 (file)
@@ -274,7 +274,7 @@ class FormBuilder
                     $actionLayoutKey = 'form.';
                     break;
                 case 'confirmation':
-                    $actionLayoutKey = 'confirmation.';
+                    $actionLayoutKey = 'confirmationView.';
                     break;
                 case 'process':
                     $actionLayoutKey = 'postProcessor.';
index 46ab09e..f569867 100644 (file)
@@ -79,7 +79,7 @@ the following code could be applied.
     }
 
     # customize confirmation view
-    confirmation {
+    confirmationView {
       layout {
         textline (
           <div class="form-group">
@@ -1376,7 +1376,7 @@ including their default layout.
     # confirmation view
     # ###
 
-    confirmation {
+    confirmationView {
       layout {
         confirmation (
           <containerWrap />