Project

General

Profile

Actions

Bug #102283

closed

Extbase file upload using type converters not possible any more

Added by Torben Hansen over 1 year ago. Updated 4 months ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Extbase
Target version:
-
Start date:
2023-10-30
Due date:
% Done:

100%

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

Description

With #97214, the $_FILES globale variable is not used within extbase any more. The change however misses, that the uploaded files previously were merged into extbase arguments, so currently a file uploaded through the Fluid upload ViewHelper (example: <f:form.upload property="file" />) is not available as a mapped property. This results in, that all 3rd party code, which makes use of a custom TypeConverter for file uploads (like the one we use in ext:form or Helmuts Upload Example) does not work any more, since files are simply not uploaded and mapped any more.


Related issues 4 (0 open4 closed)

Related to TYPO3 Core - Task #101501: Remove backwards-compat files array in extbaseClosed2023-07-31

Actions
Related to TYPO3 Core - Bug #102271: Broken backwards compatibility in request builder for file uploads as object propertyClosed2023-10-27

Actions
Related to TYPO3 Core - Epic #103022: Extbase file upload handlingClosed2024-03-31

Actions
Related to TYPO3 Core - Task #97214: Use uploadedFile objects instead of $_FILESClosed2022-03-20

Actions
Actions #1

Updated by Torben Hansen over 1 year ago

  • TYPO3 Version changed from 12 to 13
Actions #2

Updated by Daniel Goerz over 1 year ago

  • Related to Task #97214: Use uploadedFile objects instead of $_FILES added
Actions #3

Updated by Daniel Goerz over 1 year ago

To add my comment from the EXT:form Patch:

"I dont think the EXT:form patch is the "correct" fix. We should do the array_merge_recursive in the extbase RequestBuilder instead to have the uploaded files also available as arguments in the ActionController, so that we can map them on object properties via PropertyMapper."

Actions #4

Updated by Torben Hansen over 1 year ago

As the backwards compatibility has been removed in v13 (see https://review.typo3.org/c/Packages/TYPO3.CMS/+/80106), we most likely need "something new" to handle file uploads in extbase

Actions #5

Updated by Torben Hansen over 1 year ago

  • Related to deleted (Task #97214: Use uploadedFile objects instead of $_FILES)
Actions #6

Updated by Torben Hansen over 1 year ago

  • Related to Task #101501: Remove backwards-compat files array in extbase added
Actions #7

Updated by Garvin Hicking over 1 year ago

  • Related to Bug #102271: Broken backwards compatibility in request builder for file uploads as object property added
Actions #8

Updated by Daniel Goerz 12 months ago

  • Related to Epic #103022: Extbase file upload handling added
Actions #9

Updated by Daniel Goerz 12 months ago

  • Related to Task #97214: Use uploadedFile objects instead of $_FILES added
Actions #10

Updated by Garvin Hicking 6 months ago

  • Category set to Extbase
Actions #11

Updated by Gerrit Code Review 6 months ago

  • Status changed from New to Under Review

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

Actions #12

Updated by Gerrit Code Review 6 months ago

Patch set 2 for branch main of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/c/Packages/TYPO3.CMS/+/85449

Actions #13

Updated by Anonymous 6 months ago

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

Updated by Benni Mack 4 months ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF