Bug #85323: Create dir if user create a local storage - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #85323

closed

Create dir if user create a local storage

Added by Guido Schmechel over 6 years ago. Updated 5 months ago.

Status:

Closed

Priority:

Could have

Assignee:

Stefan Bürk

Category:

File Abstraction Layer (FAL)

Target version:

Start date:

2018-06-21

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

PHP Version:

Tags:

storage

Complexity:

Is Regression:

Sprint Focus:

Description

TYPO3 Version 9.4.dev

I create a file storage to test another issue. I use the Local Driver add a fictitious directory path and add the storage. If i check the "file list" module, i see the new storage and the hint "Name (Offline)" There is no hint that this directory doesn't exist.

Beware of the other fal driver/adapter it was a nice feature to create the path automatically or add a hint that the path is missing.

Actions

Copy link

Updated by Susanne Moog over 5 years ago

Tags set to storage

Actions

Copy link

Updated by Stefan Bürk about 1 year ago

Tracker changed from Feature to Bug
Assignee set to Stefan Bürk
TYPO3 Version set to 11

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

Status changed from New to Under Review

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

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

Updated by Stefan Bürk about 1 year ago

We talked recently on the FAL code sprint about this.
TYPO3 should not create the base folder for a storage.
It must be created and ensure before or from outside.

The critical point is the "absolut" path posibility
and other checks.

However, we came up to make the error message in the file
kind of speaking in that case to point to it. I will
investigatge further do inform on saving the record or
something like that.

Actions

Copy link

Updated by Stefan Bürk about 1 year ago

Has duplicate Task #102022: "vanilla" mkdir @ core/Classes/Resource/Driver/LocalDriver.php added

Actions

Copy link

Updated by Stefan Bürk about 1 year ago

Has duplicate deleted (Task #102022: "vanilla" mkdir @ core/Classes/Resource/Driver/LocalDriver.php)

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#10

Updated by Gerrit Code Review about 1 year ago

Patch set 1 for branch 12.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/+/81277

Actions

Copy link

#11

Updated by Gerrit Code Review about 1 year ago

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

Actions

Copy link

#12