Project

General

Profile

Actions

Bug #64759

closed

File list shows folder identifier instead of path

Added by Christian Weiske about 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2015-02-02
Due date:
% Done:

100%

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

Description

The file list (File > File list) shows the folder identifier on the top right:

fo-d5cd887562a7435799099e0a1f92a4c3/ 7 Files, 861 KB

This works fine for local storages which use the path as identifier, but not for other FAL drivers that have "real" (unreadable) IDs.


Files


Related issues 3 (0 open3 closed)

Related to TYPO3 Core - Bug #58150: Improve "Are you sure you want to delete" messageClosed2014-04-24

Actions
Related to TYPO3 Core - Bug #64840: File move message shows identifiers instead of namesClosed2015-02-04

Actions
Precedes TYPO3 Core - Bug #67148: Can only access first mount pointClosed2015-05-26

Actions
Actions #1

Updated by Christian Weiske about 9 years ago

This also happens in the file selection popup.

Actions #2

Updated by Christian Weiske about 9 years ago

The file selector shows the identifiers at 3 places.

Actions #3

Updated by Frans Saris about 9 years ago

Not sure how to fix this. You are right that in your case the name would make more sense here. But for hierarchical file systems the full path makes more sense.

Maybe we should add the folder name after the icon just like is done for database elements.

Do you have any other suggestions?

Actions #4

Updated by Frans Saris about 9 years ago

  • Status changed from New to Needs Feedback
Actions #5

Updated by Frans Saris about 9 years ago

Place to fix the title is typo3/sysext/filelist/Classes/FileList.php line 243

Maybe we could add a getReadablePath() to the Folder object that walks through all parent folders until root of storage and glues al folder names together with a / as separator. That would give the best result in most cases I guess.

Actions #6

Updated by Christian Weiske about 9 years ago

I think the full path should be calculated, and then trimmed to some 30 characters or so (the end on the right being fully visible).
The full path could be shown in the title attribute.

In the file selection popup, the single name may be adequate.

Actions #8

Updated by Christian Weiske about 9 years ago

Flexforms have the same problem when using folder selection - see screenshot:

<config>
    <type>group</type>
    <internal_type>folder</internal_type>
    <allowed>*</allowed>
    <size>1</size>
    <maxitems>1</maxitems>
    <minitems>1</minitems>
</config>
Actions #9

Updated by Markus Klein about 9 years ago

  • Status changed from Needs Feedback to Accepted
  • Target version set to 7.4 (Backend)

Additionally the storage-id has to be translated to the storage name.

Actions #10

Updated by Gerrit Code Review about 9 years ago

  • Status changed from Accepted 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/38763

Actions #11

Updated by Gerrit Code Review about 9 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/38763

Actions #12

Updated by Gerrit Code Review about 9 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/38763

Actions #13

Updated by Gerrit Code Review about 9 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/38763

Actions #14

Updated by Gerrit Code Review about 9 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/38763

Actions #15

Updated by Gerrit Code Review about 9 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/38763

Actions #16

Updated by Gerrit Code Review almost 9 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/38763

Actions #17

Updated by Gerrit Code Review almost 9 years ago

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

Actions #18

Updated by Christian Weiske almost 9 years ago

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

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF