Bug #82471

Ext:form – Read-only form elements appear in confirmation screen and email

Added by Mario Lubenka 10 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
Could have
Assignee:
-
Category:
Form Framework
Target version:
Start date:
2017-09-13
Due date:
% Done:

100%

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

Description

I saw that read-only form elements such as Static Text and Content element are still shown in the form confirmation and in the emails. That's a bug because the user can't enter information there and the "-" value suggests that the user entered this value.

E-Mail:

How to reproduce
  1. Add the elements "Static Text" and "Content element" to your form
  2. Submit the form in frontend
  3. Check the email: you should see the following:
    Static text: -
    Content element: -
    

form_bug_text_contentelement_mail.png View (15.8 KB) Mario Lubenka, 2017-09-13 12:21

form_bug_text_contentelement_preview.png View (30.5 KB) Mario Lubenka, 2017-09-13 12:21

Associated revisions

Revision d3470548 (diff)
Added by Ralf Zimmermann 8 months ago

[BUGFIX] EXT:form - hide readonly elements in summary page and emails

Hide readonly form elements within the summary page and emails.

Resolves: #82471
Releases: master, 8.7
Change-Id: I9c42754c935c0b2c62ff1a70f43e7a13d3594f82
Reviewed-on: https://review.typo3.org/54131
Reviewed-by: Daniel Lorenz <>
Tested-by: Daniel Lorenz <>
Tested-by: TYPO3com <>
Reviewed-by: Susanne Moog <>
Tested-by: Susanne Moog <>

Revision 1c5f3d9a (diff)
Added by Ralf Zimmermann 5 months ago

[BUGFIX] Hide readonly elements in summary page and emails

Hide readonly form elements within the summary page and emails.

Resolves: #82471
Releases: master, 8.7
Change-Id: I9c42754c935c0b2c62ff1a70f43e7a13d3594f82
Reviewed-on: https://review.typo3.org/55837
Reviewed-by: Björn Jacob <>
Tested-by: Björn Jacob <>
Tested-by: TYPO3com <>
Reviewed-by: Ralf Zimmermann <>
Tested-by: Ralf Zimmermann <>
Reviewed-by: Oliver Hader <>
Tested-by: Oliver Hader <>

History

#1 Updated by Gerrit Code Review 10 months 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/54131

#2 Updated by Gerrit Code Review 8 months 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/54131

#3 Updated by Ralf Zimmermann 8 months ago

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

#4 Updated by Česlav Przywara 7 months ago

Hi,

Is there any reason why the fix has not been cherry-picked for TYPO3 8.7.9 that has been released yesterday?

#5 Updated by Bjoern Jacob 7 months ago

No specific reason. Was forgotten during sprint. We will take care of it. Thx for mentioning.

#6 Updated by Gerrit Code Review 5 months 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/55837

#7 Updated by Gerrit Code Review 5 months 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/55837

#8 Updated by Gerrit Code Review 5 months 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/55837

#9 Updated by Gerrit Code Review 5 months 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/55837

#10 Updated by Bjoern Jacob 5 months ago

  • Target version set to next-patchlevel

#11 Updated by Gerrit Code Review 5 months 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/55837

#12 Updated by Gerrit Code Review 5 months 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/55837

#13 Updated by Ralf Zimmermann 5 months ago

  • Status changed from Under Review to Resolved

Also available in: Atom PDF