You are right. This is bogus. My imp/lib/MIME/Contents.php is out of sync.

I'm pretty sure I fixed this a month ago:

http://cvs.horde.org/diff.php/imp/lib/MIME/Contents.php?r1=1.145&r2=1.146&ty=h

You mean reproduce (a) findBody() getting a multipart/alternative, or 
(b) findBody() not selecting one from among the alternatives?



I didn't think to investigate (a) (it seems reasonable given a 
multipart message), but once that's the case, (b) pretty certainly 
follows.

I can't reproduce this, by the way.

It takes the whole thing, i.e., all alternatives. Hence such messages 
are duplicated in the reply (plain and html). As an additional side 
effect, there is no quoted-printable decoding of the sub-parts, so 
user sees all the confusing encoding characters.



Suggest enhancement to select out one of the alternatives and apply 
proper decoding to it, e.g.,

cvs diff -u -r HEAD imp/lib/Compose.php

Index: imp/lib/Compose.php

===================================================================

RCS file: /repository/imp/lib/Compose.php,v

retrieving revision 1.105

diff -u -r1.105 Compose.php

--- imp/lib/Compose.php 2 Nov 2004 01:56:02 -0000       1.105

+++ imp/lib/Compose.php 2 Nov 2004 05:23:51 -0000

@@ -154,7 +154,6 @@

          }



          $mime_part = &$imp_contents->getDecodedMIMEPart($this->_mimeid);

-        $body = $mime_part->getContents();



          //if ($mime_message->getType() == 'multipart/encrypted') {

              /* TODO: Maybe someday I can figure out how to show embedded

@@ -162,6 +161,28 @@

          //    return '[' . _("Original message was encrypted") . ']';

          //}



+       if ($mime_part->getType() == 'multipart/alternative')

+       {

+           // Select the text/plain  alternative if possible, else 
text/html, else all

+           $parts = $mime_part->getParts();

+           $prefTypes = array('text/plain', 'text/html');

+           $done = false;

+           for ($i = 0; !$done && $i < count($prefTypes); $i++)

+           {

+               for ($j = 1; !$done && $j <= count($parts); $j++)

+               {

+                   if ($parts[$j]->getType() == $prefTypes[$i])

+                   {

+                       $mime_part = $parts[$j];

+                       $mime_part->transferDecodeContents();

+                       $done = true;

+                   }

+               }

+           }

+       }

+

+        $body = $mime_part->getContents();

+

          if ($mime_part->getSubType() == 'html') {

              require_once 'Horde/Text/Filter.php';

              return Text_Filter::filter($body, 'html2text');