[dev] Re: [cvs] commit: imp/lib/MIME/Viewer zip.php imp/docs CHANGES

[Michael Cochrane]

Directories shouldn't be downloadable :-)

Also when the archive is displayed inline the font for the links is messed up
and the columns don't line up... will probably need to define some fixed width
style for things like this....

- Mike :-)


Index: imp/lib/MIME/Viewer/zip.php
===================================================================
RCS file: /httpd/horde/imp/lib/MIME/Viewer/zip.php,v
retrieving revision 1.3
diff -I$Horde -I$Revision -I$Date -u -r1.3 zip.php
--- imp/lib/MIME/Viewer/zip.php	2002/07/09 21:57:39	1.3
+++ imp/lib/MIME/Viewer/zip.php	2002/07/10 01:24:14
@@ -102,7 +102,7 @@
     function _callback($key, $val)
     {
         $name = trim($val['name']);
-        if (!empty($val['size']) &&
+        if (!empty($val['size']) && !($val['_attr'] & 0x10) &&
             ((($val['_method'] == 0x8) && function_exists('gzinflate')) ||
             ($val['_method'] == 0x0))) {
             $val['name'] = str_replace($name,
$this->contents->linkView($this->mime_part, DOWNLOAD_RENDER, $name,
sprintf(_("View %s"), $name), array('zip_attachment' => (urlencode($key) + 1))),
$val['name']); 


Quoting Michael M Slusarz <slusarz@bigworm.colorado.edu>:

> slusarz     2002/07/08 10:07:13 PDT
> 
>   Modified files:
>     lib/MIME/Viewer      zip.php 
>     docs                 CHANGES 
>   Log:
>   More correct authorship information; add Horde tag.
>   
>   Revision  Changes    Path
>   1.519     +2 -1      imp/docs/CHANGES
>   1.2       +3 -1      imp/lib/MIME/Viewer/zip.php
> 
>   Chora Links:
>   http://cvs.horde.org/diff.php/imp/docs/CHANGES?r1=1.518&r2=1.519&ty=u
>  
> http://cvs.horde.org/diff.php/imp/lib/MIME/Viewer/zip.php?r1=1.1&r2=1.2&ty=u
> 
> -- 
> To unsubscribe, mail: cvs-unsubscribe@lists.horde.org


-- 

The purpose of learning is growth, and our minds, unlike our bodies, can
continue growing as we continue to live. - Morris Adler 
signature from Siglets.com

-------------------------------------------------------------------------
This mail sent from Mike's CVS HEAD install of IMP: http://horde.org/imp/