[dev] [cvs] commit: imp/lib/MIME/Viewer appledouble.php

[Jan Schneider]

Zitat von Michael M Slusarz <slusarz at horde.org>:

> Quoting Jan Schneider <jan at horde.org>:
>
>> jan         2008-10-06 11:57:08 EDT
>>
>>  Modified files:
>>    lib/MIME/Viewer      appledouble.php
>>  Log:
>>  Force multipart/appledouble parts to be rendered inline.
>>
>>  Revision  Changes    Path
>>  1.26      +9 -1      imp/lib/MIME/Viewer/appledouble.php
>
> As previously noted in a discussion on this list, appledouble  
> attachments are often large and are extremely memory-intensive to  
> process.  Setting this to view inline is going to cause a bunch more  
> white-screens (memory exhaustion errors) which will occur in the  
> main browsing screen rather than a popup view window, which means it  
> is much more invasive.
>
> I would prefer we hold off on this kind of change until we can get  
> the new IMAP/MIME libs in place.  That would be a better time to  
> reevaluate and see if our resource usage has been lowered  
> sufficiently to make this a good idea.

Without this change, appledouble parts nested inside other multipart  
message parts won't show up at all. User won't even notice that there  
is such a message. I'd rather have behavior that uses more resources  
which can be unlimited by the admin if they decide to do so, than  
broken behavior that admins can't do anything about at all.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/