MIME parsed incorrectly - patch available

[rasky@…]

Starting from 0.1-rc1, Roundcube does not display correctly the attached mail. It looks like a regression in MIME parsing.

[rasky@…]

Mail not displayed correctly

[rasky@…]

This patch fixes the problem. Why is there a hardcoded "5" value? What is it for? There are many hardcoded indices for parts later in the same function...

Index: program/include/rcube_imap.inc
===================================================================
--- program/include/rcube_imap.inc      (revision 903)
+++ program/include/rcube_imap.inc      (working copy)
@@ -1054,7 +1054,8 @@ class rcube_imap

       $struct->parts = array();
       for ($i=0, $count=0; $i<count($part); $i++)
-        if (is_array($part[$i]) && count($part[$i]) > 5)
+        if (is_array($part[$i]) /*&& count($part[$i]) > 5*/)
           $struct->parts[] = $this->_structure_part($part[$i], ++$count, $struct->mime_id);

       return $struct;

[prodigy7]

I've the same problem and this fix solve this. Please apply this fix to the current svn version. Thx

[zeridon]

Confirmed, patch is working

version used: svn revision: 1049

attaching a sample mail ...

[zeridon]

Sample html mail which does not display in RC. Most of the headers are cut and part of info is obfuscated

[seansan]

Review in 0.1.1

[yitbos]

Replying to rasky@develer.com:

Starting from 0.1-rc1, Roundcube does not display correctly the attached mail. It looks like a regression in MIME parsing.

This still is not part of the released version

[alec]

Fixed in r1480.