Bug #83628

Epic #83669: Improve file list / file browser

EXT:filelist Can not rename a file with non LocalDriver storage

Added by Michael Oehlhof almost 2 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2018-01-21
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

When trying to rename a file with a non LocalDriver storage nothing happened when clicking the "Save and Close" button.

In the browser console you can see that the AJAX call with the route /ajax/file/exists ha a cropped value for the parameter fileTarget.

Problem is the wrong usage of the folder identifier which works currently only with the LocalDriver.

For reproducing you can use the TeamRuhr FAL Driver (Version >= 2.0.4)
https://extensions.typo3.org/extension/teamruhr_fal_driver/

Associated revisions

Revision 24779fb9 (diff)
Added by Michael Oehlhof over 1 year ago

[BUGFIX] Renaming files on non LocalDriver storages

Fixing determination of the parent folder of a file inside the
storage, so it can be used as destination for the renaming.

Resolves: #83628
Releases: master, 8.7
Change-Id: I2b905dc7ba5e3d6fa8bec2b5a728d27f3c4a3258
Reviewed-on: https://review.typo3.org/55421
Tested-by: TYPO3com <>
Reviewed-by: Frans Saris <>
Tested-by: Frans Saris <>
Reviewed-by: Tymoteusz Motylewski <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Frank Naegler <>
Tested-by: Frank Naegler <>

Revision c9a1f313 (diff)
Added by Michael Oehlhof over 1 year ago

[BUGFIX] Renaming files on non LocalDriver storages

Fixing determination of the parent folder of a file inside the
storage, so it can be used as destination for the renaming.

Resolves: #83628
Releases: master, 8.7
Change-Id: I2b905dc7ba5e3d6fa8bec2b5a728d27f3c4a3258
Reviewed-on: https://review.typo3.org/56475
Tested-by: TYPO3com <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>

History

#1 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from In Progress 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/55421

#2 Updated by Michael Oehlhof almost 2 years ago

  • Parent task set to #82206

#3 Updated by Riccardo De Contardi almost 2 years ago

  • Parent task changed from #82206 to #83669

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

#5 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/55421

#6 Updated by Gerrit Code Review over 1 year ago

Patch set 1 for branch TYPO3_8-7 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/56475

#7 Updated by Michael Oehlhof over 1 year ago

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

#8 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF