Bug #82881
closed
Story #69617: FormEngine bugs
Wrong page uid given to getDefaultUploadFolder for InlineControlContainer
100%
Description
When a user adds a media element to a page object, the function getDefaultUploadFolder should get the page UID of the current page as an argument instead of the UID of the parent page.
For all other scenarios (e.g. tt_content, extbase records with file relations), this works fine. Only for file relations on pages, the wrong PID is passed.
The page uid is only used in the hook $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_userauthgroup.php']['getDefaultUploadFolder']. The bug may lead to unexpected behaviour, if you want to configure the default upload folder on page uid basis.
Updated by Frans Saris about 7 years ago
Tnx for digging into this. This is only incorrect for page records. Not for other records correct?
Could you push a patch to adjust this to our review system?
Frans
Updated by Torben Hansen about 7 years ago
- Assignee set to Torben Hansen
Yes, this seems only to be wrong for inline elements on a page. All other records (e.g. news, tt_content, sys_file_collection) works.
I will create a patch for the change.
Updated by Gerrit Code Review about 7 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/54513
Updated by Gerrit Code Review over 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/54513
Updated by Christian Kuhn about 6 years ago
- Status changed from Under Review to New
Updated by Susanne Moog almost 6 years ago
- Category changed from File Abstraction Layer (FAL) to FormEngine aka TCEforms
Updated by Gerrit Code Review almost 5 years ago
- Status changed from New to Under Review
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/c/Packages/TYPO3.CMS/+/54513
Updated by Gerrit Code Review almost 5 years ago
Patch set 1 for branch 9.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/+/63868
Updated by Susanne Moog almost 5 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 734e6bca897382c66ba79f67b82687ddd90f2169.
Updated by Benni Mack almost 5 years ago
- Status changed from Resolved to Closed