Bug #95290
closedBE - Form content - Finisher overriding issues
100%
Description
TYPO3 v10 (6 issues)¶
CONTEXT : Backend - "Form" content type (form_formframework): a form with finisher is selected and "Override finisher settings" is enabled
ISSUE 1 : Deleting an invalid IRRE record prevents saving (validation failure)
When a IRRE record (with initially invalid field) is added and just deleted, the saving is inhibited.
Steps to reproduce :
1) create a form with finisher containing IRRE fields, such as "Email to sender"
2) add a Form Content Element to any page
3) choose that form ("Form definition")
4) enable "Override finisher settings"
5) switch to "Email to sender" tab and add a "Recipient" (or Reply-To Recipient, CC Recipient, BCC Recipient)
6) the "Email Address" field is invalid
7) delete the newly added recipient, without making any changes
8) try to save: saving stop you ("The fields marked with an exclamation mark are not yet correctly filled in. Please complete them properly.")
ISSUE 2 : If an IRRE record is added and editing is ended, by "Close" button or changing BE module, the GUI exits without alert.
Steps to reproduce :
1-5) same as ISSUE 1
6) click on Close button or change module (by main menu)
7) it exits without alert
ISSUE 3 : If you delete an existing section, no alert are triggered when exiting the edit (by "Close" button or changing BE module)
Steps to reproduce :
1-5) same as ISSUE 1
6) add a new "Recipient"
7) save form
8) delete the last added recipient
9) click on Close button or change module (by main menu)
7) it exits without alert
ISSUE 4 : Toggle action is confusing; it should open / close all sections the same way. Currently open ones are closed and vice versa.
The behavior should be like that implemented in v11.
Steps to reproduce :
1-5) same as ISSUE 1
6) click on "Toggle all" button
ISSUE 5 : If you click close (horizontally, right side), outside, to the "Toggle all" button, all sections are toggled.
The js handler is binded to "div" click event, instead of inner "a" click event.
Steps to reproduce :
1-5) same as ISSUE 1
6) click outside the "Toggle all" button, on the right side, on the same horizontal line
7) all sections are toggled
ISSUE 6 : On section deleting, the alert message is not defined/translated ("Are you sure you want to delete this section?")
Steps to reproduce :
1-5) same as ISSUE 1
6) set the BE lang other than English
7) delete the newly added recipient, without making any changes
8) the alert message is in English language
_________________________ * _________________________
TYPO3 v11 (4 issues)¶
CONTEXT : Backend - "Form" content type (form_formframework): a form with finisher is selected and "Override finisher settings" is enabled
ISSUE 1 : as for ISSUE 1 in v10
ISSUE 2 : as for ISSUE 2 in v10
ISSUE 3 : as for ISSUE 3 in v10
ISSUE 4 : as for ISSUE 6 in v10
Updated by Davide Alghi about 3 years ago
Here my proposals to fix issues
TYPO3 v10: https://github.com/TYPO3/typo3/pull/321
TYPO3 master: https://github.com/TYPO3/typo3/pull/320
Updated by Davide Alghi about 3 years ago
- Related to Task #95291: Fixes FormContent-FinisherOverriding master added
Updated by Davide Alghi about 3 years ago
- Related to Task #95292: Fixes FormContentFinisherOverriding issues added
Updated by Benni Mack about 3 years ago
- Related to Task #95591: Fixes FormContentFinisherOverriding issues added
Updated by Christian Kuhn about 3 years ago
- Related to Task #95589: Fixes FormContent-FinisherOverriding master added
Updated by Björn Jacob almost 3 years ago
- Sprint Focus set to Remote Sprint
We have to test this and discuss possible solutions.
Updated by Gerrit Code Review almost 3 years ago
- Status changed from New to Under Review
Patch set 1 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/73699
Updated by Gerrit Code Review over 2 years ago
Patch set 1 for branch 11.5 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/73706
Updated by Anonymous over 2 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset e6139147030754769b82e71699af9b6de8d01b02.