Project

General

Profile

Actions

Bug #85323

closed

Create dir if user create a local storage

Added by Guido Schmechel almost 6 years ago. Updated 6 months ago.

Status:
Resolved
Priority:
Could have
Assignee:
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2018-06-21
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
11
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 #1

Updated by Susanne Moog about 5 years ago

  • Tags set to storage
Actions #2

Updated by Stefan Bürk 6 months ago

  • Tracker changed from Feature to Bug
  • Assignee set to Stefan Bürk
  • TYPO3 Version set to 11
Actions #3

Updated by Gerrit Code Review 6 months 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 #4

Updated by Gerrit Code Review 6 months 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 #5

Updated by Stefan Bürk 6 months 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 #6

Updated by Stefan Bürk 6 months ago

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

Updated by Stefan Bürk 6 months ago

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

Updated by Gerrit Code Review 6 months 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 #9

Updated by Gerrit Code Review 6 months 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 #10

Updated by Gerrit Code Review 6 months 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 #11

Updated by Gerrit Code Review 6 months 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 #12

Updated by Stefan Bürk 6 months ago

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

Also available in: Atom PDF