[Tickets #3230] RESOLVED: download all as zip under IE.

[bugs@bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/?id=3230
-----------------------------------------------------------------------
 Ticket             | 3230
 Updated By         | josephmalone at higlum.com
 Summary            | download all as zip under IE.
 Queue              | IMP
 Version            | 4.0.4
 State              | Bogus
 Priority           | 1. Low
 Type               | Bug
 Owners             | Michael Slusarz
-----------------------------------------------------------------------


josephmalone at higlum.com (2006-03-20 10:18) wrote:

> Found that when running under Internet Explorer certain attachments 
> will fail to be named correctly by the view.php scripts download_all 
> option.  Instead of $zipfile = trim($headers->getValue('subject')); 
> returning a useable name, a seemingly random 6-8 character name is 
> generated.
>
> Subsequently the if statement which appends the .zip to a file name 
> fails and causes the file to be saved without an extension.  While 
> this does not compromise the archive, it does cause confusion to some 
> users.
>
> Finally, when using the download all with multiple PDF attachments, I 
> have observed that an attempt to save the file returns an error 
> message of "The file is corrupt or contains no data".
>
> As a patch I added the following lines to view.php after
> "$zipfile = trim($headers->getValue('subject'));":
>
>     if ($browser->getBrowser() == 'msie')
>         $zipfile = _("attachment");
>
> and before
> "if (empty($zipfile)) {
>         $zipfile = _("attachments.zip");
>     } else {
>         $zipfile .= '.zip';
>     }"
>
> Under firefox the original function works correctly.

I had this same proble.  The above code fixed it for me.