From 5b7b101a8ad743879cfa10a8f286bd38f3a6df71 Mon Sep 17 00:00:00 2001 From: Pierre Rudloff Date: Fri, 28 Dec 2018 20:01:46 +0100 Subject: [PATCH] refactor(FrontController): We don't need to set the filename twice --- controllers/FrontController.php | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/controllers/FrontController.php b/controllers/FrontController.php index 370d4d8..e0b2633 100644 --- a/controllers/FrontController.php +++ b/controllers/FrontController.php @@ -473,15 +473,15 @@ class FrontController } $webpageUrl = $request->getQueryParam('url'); - return $response->withHeader('Content-Disposition', 'attachment; filename="'.pathinfo( - $this->download->getFileNameWithExtension( + return $response->withHeader( + 'Content-Disposition', + 'attachment; filename="'.$this->download->getFileNameWithExtension( 'mkv', $webpageUrl, $format, $this->sessionSegment->getFlash($webpageUrl) - ), - PATHINFO_FILENAME - ).'.mkv"'); + ) + ); } /**