Fix for downloading of lightbox zip in Safari

[adrianbj]

Just wanted to let you all know that currently if you try to download the contents of the lightbox (or whatever it is called in the default installation) as a zip file with Safari (on Windows at least, not sure about Mac), then it will get downloaded as zipdownload.php, instead of pictures.zip

This is easily fixed by changing (in archive.php):

Code Select Expand

@header("Content-disposition: attachment; filename=$filename");

to:

Code Select Expand

@header('Content-Disposition: attachment; filename="' . $filename . '"');

Hope that helps everyone!

[Nibbler]

Why are double quotes needed around the filename? The RFC does not require this.

[adrianbj]

Nibbler,

I honestly don't know why they are required - I haven't looked into it. All I know is that it works like this and doesn't the other way it doesn't.

[Nibbler]

All I know is that it works like this and doesn't the other way it doesn't.

That sentence don't make no sense it doesn't.

[adrianbj]

Yeah, I am real smart I am