Project

General

Profile

Actions
Copy link

Bug #92806

closed

Uncatched exception when copying a folder to the same parent folder

Added by Xavier Perseguers about 4 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Richard Haeser
Category:
Backend User Interface
Target version:
-
Start date:
2020-11-09
Due date:
% Done:

100%

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

Screenshot 2021-01-13 at 09.00.10.png (25.1 KB) Screenshot 2021-01-13 at 09.00.10.png Richard Haeser, 2021-01-13 08:01
Actions
Copy link
 #1

Updated by Riccardo De Contardi almost 4 years ago

  • Category set to Backend User Interface
Actions
Copy link
 #2

Updated by Richard Haeser almost 4 years 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

Actions
Copy link
 #4

Updated by Richard Haeser almost 4 years ago

  • Assignee set to Richard Haeser

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

But I'm on the issue itself!

Actions
Copy link
 #5

Updated by Xavier Perseguers almost 4 years 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 :)

Actions
Copy link
 #6

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

Actions
Copy link
 #7

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

Actions
Copy link
 #8

Updated by Gerrit Code Review almost 4 years 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

Actions
Copy link
 #9

Updated by Gerrit Code Review almost 4 years 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

Actions
Copy link
 #10

Updated by Gerrit Code Review almost 4 years 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

Actions
Copy link
 #11

Updated by Gerrit Code Review almost 4 years 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

Actions
Copy link
 #12

Updated by Gerrit Code Review almost 4 years 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

Actions
Copy link
 #13

Updated by Gerrit Code Review almost 4 years ago

Patch set 8 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

Actions
Copy link
 #14

Updated by Gerrit Code Review over 3 years ago

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/+/68111

Actions
Copy link
 #15

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

Actions
Copy link
 #16

Updated by Gerrit Code Review over 3 years 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/+/68111

Actions
Copy link
 #17

Updated by Gerrit Code Review over 3 years ago

Patch set 9 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

Actions
Copy link
 #18

Updated by Gerrit Code Review over 3 years ago

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

Actions
Copy link
 #19

Updated by Richard Haeser over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions
Copy link
 #20

Updated by Benni Mack over 3 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF