Bug #77344

EXT:form - global layout for confirmation view colides with confirmation = 1

Added by Bjoern Jacob over 1 year ago. Updated over 1 year ago.

Status:
Resolved
Priority:
Should have
Category:
Form Framework
Target version:
-
Start date:
2016-08-02
Due date:
% Done:

100%

TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:
On Location Sprint

Description

Checkout the following form. confirmation = 1 defines to show the confirmation view. The next few lines define a global layout change for the checkbox element on the confirmation view. This leads to a naming conflict resulting that the confirmation view is not shown. Right now the code checks for confirmation === 1.


enctype = multipart/form-data
method = post
prefix = tx_form

form {
  layout {
    checkbox = <label /><div><input /></div>
  }
}

confirmation = 1
confirmation {
  layout {
    checkbox (
    <label /><div><inputvalue /></div>
    )
  }
}

postProcessor {
  layout {
    checkbox = <td colspan="2"><div><inputvalue /></div></td>
  }
}

postProcessor {
    1 = mail
    1 {
        recipientEmail = test@example.com
        senderEmail = test@example.com
        subject = Test
    }
}
10 = CHECKBOX
10 {
    type = checkbox
    name = contact_checkbox
    value = Check value.
    label {
        value = Check label?
    }
}
20 = SUBMIT
20 {
    type = submit
    name = 3
    value = Send
}


Related issues

Blocks TYPO3 Core - Bug #76823: EXT:form - checkboxes are missing in html mails Closed 2016-06-28

Associated revisions

Revision d9189857 (diff)
Added by Ralf Zimmermann over 1 year ago

[!!!][BUGFIX] Rename configuration for confirmation view

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 <>
Tested-by: Bjoern Jacob <>
Reviewed-by: Markus Hölzle <>
Tested-by: Markus Hölzle <>
Reviewed-by: Anja Leichsenring <>
Tested-by: Anja Leichsenring <>

History

#1 Updated by Gerrit Code Review over 1 year ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49304

#2 Updated by Gerrit Code Review over 1 year ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49304

#3 Updated by Gerrit Code Review over 1 year ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49305

#4 Updated by Gerrit Code Review over 1 year ago

Patch set 2 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49305

#5 Updated by Gerrit Code Review over 1 year ago

Patch set 3 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49305

#6 Updated by Ralf Zimmermann over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF