Skip to content
Snippets Groups Projects
Commit 61137e14 authored by Oliver Bartsch's avatar Oliver Bartsch
Browse files

[BUGFIX] Apply sorting parameters on filelist initialization

TO be consistent through the backend, filelist
instances are used for all the various file
and folder browsers in the backend.

To enable sorting, the corresponding request
parameters are now properly applied on
initialization of the filelist instances.

Resolves: #101307
Releases: main, 12.4
Change-Id: I86409831abb7f553875806a7e4f0e91f8d71d58e
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/79959


Tested-by: default avatarcore-ci <typo3@b13.com>
Reviewed-by: default avatarOliver Bartsch <bo@cedev.de>
Tested-by: default avatarOliver Bartsch <bo@cedev.de>
parent f5f0cacb
Branches
Tags
No related merge requests found
......@@ -67,7 +67,13 @@ class CreateFolderBrowser extends AbstractResourceBrowser
$markup[] = '</div>';
// Create the filelist
$this->filelist->start($this->selectedFolder, MathUtility::forceIntegerInRange($this->currentPage, 1, 100000), 'asc', false, Mode::BROWSE);
$this->filelist->start(
$this->selectedFolder,
MathUtility::forceIntegerInRange($this->currentPage, 1, 100000),
$this->getRequest()->getQueryParams()['sort'] ?? '',
($this->getRequest()->getQueryParams()['reverse'] ?? '') === '1',
Mode::BROWSE
);
$this->filelist->setResourceDisplayMatcher($this->resourceDisplayMatcher);
$this->filelist->setResourceSelectableMatcher($this->resourceSelectableMatcher);
$markup[] = $this->filelist->render(null, $this->view);
......
......@@ -76,7 +76,13 @@ class FolderBrowser extends AbstractResourceBrowser
$markup[] = '</div>';
// Create the filelist
$this->filelist->start($this->selectedFolder, MathUtility::forceIntegerInRange($this->currentPage, 1, 100000), 'asc', false, Mode::BROWSE);
$this->filelist->start(
$this->selectedFolder,
MathUtility::forceIntegerInRange($this->currentPage, 1, 100000),
$this->getRequest()->getQueryParams()['sort'] ?? '',
($this->getRequest()->getQueryParams()['reverse'] ?? '') === '1',
Mode::BROWSE
);
$this->filelist->setResourceDisplayMatcher($this->resourceDisplayMatcher);
$this->filelist->setResourceSelectableMatcher($this->resourceSelectableMatcher);
$markup[] = $this->filelist->render(null, $this->view);
......
......@@ -71,7 +71,13 @@ class FileLinkHandler extends AbstractResourceLinkHandler
$markup[] = '</div>';
// Create the filelist
$this->filelist->start($this->selectedFolder, MathUtility::forceIntegerInRange($this->currentPage, 1, 100000), 'asc', false, Mode::BROWSE);
$this->filelist->start(
$this->selectedFolder,
MathUtility::forceIntegerInRange($this->currentPage, 1, 100000),
$request->getQueryParams()['sort'] ?? '',
($request->getQueryParams()['reverse'] ?? '') === '1',
Mode::BROWSE
);
$this->filelist->setResourceDisplayMatcher($this->resourceDisplayMatcher);
$this->filelist->setResourceSelectableMatcher($this->resourceSelectableMatcher);
......
......@@ -58,7 +58,13 @@ class FolderLinkHandler extends AbstractResourceLinkHandler
$markup[] = '</div>';
// Create the filelist
$this->filelist->start($this->selectedFolder, MathUtility::forceIntegerInRange($this->currentPage, 1, 100000), 'asc', false, Mode::BROWSE);
$this->filelist->start(
$this->selectedFolder,
MathUtility::forceIntegerInRange($this->currentPage, 1, 100000),
$request->getQueryParams()['sort'] ?? '',
($request->getQueryParams()['reverse'] ?? '') === '1',
Mode::BROWSE
);
$this->filelist->setResourceDisplayMatcher($this->resourceDisplayMatcher);
$this->filelist->setResourceSelectableMatcher($this->resourceSelectableMatcher);
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment