Bug #81770

EXT:form - do not show hidden field on confirmation page

Added by Henrik Elsner over 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Must have
Category:
Form Framework
Target version:
Start date:
2017-07-03
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:
Remote Sprint

Description

Hidden fields are shown on the submit/ confirmation page.
imho that's not the correct behaviour

Bildschirmfoto 2017-07-03 um 09.35.20.png View - form (22.6 KB) Henrik Elsner, 2017-07-03 09:37

Bildschirmfoto 2017-07-03 um 09.37.15.png View - submit page (26.6 KB) Henrik Elsner, 2017-07-03 09:37


Related issues

Related to TYPO3 Core - Bug #82518: Broken composite form element check in RenderAllFormValuesViewHelper Closed 2017-09-20
Related to TYPO3 Core - Task #84450: Backport for 81770 Closed 2018-03-17

Associated revisions

Revision f4570f98 (diff)
Added by Ralf Zimmermann about 2 years ago

[BUGFIX] EXT:form - exclude hidden fields within summary pages and mails

Add a "renderingOptions" property called "_isHiddenFormElement" to
prevent form elements from being shown within summary pages and emails.

Resolves: #81770
Releases: master, 8.7
Change-Id: Iea3232879fc2be2cae00e5ed5cea565e68e192f3
Reviewed-on: https://review.typo3.org/53383
Reviewed-by: Daniel Lorenz <>
Tested-by: Daniel Lorenz <>
Tested-by: TYPO3com <>
Reviewed-by: Bjoern Jacob <>
Tested-by: Bjoern Jacob <>
Reviewed-by: Anja Leichsenring <>
Tested-by: Anja Leichsenring <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>

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

[BUGFIX] Exclude hidden fields within summary pages and mails

Add a "renderingOptions" property called "_isHiddenFormElement" to
prevent form elements from being shown within summary pages and emails.

Resolves: #81770
Releases: master, 8.7
Change-Id: Iea3232879fc2be2cae00e5ed5cea565e68e192f3
Reviewed-on: https://review.typo3.org/56359
Reviewed-by: Björn Jacob <>
Tested-by: Björn Jacob <>
Reviewed-by: Mathias Brodala <>
Tested-by: TYPO3com <>
Reviewed-by: Kay Strobach <>
Tested-by: Kay Strobach <>
Reviewed-by: Benjamin Kott <>
Tested-by: Benjamin Kott <>

History

#1 Updated by Ralf Zimmermann over 2 years ago

  • Status changed from New to Accepted
  • Assignee set to Ralf Zimmermann
  • Priority changed from Should have to Must have

#2 Updated by Bjoern Jacob over 2 years ago

  • Subject changed from EXT:form to EXT:form - do not show hidden field on confirmation page
  • Description updated (diff)

#3 Updated by Gerrit Code Review over 2 years ago

  • Status changed from Accepted 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/53383

#4 Updated by Gerrit Code Review over 2 years 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/53383

#5 Updated by Bjoern Jacob over 2 years ago

  • Category set to Form Framework

#6 Updated by Gerrit Code Review over 2 years ago

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

#7 Updated by Bjoern Jacob over 2 years ago

  • TYPO3 Version changed from 9 to 8

#8 Updated by Daniel Goerz over 2 years ago

  • Description updated (diff)

#9 Updated by Bjoern Jacob about 2 years ago

  • Sprint Focus set to Remote Sprint

#10 Updated by Gerrit Code Review about 2 years ago

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

#11 Updated by Gerrit Code Review about 2 years ago

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

#12 Updated by Gerrit Code Review about 2 years ago

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

#13 Updated by Gerrit Code Review about 2 years ago

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

#14 Updated by Ralf Zimmermann about 2 years ago

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

#15 Updated by Mathias Brodala about 2 years ago

  • Related to Bug #82518: Broken composite form element check in RenderAllFormValuesViewHelper added

#16 Updated by Bjoern Jacob over 1 year ago

  • Target version set to next-patchlevel

#17 Updated by Bjoern Jacob over 1 year ago

#18 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Resolved to Under Review

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

#19 Updated by Gerrit Code Review over 1 year ago

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

#20 Updated by Gerrit Code Review over 1 year ago

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

#21 Updated by Bjoern Jacob over 1 year ago

  • Target version changed from next-patchlevel to 8.7.13

#22 Updated by Gerrit Code Review over 1 year ago

Patch set 4 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56359

#23 Updated by Ralf Zimmermann over 1 year ago

  • Status changed from Under Review to Resolved

#24 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF