[imp] problems with some types of multipart attachments

[Michael M Slusarz]

Quoting Ben Poliakoff <benp at reed.edu>:

> Hi All,
>
> I've encountered a problem with at least two types of multipart MIME
> messages (multipart/signed and multipart/appledouble).
>
> I'm running Horde/Imp from CVS HEAD (updated within the last few days).
> I've encountered this problem on two separate web servers.  The IMAP
> server is running Cyrus IMAP 2.2.8.
>
> The problem manifests itself when I try to view or download one of
> the "sub parts" of the multipart attachment.  In both cases (view or
> download) I get a "blank" browser window.
>
> If I look at the source of the pages that I'm getting from my servers I
> can't help but notice that the problematic URLs are missing key
> elements.
>
> When I look at the "view" URL for a non multipart attachment I see
> something like this:
>
> /horde/imp/view.php?popup_view=1&index=39&mailbox=INBOX.attachments&actionID=view_attach&id=2&mimecache=50e9a2337924b0c9179d0c603cf79c2b
>
> And I'm able to view or download the attached file.
>
> But the "view" URL for these problematic attachments looks more like
> this:
>
> /horde/imp/view.php?popup_view=1&actionID=view_attach&id=2&mimecache=1710837a257a3e9d5e1062e8a3e32e4d
>
> In other words it's missing the arguments that specify the message index
> number and the mailbox (i.e.: "index=39&mailbox=INBOX.attachments").
>
> If I dig around and find the message index number, then I can add it
> (and the mailbox) as arguments to the URL and I'm able to view or
> download the message.
>
> Does anyone have any idea why the "index" and "mailbox" arguments might
> be missing when viewing these multipart MIME messages?

Because, for embedded MIME parts, the 'index' and 'mailbox' doesn't 
exist as the
MIME part exists entirely independent of the underlying IMAP message data.

I've moved an overzealous empty $index check to the correct position in 
view.php
so this should be working again.

michael

_______________________________________
Michael Slusarz [slusarz at curecanti.org]