[Tickets #3230] NEW: 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
 Created By         | tdrewry at bu.edu
 Summary            | download all as zip under IE.
 Queue              | IMP
 Version            | 4.0.4
 State              | Unconfirmed
 Priority           | 1. Low
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


tdrewry at bu.edu (2006-01-06 19:11) 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.