Bug #85544
closedThe form definition upgrade wizard does not update all relevant plugin settings
100%
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.
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by RVVN no-lastname-given about 6 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 ?
Updated by RVVN no-lastname-given about 6 years ago
OK, no pressure, I just wanted to report at the right place ;)
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Oliver Hader about 6 years ago
- Target version set to 7.6.31 & 8.7.18 & 9.3.3
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Ralf Zimmermann about 6 years ago
- Related to Bug #85685: Uncaught TYPO3 Exception in form update wizard added
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Gerrit Code Review about 6 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
Updated by Ralf Zimmermann about 6 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 650ce3d5955dac5cd4208a55d04adbbedfc9af4b.
Updated by Benni Mack almost 6 years ago
- Status changed from Resolved to Closed
Updated by Ralf Hettinger almost 6 years ago
- Related to Task #82328: EXT:form - ship own uploads (sub)folder for form definitions added