Nemcio/GitList
1
0
Fork 0
mirror of https://github.com/klaussilveira/gitlist.git synced 2025-11-17 19:20:56 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #588 from bjoernffm/master

Browse Source 
Fixed issue #584
This commit is contained in:
Klaus Silveira
2015-04-29 17:56:08 -03:00
parent dbc5b0e53f ee1273529b
commit 8ed6d8b978
1 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/GitList/Controller/TreeController.php
View File

@@ -93,7 +93,18 @@ class TreeController implements ControllerProviderInterface
$repository->createArchive($tree, $file, $format); $repository->createArchive($tree, $file, $format);
} }
return new BinaryFileResponse($file); /**
* Generating name for downloading, lowercasing and removing all non
* ascii and special characters
*/
$filename = strtolower($branch);
$filename = preg_replace('#[^a-z0-9]#', '_', $filename);
$filename = preg_replace('#_+#', '_', $filename);
$filename = $filename . '.' . $format;
$response = new BinaryFileResponse($file);
$response->setContentDisposition('attachment', $filename);
return $response;
})->assert('format', '(zip|tar)') })->assert('format', '(zip|tar)')
->assert('repo', $app['util.routing']->getRepositoryRegex()) ->assert('repo', $app['util.routing']->getRepositoryRegex())
->assert('branch', $app['util.routing']->getBranchRegex()) ->assert('branch', $app['util.routing']->getBranchRegex())