Skip to content
Snippets Groups Projects
Commit c274a4b4 authored by Ralf Zimmermann's avatar Ralf Zimmermann Committed by Susanne Moog
Browse files

[BUGFIX] Proper form definition validation if backend language changes

If a form element property is not defined through a form editor
inspector, the "predefinedDefaults" value from the form editor setup
will be used for some data integrity checks (such properties are
immutable).

Now, such checks against the "predefinedDefaults" values are *only* used
in scenarios where form elements are newely created. All the following
integrety checks on this value will be based on hmac validation.

In addition, this patchset fixes a faulty validation in the following
(edgy) scenario:

* backend language is EN
* open the form edtior and add a ContentElement form element
* switch to another browser tab and change the backend language to DE
* clear the cache
* go back to the form editor and click the save button

This is done by getting all translations (for all backend languages)
for the untranslated! "predefinedDefaults" value and compare the
(already translated) value (from the form definition) against the
possible translations from "predefinedDefaults".

There is an extended scenario which is out of scope for fixing:

* the same scenario as above + delete the previous chosen backend
  language within the maintenance tool

Resolves: #87520
Releases: master, 9.5
Change-Id: I6f486956c24121c0065b67b4f2179301e2a344c4
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/60638


Tested-by: default avatarTYPO3com <noreply@typo3.com>
Tested-by: default avatarTobi Kretschmann <tobi@tobishome.de>
Tested-by: default avatarSascha Rademacher <sascha.rademacher+typo3@gmail.com>
Tested-by: default avatarJulian Geils <j_geils@web.de>
Tested-by: default avatarSusanne Moog <look@susi.dev>
Reviewed-by: default avatarTobi Kretschmann <tobi@tobishome.de>
Reviewed-by: default avatarSascha Rademacher <sascha.rademacher+typo3@gmail.com>
Reviewed-by: default avatarJulian Geils <j_geils@web.de>
Reviewed-by: default avatarSusanne Moog <look@susi.dev>
parent b594b0aa
Branches
Tags
No related merge requests found
Showing
with 193 additions and 33 deletions
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment