Project

General

Profile

Actions

Bug #56746

closed

File list always fetches all files in folder

Added by Christian Weiske about 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2014-03-11
Due date:
% Done:

50%

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

Description

When using the filelist module to list files in a folder of a FAL storage, FileList.php always requests all files at once, even though only 40 are displayed at the same time.

This is problematic if there is a huge number of files in that folder that need to be fetched remotely via an API.

It would be better if at first a call would be made to fetch the number of files in that folder (to make pagination calculation work), and then only request needed files by setting $start and $numberOfItems arguments in getFilesInFolder() correctly.

Happens with TYPO3 6.2beta6.


Related issues 7 (0 open7 closed)

Related to TYPO3 Core - Feature #64764: FileList and ElementBrowser must implement a paginate system for filesClosed2015-02-02

Actions
Related to TYPO3 Core - Task #66327: Rename FAL driver count methodsClosedMathias Brodala2015-04-08

Actions
Related to TYPO3 Core - Bug #75370: OutOfBoundsException in LocalDriver.phpClosed2016-04-01

Actions
Related to TYPO3 Core - Bug #73754: LocalDriver sorting bug if multiple file with same timestampClosedFrank Nägler2016-02-29

Actions
Related to TYPO3 Core - Feature #66466: Tell FAL driver that we will fetch 40 files nowClosed2015-04-17

Actions
Has duplicate TYPO3 Core - Bug #57811: filelist and file wizard very slow for more file than usualClosed2014-04-10

Actions
Precedes TYPO3 Core - Feature #66464: File selection popup does not use FAL driver count*() methodsClosed2015-04-17

Actions
Actions #1

Updated by Gerrit Code Review about 10 years ago

  • Status changed from New 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 https://review.typo3.org/28560

Actions #2

Updated by Gerrit Code Review about 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 https://review.typo3.org/28560

Actions #3

Updated by Gerrit Code Review over 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/28560

Actions #4

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/28560

Actions #5

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/28560

Actions #6

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/28560

Actions #7

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

Actions #8

Updated by Christian Weiske about 9 years ago

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

Updated by José Ricardo over 7 years ago

Is this solution also applied for TYPO3 7.6? I'm getting the same issue there. (TYPO3 7.6.11)

Actions #10

Updated by André Richter over 6 years ago

  • Priority changed from Should have to Must have

The solution isn't applied for the search-function (v8.7).
When searching for a file in dam the new parameters ($sort,...) are not used when calling the function getFoldersInFolder.
searchByName (core/Classes/Resource/FileRepository.php)

Actions #11

Updated by André Richter over 6 years ago

  • % Done changed from 100 to 50
Actions #12

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions #13

Updated by Benni Mack about 4 years ago

  • Related to Feature #66466: Tell FAL driver that we will fetch 40 files now added
Actions

Also available in: Atom PDF