Bug #55371

Ajax handler TYPO3_tcefile::process is broken

Added by Frans Saris over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-01-27
Priority:Must have Due date:
Assigned To:Frans Saris % Done:

100%

Category:File Abstraction Layer (FAL) Spent time: -
Target version:6.2.0
TYPO3 Version:6.2 Is Regression:No
PHP Version: Sprint Focus:
Complexity:medium

Description

Multiple changes in \TYPO3\CMS\Core\Utility\File\ExtendedFileUtility broke the TYPO3_tcefile ajax handler.

printLogErrorMessages() is deprecated in favour of getErrorMessages() but still used.

The return value of getErrorMessages() removed and this breaks the ajax processing.


Related issues

related to Core - Feature #44288: Add delete-button in filelist module Resolved 2013-01-03
related to Core - Bug #56084: t3editor is not usable any more Resolved 2014-02-18

Associated revisions

Revision 5cf92725
Added by Frans Saris over 1 year ago

[BUGFIX] Ajax handler TYPO3_tcefile::process is broken

Several changes in core broke the ajax upload handler response.
There is no error response and it tries to json_encode complete
File objects.

This changes reverts some of the "cleanup" changes fixes the
response and makes sure only the error message from current
request are used.

Resolves: #55371
Releases: 6.2, 6.1
Change-Id: Ia09f5ffd5e002b1ba50d4fafa747340462d15148
Reviewed-on: https://review.typo3.org/27080
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter

History

#1 Updated by Gerrit Code Review over 1 year 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/27080

#2 Updated by Gerrit Code Review over 1 year 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/27080

#3 Updated by Gerrit Code Review over 1 year 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/27080

#4 Updated by Gerrit Code Review over 1 year 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/27080

#5 Updated by Gerrit Code Review over 1 year 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/27080

#6 Updated by Frans Saris over 1 year ago

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

Also available in: Atom PDF