diff options
author | Carl Schwan <carl@carlschwan.eu> | 2022-02-12 00:26:25 +0300 |
---|---|---|
committer | Carl Schwan <carl@carlschwan.eu> | 2022-02-17 19:06:33 +0300 |
commit | 028ca0935c2a0be18bad5997167289be013d6a3d (patch) | |
tree | 266b239cb41ad4e8fa3b548bb98abbb223cb82f1 | |
parent | fbbbc2ec3445aa777f56c97a0a383a2e8566be54 (diff) |
Don't redirect when loading files index page
Currently we are redirecting from ?dir=/&fileid=2 to ?dir=/. This is an
issue because we then need to load two pages with full file system setup
and authentification instead of one and the assets won't start loading
until the second page is delivered to the user.
Additionally when loading ?dir=/, we then change the url back to
?dir=/&fileid=2 (without reload) so that the next time we load the page
again we do the same thing again.
Depending on the speed of the server and internet connection we can save
100ms to 400ms, improving the user experience.
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
-rw-r--r-- | apps/files/lib/Controller/ViewController.php | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/apps/files/lib/Controller/ViewController.php b/apps/files/lib/Controller/ViewController.php index bfd931c0b2a..587f75f3f02 100644 --- a/apps/files/lib/Controller/ViewController.php +++ b/apps/files/lib/Controller/ViewController.php @@ -184,7 +184,7 @@ class ViewController extends Controller { * @throws NotFoundException */ public function index($dir = '', $view = '', $fileid = null, $fileNotFound = false, $openfile = null) { - if ($fileid !== null) { + if ($fileid !== null && $dir === '') { try { return $this->redirectToFile($fileid); } catch (NotFoundException $e) { |