Project

General

Profile

Actions

Bug #85544

closed

The form definition upgrade wizard does not update all relevant plugin settings

Added by Ralf Zimmermann over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Category:
Form Framework
Start date:
2018-07-12
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

The form definition upgrade wizard (rename .yaml => .form.yaml) does not update all relevant plugin settings.
The form definition renaming wizard updates the "settings.persistenceIdentifier" property within the flexform but ignores the "Override finisher settings" options.

The form plugin tries to prevent unintentionally finisher settings overrides.
This could be happened if you have an already overridden finisher option and select another form definition.
In some cases it could be, that a finisher override from form definition "A" is set as an finisher override for form definition "B" if you switch from form definition "A" to "B".
To prevent this, every dynamic flexform sheet (every finisher, which allows the form plugin to override settings, appears in another tab) has an unique name which is composed of properties of the form definition.
This name is an md5 hash from the following form definition properties:

  • persistenceIdentifier
  • prototypeName
  • formIdentifier
  • finisherIdentifier

Since the form definition renaming wizard renames the "persistenceIdentifier" propery, the calculation for the unique name for the flexform sheets for the finisher overrides results in another md5 hash. The result is that the corresponding sheets are no longer associated with the currently selected form and all overridable finisher options are empty.


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #85685: Uncaught TYPO3 Exception in form update wizardClosed2018-07-30

Actions
Related to TYPO3 Core - Task #82328: EXT:form - ship own uploads (sub)folder for form definitionsClosed2017-09-07

Actions
Actions #1

Updated by Gerrit Code Review over 5 years 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/57582

Actions #2

Updated by Gerrit Code Review over 5 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/57582

Actions #3

Updated by Gerrit Code Review over 5 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/57582

Actions #4

Updated by Ralf Zimmermann over 5 years ago

  • Is Regression set to Yes
Actions #5

Updated by RVVN no-lastname-given over 5 years ago

Hi,

as I said on the slack channel, patch 3 didn't work for me.

I used the following way :
• rename file with .form.yaml suffix
• run the first wizard "rename to .form.yaml"
• run the second wizard "fix flexform after rename"

Sheets indexes have been modified in DB by the 2nd wizard but the form plugin still can't "use" these sheets. All the values displayed in finishers tabs are the default ones, and some are missing (Redirect Page for example)

Maybe hashes still are wrong ?

Actions #6

Updated by Ralf Zimmermann over 5 years ago

Thanks, iam still on it...

Actions #7

Updated by RVVN no-lastname-given over 5 years ago

OK, no pressure, I just wanted to report at the right place ;)

Actions #8

Updated by Gerrit Code Review over 5 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/57582

Actions #9

Updated by Gerrit Code Review over 5 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/57582

Actions #10

Updated by Gerrit Code Review over 5 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/57582

Actions #11

Updated by Oliver Hader over 5 years ago

  • Target version set to 7.6.31 & 8.7.18 & 9.3.3
Actions #12

Updated by Gerrit Code Review over 5 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/57582

Actions #13

Updated by Gerrit Code Review over 5 years ago

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

Actions #14

Updated by Gerrit Code Review over 5 years ago

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

Actions #15

Updated by Ralf Zimmermann over 5 years ago

  • Related to Bug #85685: Uncaught TYPO3 Exception in form update wizard added
Actions #16

Updated by Gerrit Code Review over 5 years ago

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

Actions #17

Updated by Gerrit Code Review over 5 years ago

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

Actions #18

Updated by Gerrit Code Review over 5 years ago

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

Actions #19

Updated by Gerrit Code Review over 5 years ago

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

Actions #20

Updated by Gerrit Code Review over 5 years ago

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/57731

Actions #21

Updated by Gerrit Code Review over 5 years 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/57731

Actions #22

Updated by Gerrit Code Review over 5 years 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/57731

Actions #23

Updated by Gerrit Code Review over 5 years ago

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

Actions #24

Updated by Ralf Zimmermann over 5 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions #26

Updated by Ralf Hettinger over 5 years ago

  • Related to Task #82328: EXT:form - ship own uploads (sub)folder for form definitions added
Actions

Also available in: Atom PDF