Bug #48893: Deleting files with references?! - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #48893

closed

Deleting files with references?!

Added by Claus Harup over 11 years ago. Updated about 7 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Frans Saris

Category:

File Abstraction Layer (FAL)

Target version:

6.2.0

Start date:

2013-06-05

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

6.1

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

I have setup a simple editor - granting access to fileadmin/user_upload/

The editor is able to delete files that have references?!?!?!

See attachement for details....

Am I missing some configuration?

Files

fal.png (52.6 KB) fal.png

Claus Harup, 2013-06-05 17:21

Related issues 1 (0 open — 1 closed)

Actions

Copy link

Updated by Frans Saris over 11 years ago

When he tries to delete the file he gets a error message that it isn't posible because file is in use.

Actions

Copy link

Updated by Claus Harup over 11 years ago

Hi Frans

I found out that that was the case, but would it not be more userfriendly if the deleteicon wasnt even there?

/Claus

Actions

Copy link

Updated by Ernesto Baschny over 11 years ago

Target version set to 6.2.0

Not more user-friendly, because it would look like a "bug" ("Where is the delete option?? I'll report that on the core bug tracker...") instead of explaining why he cannot delete.

But I agree that the user interface could be improved in this area, will try to get some input from the UX team on that. thanks for bringing it up.

Actions

Copy link

Updated by Philipp Gampe over 11 years ago

IMHO there should be the icon, but it should not be linked to the action, but rather to an JS pop (onclick) that explains the reason why it is not possible to delete the file.

Actions

Copy link

Updated by Sebastian Fischer over 10 years ago

Currently you get the following message if you try to delete a referenced file.

File not deleted
The file cannot be deleted since it is still used at the following places:
TCA Test [/Home/]

Do we need more informations for the editor?

Actions

Copy link

Updated by Andrea Herzog-Kienast almost 10 years ago

Assignee set to Frans Saris

You can delete a file with references!
If you have a folder structure like

folder
-- subfolder
--- 3subfolder

The 3subfolder contains files which are used in content elements and are referenced.
If you delete the parent folder "folder" all 3 folders are deleted - even if there are referenced files in the folder.

Actions

Copy link

Updated by Frans Saris almost 10 years ago

Is Regression set to No
Sprint Focus set to On Location Sprint

Actions

Copy link

Updated by Frans Saris almost 10 years ago

Status changed from New to In Progress

Actions

Copy link

Updated by Gerrit Code Review almost 10 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 http://review.typo3.org/36413

Actions

Copy link

#10

Updated by Gerrit Code Review almost 10 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#11

Updated by Gerrit Code Review almost 10 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#12

Updated by Gerrit Code Review almost 10 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#13

Updated by Gerrit Code Review almost 10 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#14

Updated by Gerrit Code Review almost 10 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#15

Updated by Gerrit Code Review almost 10 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#16

Updated by Gerrit Code Review almost 10 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#17

Updated by Gerrit Code Review almost 10 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#18

Updated by Gerrit Code Review almost 10 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#19

Updated by Gerrit Code Review almost 10 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#20

Updated by Gerrit Code Review almost 10 years ago

Patch set 12 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#21

Updated by Gerrit Code Review almost 10 years ago

Patch set 13 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36413

Actions

Copy link

#22