Bug #64759
closed
File list shows folder identifier instead of path
Added by Christian Weiske over 9 years ago. Updated over 6 years ago.
100%
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
| 2015-02-02 filelist folder identifier.png (28.4 KB) 2015-02-02 filelist folder identifier.png | Christian Weiske, 2015-02-02 12:39 | ||
| 2015-02-04 typo3 file selector ids.png (49.3 KB) 2015-02-04 typo3 file selector ids.png | Christian Weiske, 2015-02-04 09:09 | ||
| 2015-03-12 flexform folder path.png (14.6 KB) 2015-03-12 flexform folder path.png | Christian Weiske, 2015-03-12 09:06 |
Updated by Christian Weiske over 9 years ago
This also happens in the file selection popup.
Updated by Christian Weiske over 9 years ago
The file selector shows the identifiers at 3 places.
Updated by Frans Saris over 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?
Updated by Frans Saris over 9 years ago
- Status changed from New to Needs Feedback
Updated by Frans Saris over 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.
Updated by Christian Weiske over 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.
Updated by Christian Weiske about 9 years ago
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>
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.
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
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
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
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
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
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
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
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
Updated by Christian Weiske almost 9 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset a85f18cd673a352a3680adaef8077b10a0b1cca4.
Updated by Riccardo De Contardi over 6 years ago
- Status changed from Resolved to Closed