Skip to content

Commit

Permalink
[BUGFIX] Apply proper url encoding to query params in form engine popups
Browse files Browse the repository at this point in the history
With #101231 a missing URL encoding in FormEngine was triggered
which caused contents like "svg;disallowed=|irre-object-id" to be
added to bparams which got transformed into "svg&disallowed" in
some setups (ddev, most likely caused by the ddev-router-proxy).

This "change" causes the irre object id to be cut off from the
bparams in IREE file list element browser, which is why "select"
actions on file list elements were silently ignored.

Resolves: #101433
Related: #101231
Releases: main, 12.4, 11.5
Change-Id: I89dd53131f9ecb3b199bbcd2d1abf6be87f819da
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80172
Tested-by: Benjamin Franzke <ben@bnf.dev>
Reviewed-by: Benjamin Franzke <ben@bnf.dev>
Tested-by: core-ci <typo3@b13.com>
  • Loading branch information
bnf committed Jul 25, 2023
1 parent 3999784 commit b651f12
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -114,7 +114,7 @@ define(['jquery',
FormEngine.openPopupWindow = function(mode, params) {
return Modal.advanced({
type: Modal.types.iframe,
content: FormEngine.browserUrl + '&mode=' + mode + '&bparams=' + params,
content: FormEngine.browserUrl + '&' + (new URLSearchParams({mode: mode, bparams: params})).toString(),
size: Modal.sizes.large
});
};
Expand Down

0 comments on commit b651f12

Please sign in to comment.