Bug #92806

Uncatched exception when copying a folder to the same parent folder

Added by Xavier Perseguers 2 months ago. Updated about 5 hours ago.

Status:
Under Review
Priority:
Should have
Category:
Backend User Interface
Target version:
-
Start date:
2020-11-09
Due date:
% Done:

0%

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

Description

How to reproduce:

  1. Create a folder user_upload/test
  2. Copy this folder to the clipboard
  3. Go to user_upload/
  4. Paste the folder from the clipboard

What happens?

  • TYPO3 crashes with #1476107295 TYPO3\CMS\Core\Error\Exception since mkdir() failed.

Expected:

  • A (nice) error message explaining you can't copy a folder to the same parent folder (or something like that).
  • Alternatively: show the duplication dialog and allow to rename copied folder to a unique name (or cancel the operation)

Files

#1

Updated by Riccardo De Contardi about 2 months ago

  • Category set to Backend User Interface
#2

Updated by Richard Haeser 7 days ago

Can confirm in current master as well. I would opt to keep it the same as when you do the same action with a file as the current bugfix. We might want to check if this is something we can enhance in general.

See attached screenshot

#3

Updated by Xavier Perseguers 7 days ago

#4

Updated by Richard Haeser 7 days ago

  • Assignee set to Richard Haeser

I always struggle getting the image in the text...

But I'm on the issue itself!

#5

Updated by Xavier Perseguers 7 days ago

Richard Haeser wrote in #note-4:

I always struggle getting the image in the text...

That's a bad idea to upload files with blank spaces. But I know how easy it happens. The trick in that case is to replace the blank spaces with %20 when trying to embed the image :)

#6

Updated by Gerrit Code Review 7 days 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/c/Packages/TYPO3.CMS/+/67442

#7

Updated by Gerrit Code Review 7 days 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/c/Packages/TYPO3.CMS/+/67442

#8

Updated by Gerrit Code Review 7 days 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/c/Packages/TYPO3.CMS/+/67442

#9

Updated by Gerrit Code Review 3 days 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/c/Packages/TYPO3.CMS/+/67442

#10

Updated by Gerrit Code Review 1 day 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/c/Packages/TYPO3.CMS/+/67442

#11

Updated by Gerrit Code Review about 6 hours 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/c/Packages/TYPO3.CMS/+/67442

#12

Updated by Gerrit Code Review about 5 hours ago

Patch set 7 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/+/67442

Also available in: Atom PDF