Do you have debug mode on in your settings? You're probably running into a fatal error, I don't believe download is a method available on that object.

Storage also isn't a response object.

Try something more like this:

public function get($filename){
 
 $entry = Fileentry::where('filename', '=', $filename)->firstOrFail();
 $file = Storage::disk('local')->get($entry->filename);
 
 return (new Response($file, 200))
              ->header('Content-Type', $entry->mime);
 }

See this for more.

It worked!

$file = Storage::disk('local')->get('sheets/models/sheet1.xlsx');
return (new Response($file, 200))->header('Content-Type', 'application/vnd.ms-excel');

But how can I set the filename? When I download, I am getting the action name as filename and no extension (should be xlsx).

Just took a quick look at the laravel docs. It'd be easier for you to just do this:

return Response::download('sheets/models/sheet1.xlsx');

Not sure how good the quality of code on that other site is. ;)

@willvincent, this doesn't work for files in Storage (or I am doing something wrong). Maybe in public directory: public/files/test.txt. If I try this code I get FileNotFoundException. I also tried prepending with storage_path() function. But doesn't work too.

Hmm.. alright, well in that case, add this header to set the filename:

header('Content-Disposition: attachment; filename="sheet1.xlsx"');

So all together it should be:

$file = Storage::disk('local')->get('sheets/models/sheet1.xlsx');
return (new Response($file, 200))
    ->header('Content-Type', 'application/vnd.ms-excel')
    ->header('Content-Disposition: attachment; filename="sheet1.xlsx"');

If you can get the filesize, you may also want to include a Content-Length header.

Awesome. Just one more question. Do you know how to get the filename with Storage? I tried with:

Storage::disk('local')->name('sheets/models/sheet1.xlsx');

But this method does not exist. I took a look at Laravel API and I found the method here: http://laravel.com/api/5.0/Illuminate/Filesystem/Filesystem.html

However, if I take a look in Illuminate\Filesystem\FilesystemAdapter the method name() is not there, so I am little confused here.

Technically, you're already passing a filename to the get method.. so you should already know it.

If what you're really after is how to get the filename off that string you've been using in your example, then just explode the string on / and grab the last element.

$parts = explode('/', 'sheets/models/sheet1.xlsx');
$filename = array_pop($parts);