reading of body structure most part finished

8 files changed, 456 insertions, 112 deletions

diff --git a/noncore/net/mail/imapwrapper.cpp b/noncore/net/mail/imapwrapper.cpp
index e6e9fc9..27f1410 100644
--- a/noncore/net/mail/imapwrapper.cpp
+++ b/noncore/net/mail/imapwrapper.cpp
@@ -24,9 +24,13 @@ void IMAPwrapper::imap_progress( size_t current, size_t maximum )
 void IMAPwrapper::login()
 {
-    logout();
     const char *server, *user, *pass;
     uint16_t port;
     int err = MAILIMAP_NO_ERROR;
 
+    /* we are connected this moment */
+    /* TODO: setup a timer holding the line or if connection closed - delete the value */
+    if (m_imap) {   
+        return;
+    }
     server = account->getServer().latin1();
     port = account->getPort().toUInt();
@@ -36,6 +40,5 @@ void IMAPwrapper::login()
     m_imap = mailimap_new( 20, &imap_progress );
     /* connect */
-  //  err = mailimap_socket_connect( m_imap, (char*)server, port );
-  if (account->getSSL()) {
+    if (account->getSSL()) {
         err = mailimap_ssl_connect( m_imap, (char*)server, port );
     } else {
@@ -78,5 +81,5 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
     clist *result;
     clistcell *current;
-    mailimap_fetch_att *fetchAtt,*fetchAttFlags,*fetchAttDate;
+//    mailimap_fetch_att *fetchAtt,*fetchAttFlags,*fetchAttDate,*fetchAttSize;
     mailimap_fetch_type *fetchType;
     mailimap_set *set;
@@ -91,5 +94,4 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
     if ( err != MAILIMAP_NO_ERROR ) {
         qDebug("error selecting mailbox: %s",m_imap->imap_response);
-        logout();
         return;
     }
@@ -99,5 +101,4 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
     if (last == 0) {
         qDebug("mailbox has no mails");
-        logout();
         return;
     }
@@ -106,17 +107,12 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
     /* the range has to start at 1!!! not with 0!!!! */
     set = mailimap_set_new_interval( 1, last );
-    fetchAtt = mailimap_fetch_att_new_envelope();
-    fetchAttFlags = mailimap_fetch_att_new_flags();
-    fetchAttDate = mailimap_fetch_att_new_internaldate();
-
-    //fetchType = mailimap_fetch_type_new_fetch_att(fetchAtt);
     fetchType = mailimap_fetch_type_new_fetch_att_list_empty();
-    mailimap_fetch_type_new_fetch_att_list_add(fetchType,fetchAtt);
-    mailimap_fetch_type_new_fetch_att_list_add(fetchType,fetchAttFlags);
-    mailimap_fetch_type_new_fetch_att_list_add(fetchType,fetchAttDate);
+    mailimap_fetch_type_new_fetch_att_list_add(fetchType,mailimap_fetch_att_new_envelope());
+    mailimap_fetch_type_new_fetch_att_list_add(fetchType,mailimap_fetch_att_new_flags());
+    mailimap_fetch_type_new_fetch_att_list_add(fetchType,mailimap_fetch_att_new_internaldate());
+    mailimap_fetch_type_new_fetch_att_list_add(fetchType,mailimap_fetch_att_new_rfc822_size());
 
     err = mailimap_fetch( m_imap, set, fetchType, &result );
     mailimap_set_free( set );
-    /* cleans up the fetch_att's too! */
     mailimap_fetch_type_free( fetchType );
 
@@ -124,8 +120,8 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
 
     if ( err == MAILIMAP_NO_ERROR ) {
-        current = clist_begin(result);
+        
         mailimap_msg_att * msg_att;
         int i = 0;
-        while ( current != 0 ) {
+        for (current = clist_begin(result); current != 0; current=clist_next(current)) {
             ++i;
             msg_att = (mailimap_msg_att*)current->data;
@@ -136,11 +132,9 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
                 target.append(m);
             }
-            current = current->next;
         }
     } else {
         qDebug("Error fetching headers: %s",m_imap->imap_response);
     }
-    logout();
-    clist_free(result);
+    mailimap_fetch_list_free(result);
 }
 
@@ -220,4 +214,5 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
     mailimap_msg_att_dynamic*flist;
     mailimap_flag_fetch*cflag;
+    int size;
     QBitArray mFlags(7);
     QStringList addresslist;
@@ -226,9 +221,8 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
         return m;
     }
-
-    c = clist_begin(m_att->att_list);
-    while ( c ) {
+    m = new RecMail();
+    for (c = clist_begin(m_att->att_list); c!=NULL;c=clist_next(c) ) {
         current = c;
-        c = c->next;
+        size = 0;
         item = (mailimap_msg_att_item*)current->data;
         if (item->att_type!=MAILIMAP_MSG_ATT_ITEM_STATIC) {
@@ -238,5 +232,5 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
             }
             cf = flist->att_list->first;
-            while (cf) {
+            for (cf = clist_begin(flist->att_list); cf!=NULL; cf = clist_next(cf)) {
                 cflag = (mailimap_flag_fetch*)cf->data;
                 if (cflag->fl_type==MAILIMAP_FLAG_FETCH_OTHER && cflag->fl_flag!=0) {
@@ -267,13 +261,9 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
                     mFlags.setBit(FLAG_RECENT);
                 }
-                cf = cf->next;
             }
             continue;
         }
-        if ( item->att_data.att_static->att_type == MAILIMAP_MSG_ATT_RFC822_HEADER ) {
-            qDebug( "header: \n%s", item->att_data.att_static->att_data.att_rfc822_header );
-        } else if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_ENVELOPE) {
+        if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_ENVELOPE) {
             mailimap_envelope * head = item->att_data.att_static->att_data.att_env;
-            m = new RecMail();
             m->setDate(head->env_date);
             m->setSubject(head->env_subject);
@@ -308,6 +298,6 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
             qDebug("%i %i %i - %i %i %i",d->dt_year,d->dt_month,d->dt_day,d->dt_hour,d->dt_min,d->dt_sec);
             qDebug(da.toString());
-        } else {
-            qDebug("Another type");
+        } else if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_RFC822_SIZE) {
+            size = item->att_data.att_static->att_data.att_rfc822_size;
         }
     }
@@ -319,4 +309,5 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
     if (m) {
         m->setFlags(mFlags);
+        m->setMsgsize(size);
     }
     return m;
@@ -345,11 +336,11 @@ RecBody IMAPwrapper::fetchBody(const RecMail&mail)
     if ( err != MAILIMAP_NO_ERROR ) {
         qDebug("error selecting mailbox: %s",m_imap->imap_response);
-        logout();
         return body;
     }
+ 
     result = clist_new();
     /* the range has to start at 1!!! not with 0!!!! */
     set = mailimap_set_new_interval( mail.getNumber(),mail.getNumber() );
-    fetchAtt = mailimap_fetch_att_new_body();
+    fetchAtt = mailimap_fetch_att_new_bodystructure();
     fetchType = mailimap_fetch_type_new_fetch_att(fetchAtt);
     err = mailimap_fetch( m_imap, set, fetchType, &result );
@@ -364,16 +355,18 @@ RecBody IMAPwrapper::fetchBody(const RecMail&mail)
         if (body_desc->bd_type==MAILIMAP_BODY_1PART) {
             searchBodyText(mail,body_desc->bd_data.bd_body_1part,body);
-        } else {
+        } else if (body_desc->bd_type==MAILIMAP_BODY_MPART) {
+            qDebug("Mulitpart mail");
+            searchBodyText(mail,body_desc->bd_data.bd_body_mpart,body,0);
         }
-
     } else {
         qDebug("error fetching body: %s",m_imap->imap_response);
     }
-
-    clist_free(result);
-    logout();
+    mailimap_fetch_list_free(result);
     return body;
 }
 
+/* this routine is just called when the mail has only ONE part.
+   for filling the parts of a multi-part-message there are other
+   routines 'cause we can not simply fetch the whole body. */
 void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mailDescription,RecBody&target_body)
 {
@@ -381,7 +374,18 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mai
         return;
     }
+    QString sub;
     switch (mailDescription->bd_type) {
+    case MAILIMAP_BODY_TYPE_1PART_MSG:
+        target_body.setType("text");
+        sub = mailDescription->bd_data.bd_type_text->bd_media_text;
+        target_body.setSubtype(sub.lower());
+        fillPlainBody(mail,target_body);
+        break;
     case MAILIMAP_BODY_TYPE_1PART_TEXT:
-            fillPlainBody(mail,target_body,mailDescription->bd_data.bd_type_text);
+        qDebug("Mediatype single: %s",mailDescription->bd_data.bd_type_text->bd_media_text);
+        target_body.setType("text");
+        sub = mailDescription->bd_data.bd_type_text->bd_media_text;
+        target_body.setSubtype(sub.lower());
+        fillPlainBody(mail,target_body);
         break;
     default:
@@ -391,5 +395,5 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mai
 }
 
-void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body, mailimap_body_type_text * bd)
+void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body)
 {
     const char *mb;
@@ -427,8 +431,11 @@ void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body, mailimap
                 if (msg_att_item->att_data.att_static->att_type == MAILIMAP_MSG_ATT_BODY_SECTION) {
                     char*text = msg_att_item->att_data.att_static->att_data.att_body_section->sec_body_part;
-                    int length = msg_att_item->att_data.att_static->att_data.att_body_section->sec_length;
                     msg_att_item->att_data.att_static->att_data.att_body_section->sec_body_part = 0L;
-                    body = QString(text);
-                    free(text);
+                    if (text) {
+                        body = QString(text);
+                        free(text);
+                    } else {
+                        body = "";
+                    }
                 }
             }
@@ -438,5 +445,4 @@ void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body, mailimap
         qDebug("error fetching text: %s",m_imap->imap_response);
     }
-    //clist_free(result);
     mailimap_fetch_list_free(result);
     target_body.setBodytext(body);
@@ -454,11 +460,9 @@ QStringList IMAPwrapper::address_list_to_stringlist(clist*list)
         return l;
     }
-    current = clist_begin(list);
     unsigned int count = 0;
-    while (current!= NULL) {
+    for (current=clist_begin(list);current!= NULL;current=clist_next(current)) {
         from = "";
         named_from = false;
         current_address=(mailimap_address*)current->data;
-        current = current->next;
         if (current_address->ad_personal_name){
             from+=QString(current_address->ad_personal_name);
@@ -486,2 +490,116 @@ QStringList IMAPwrapper::address_list_to_stringlist(clist*list)
     return l;
 }
+
+void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_mpart*mailDescription,RecBody&target_body,int current_recursion)
+{
+    /* current_recursion is for avoiding ugly mails which has a to deep body-structure */
+    if (!mailDescription||current_recursion==2) {
+        return;
+    }
+    qDebug("Mediatype: %s",mailDescription->bd_media_subtype);
+    clistcell*current;
+    mailimap_body*current_body;
+    for (current=clist_begin(mailDescription->bd_list);current!=0;current=clist_next(current)) {
+        current_body = (mailimap_body*)current->data;
+        if (current_body->bd_type==MAILIMAP_BODY_MPART) {
+            searchBodyText(mail,current_body->bd_data.bd_body_mpart,target_body,current_recursion+1);
+        } else if (current_body->bd_type==MAILIMAP_BODY_1PART){
+            RecPart currentPart;
+            fillSinglePart(currentPart,current_body->bd_data.bd_body_1part);
+            target_body.addPart(currentPart);
+        }
+    }
+    if (current_recursion==0) {
+        
+    }
+}
+
+void IMAPwrapper::fillSinglePart(RecPart&target_part,mailimap_body_type_1part*Description)
+{
+    if (!Description) {
+        return;
+    }
+    switch (Description->bd_type) {
+        case MAILIMAP_BODY_TYPE_1PART_TEXT:
+            target_part.setType("text");
+            fillSingleTextPart(target_part,Description->bd_data.bd_type_text);
+            break;
+        case MAILIMAP_BODY_TYPE_1PART_BASIC:
+            fillSingleBasicPart(target_part,Description->bd_data.bd_type_basic);
+            break;
+        default:
+            break;
+    }
+}
+
+void IMAPwrapper::fillSingleTextPart(RecPart&target_part,mailimap_body_type_text*which)
+{
+    if (!which) {
+        return;
+    }
+    QString sub;
+    sub = which->bd_media_text;
+    target_part.setSubtype(sub.lower());
+    target_part.setLines(which->bd_lines);
+    fillBodyFields(target_part,which->bd_fields);
+}
+
+void IMAPwrapper::fillSingleBasicPart(RecPart&target_part,mailimap_body_type_basic*which)
+{
+    if (!which) {
+        return;
+    }
+    QString type,sub;
+    switch (which->bd_media_basic->med_type) {
+    case MAILIMAP_MEDIA_BASIC_APPLICATION:
+        type = "application";
+        break;
+    case MAILIMAP_MEDIA_BASIC_AUDIO:
+        type = "audio";
+        break;
+    case MAILIMAP_MEDIA_BASIC_IMAGE:
+        type = "image";
+        break;
+    case MAILIMAP_MEDIA_BASIC_MESSAGE:
+        type = "message";
+        break;
+    case MAILIMAP_MEDIA_BASIC_VIDEO:
+        type = "video";
+        break;
+    case MAILIMAP_MEDIA_BASIC_OTHER:
+    default:
+        if (which->bd_media_basic->med_basic_type) {
+            type = which->bd_media_basic->med_basic_type;
+        } else {
+            type = "";
+        }
+        break;
+    }
+    if (which->bd_media_basic->med_subtype) {
+        sub = which->bd_media_basic->med_subtype;
+    } else {
+        sub = "";
+    }
+    qDebug("Type = %s/%s",type.latin1(),sub.latin1());
+    target_part.setType(type.lower());
+    target_part.setSubtype(sub.lower());
+    fillBodyFields(target_part,which->bd_fields);
+}
+
+void IMAPwrapper::fillBodyFields(RecPart&target_part,mailimap_body_fields*which)
+{
+    if (!which) return;
+    if (which->bd_id) {
+        qDebug("Part ID = %s",which->bd_id);
+        target_part.setIdentifier(which->bd_id);
+    } else {
+        qDebug("ID empty");
+        target_part.setIdentifier("");
+    }
+
+    clistcell*cur;
+    mailimap_single_body_fld_param*param;
+    for (cur = clist_begin(which->bd_parameter->pa_list);cur!=NULL;cur=clist_next(cur)) {
+        param = (mailimap_single_body_fld_param*)cur->data;
+    }
+}
diff --git a/noncore/net/mail/imapwrapper.h b/noncore/net/mail/imapwrapper.h
index 90f3004..f809edb 100644
--- a/noncore/net/mail/imapwrapper.h
+++ b/noncore/net/mail/imapwrapper.h
@@ -8,7 +8,11 @@ struct mailimap;
 struct mailimap_body_type_1part;
 struct mailimap_body_type_text;
+struct mailimap_body_type_basic;
+struct mailimap_body_type_mpart;
+struct mailimap_body_fields;
 struct mailimap_msg_att;
 class RecMail;
 class RecBody;
+class RecPart;
 
 class IMAPwrapper : public QObject
@@ -28,6 +32,15 @@ protected:
     void login();
     void logout();
+    
     void searchBodyText(const RecMail&mail,mailimap_body_type_1part*mailDescription,RecBody&target_body);
-    void fillPlainBody(const RecMail&mail,RecBody&target_body, mailimap_body_type_text * text_body);
+    void searchBodyText(const RecMail&mail,mailimap_body_type_mpart*mailDescription,RecBody&target_body,int current_recursion);
+
+    void fillPlainBody(const RecMail&mail,RecBody&target_body);
+    void fillSinglePart(RecPart&target_part,mailimap_body_type_1part*Description);
+    void fillSingleTextPart(RecPart&target_part,mailimap_body_type_text*which);
+    void fillSingleBasicPart(RecPart&target_part,mailimap_body_type_basic*which);
+    
+    /* just helpers */
+    static void fillBodyFields(RecPart&target_part,mailimap_body_fields*which);
     static QStringList address_list_to_stringlist(clist*list);
 
diff --git a/noncore/net/mail/libmailwrapper/imapwrapper.cpp b/noncore/net/mail/libmailwrapper/imapwrapper.cpp
index e6e9fc9..27f1410 100644
--- a/noncore/net/mail/libmailwrapper/imapwrapper.cpp
+++ b/noncore/net/mail/libmailwrapper/imapwrapper.cpp
@@ -24,9 +24,13 @@ void IMAPwrapper::imap_progress( size_t current, size_t maximum )
 void IMAPwrapper::login()
 {
-    logout();
     const char *server, *user, *pass;
     uint16_t port;
     int err = MAILIMAP_NO_ERROR;
 
+    /* we are connected this moment */
+    /* TODO: setup a timer holding the line or if connection closed - delete the value */
+    if (m_imap) {   
+        return;
+    }
     server = account->getServer().latin1();
     port = account->getPort().toUInt();
@@ -36,6 +40,5 @@ void IMAPwrapper::login()
     m_imap = mailimap_new( 20, &imap_progress );
     /* connect */
-  //  err = mailimap_socket_connect( m_imap, (char*)server, port );
-  if (account->getSSL()) {
+    if (account->getSSL()) {
         err = mailimap_ssl_connect( m_imap, (char*)server, port );
     } else {
@@ -78,5 +81,5 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
     clist *result;
     clistcell *current;
-    mailimap_fetch_att *fetchAtt,*fetchAttFlags,*fetchAttDate;
+//    mailimap_fetch_att *fetchAtt,*fetchAttFlags,*fetchAttDate,*fetchAttSize;
     mailimap_fetch_type *fetchType;
     mailimap_set *set;
@@ -91,5 +94,4 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
     if ( err != MAILIMAP_NO_ERROR ) {
         qDebug("error selecting mailbox: %s",m_imap->imap_response);
-        logout();
         return;
     }
@@ -99,5 +101,4 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
     if (last == 0) {
         qDebug("mailbox has no mails");
-        logout();
         return;
     }
@@ -106,17 +107,12 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
     /* the range has to start at 1!!! not with 0!!!! */
     set = mailimap_set_new_interval( 1, last );
-    fetchAtt = mailimap_fetch_att_new_envelope();
-    fetchAttFlags = mailimap_fetch_att_new_flags();
-    fetchAttDate = mailimap_fetch_att_new_internaldate();
-
-    //fetchType = mailimap_fetch_type_new_fetch_att(fetchAtt);
     fetchType = mailimap_fetch_type_new_fetch_att_list_empty();
-    mailimap_fetch_type_new_fetch_att_list_add(fetchType,fetchAtt);
-    mailimap_fetch_type_new_fetch_att_list_add(fetchType,fetchAttFlags);
-    mailimap_fetch_type_new_fetch_att_list_add(fetchType,fetchAttDate);
+    mailimap_fetch_type_new_fetch_att_list_add(fetchType,mailimap_fetch_att_new_envelope());
+    mailimap_fetch_type_new_fetch_att_list_add(fetchType,mailimap_fetch_att_new_flags());
+    mailimap_fetch_type_new_fetch_att_list_add(fetchType,mailimap_fetch_att_new_internaldate());
+    mailimap_fetch_type_new_fetch_att_list_add(fetchType,mailimap_fetch_att_new_rfc822_size());
 
     err = mailimap_fetch( m_imap, set, fetchType, &result );
     mailimap_set_free( set );
-    /* cleans up the fetch_att's too! */
     mailimap_fetch_type_free( fetchType );
 
@@ -124,8 +120,8 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
 
     if ( err == MAILIMAP_NO_ERROR ) {
-        current = clist_begin(result);
+        
         mailimap_msg_att * msg_att;
         int i = 0;
-        while ( current != 0 ) {
+        for (current = clist_begin(result); current != 0; current=clist_next(current)) {
             ++i;
             msg_att = (mailimap_msg_att*)current->data;
@@ -136,11 +132,9 @@ void IMAPwrapper::listMessages(const QString&mailbox,QList<RecMail> &target )
                 target.append(m);
             }
-            current = current->next;
         }
     } else {
         qDebug("Error fetching headers: %s",m_imap->imap_response);
     }
-    logout();
-    clist_free(result);
+    mailimap_fetch_list_free(result);
 }
 
@@ -220,4 +214,5 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
     mailimap_msg_att_dynamic*flist;
     mailimap_flag_fetch*cflag;
+    int size;
     QBitArray mFlags(7);
     QStringList addresslist;
@@ -226,9 +221,8 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
         return m;
     }
-
-    c = clist_begin(m_att->att_list);
-    while ( c ) {
+    m = new RecMail();
+    for (c = clist_begin(m_att->att_list); c!=NULL;c=clist_next(c) ) {
         current = c;
-        c = c->next;
+        size = 0;
         item = (mailimap_msg_att_item*)current->data;
         if (item->att_type!=MAILIMAP_MSG_ATT_ITEM_STATIC) {
@@ -238,5 +232,5 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
             }
             cf = flist->att_list->first;
-            while (cf) {
+            for (cf = clist_begin(flist->att_list); cf!=NULL; cf = clist_next(cf)) {
                 cflag = (mailimap_flag_fetch*)cf->data;
                 if (cflag->fl_type==MAILIMAP_FLAG_FETCH_OTHER && cflag->fl_flag!=0) {
@@ -267,13 +261,9 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
                     mFlags.setBit(FLAG_RECENT);
                 }
-                cf = cf->next;
             }
             continue;
         }
-        if ( item->att_data.att_static->att_type == MAILIMAP_MSG_ATT_RFC822_HEADER ) {
-            qDebug( "header: \n%s", item->att_data.att_static->att_data.att_rfc822_header );
-        } else if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_ENVELOPE) {
+        if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_ENVELOPE) {
             mailimap_envelope * head = item->att_data.att_static->att_data.att_env;
-            m = new RecMail();
             m->setDate(head->env_date);
             m->setSubject(head->env_subject);
@@ -308,6 +298,6 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
             qDebug("%i %i %i - %i %i %i",d->dt_year,d->dt_month,d->dt_day,d->dt_hour,d->dt_min,d->dt_sec);
             qDebug(da.toString());
-        } else {
-            qDebug("Another type");
+        } else if (item->att_data.att_static->att_type==MAILIMAP_MSG_ATT_RFC822_SIZE) {
+            size = item->att_data.att_static->att_data.att_rfc822_size;
         }
     }
@@ -319,4 +309,5 @@ RecMail*IMAPwrapper::parse_list_result(mailimap_msg_att* m_att)
     if (m) {
         m->setFlags(mFlags);
+        m->setMsgsize(size);
     }
     return m;
@@ -345,11 +336,11 @@ RecBody IMAPwrapper::fetchBody(const RecMail&mail)
     if ( err != MAILIMAP_NO_ERROR ) {
         qDebug("error selecting mailbox: %s",m_imap->imap_response);
-        logout();
         return body;
     }
+ 
     result = clist_new();
     /* the range has to start at 1!!! not with 0!!!! */
     set = mailimap_set_new_interval( mail.getNumber(),mail.getNumber() );
-    fetchAtt = mailimap_fetch_att_new_body();
+    fetchAtt = mailimap_fetch_att_new_bodystructure();
     fetchType = mailimap_fetch_type_new_fetch_att(fetchAtt);
     err = mailimap_fetch( m_imap, set, fetchType, &result );
@@ -364,16 +355,18 @@ RecBody IMAPwrapper::fetchBody(const RecMail&mail)
         if (body_desc->bd_type==MAILIMAP_BODY_1PART) {
             searchBodyText(mail,body_desc->bd_data.bd_body_1part,body);
-        } else {
+        } else if (body_desc->bd_type==MAILIMAP_BODY_MPART) {
+            qDebug("Mulitpart mail");
+            searchBodyText(mail,body_desc->bd_data.bd_body_mpart,body,0);
         }
-
     } else {
         qDebug("error fetching body: %s",m_imap->imap_response);
     }
-
-    clist_free(result);
-    logout();
+    mailimap_fetch_list_free(result);
     return body;
 }
 
+/* this routine is just called when the mail has only ONE part.
+   for filling the parts of a multi-part-message there are other
+   routines 'cause we can not simply fetch the whole body. */
 void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mailDescription,RecBody&target_body)
 {
@@ -381,7 +374,18 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mai
         return;
     }
+    QString sub;
     switch (mailDescription->bd_type) {
+    case MAILIMAP_BODY_TYPE_1PART_MSG:
+        target_body.setType("text");
+        sub = mailDescription->bd_data.bd_type_text->bd_media_text;
+        target_body.setSubtype(sub.lower());
+        fillPlainBody(mail,target_body);
+        break;
     case MAILIMAP_BODY_TYPE_1PART_TEXT:
-            fillPlainBody(mail,target_body,mailDescription->bd_data.bd_type_text);
+        qDebug("Mediatype single: %s",mailDescription->bd_data.bd_type_text->bd_media_text);
+        target_body.setType("text");
+        sub = mailDescription->bd_data.bd_type_text->bd_media_text;
+        target_body.setSubtype(sub.lower());
+        fillPlainBody(mail,target_body);
         break;
     default:
@@ -391,5 +395,5 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mai
 }
 
-void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body, mailimap_body_type_text * bd)
+void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body)
 {
     const char *mb;
@@ -427,8 +431,11 @@ void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body, mailimap
                 if (msg_att_item->att_data.att_static->att_type == MAILIMAP_MSG_ATT_BODY_SECTION) {
                     char*text = msg_att_item->att_data.att_static->att_data.att_body_section->sec_body_part;
-                    int length = msg_att_item->att_data.att_static->att_data.att_body_section->sec_length;
                     msg_att_item->att_data.att_static->att_data.att_body_section->sec_body_part = 0L;
-                    body = QString(text);
-                    free(text);
+                    if (text) {
+                        body = QString(text);
+                        free(text);
+                    } else {
+                        body = "";
+                    }
                 }
             }
@@ -438,5 +445,4 @@ void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body, mailimap
         qDebug("error fetching text: %s",m_imap->imap_response);
     }
-    //clist_free(result);
     mailimap_fetch_list_free(result);
     target_body.setBodytext(body);
@@ -454,11 +460,9 @@ QStringList IMAPwrapper::address_list_to_stringlist(clist*list)
         return l;
     }
-    current = clist_begin(list);
     unsigned int count = 0;
-    while (current!= NULL) {
+    for (current=clist_begin(list);current!= NULL;current=clist_next(current)) {
         from = "";
         named_from = false;
         current_address=(mailimap_address*)current->data;
-        current = current->next;
         if (current_address->ad_personal_name){
             from+=QString(current_address->ad_personal_name);
@@ -486,2 +490,116 @@ QStringList IMAPwrapper::address_list_to_stringlist(clist*list)
     return l;
 }
+
+void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_mpart*mailDescription,RecBody&target_body,int current_recursion)
+{
+    /* current_recursion is for avoiding ugly mails which has a to deep body-structure */
+    if (!mailDescription||current_recursion==2) {
+        return;
+    }
+    qDebug("Mediatype: %s",mailDescription->bd_media_subtype);
+    clistcell*current;
+    mailimap_body*current_body;
+    for (current=clist_begin(mailDescription->bd_list);current!=0;current=clist_next(current)) {
+        current_body = (mailimap_body*)current->data;
+        if (current_body->bd_type==MAILIMAP_BODY_MPART) {
+            searchBodyText(mail,current_body->bd_data.bd_body_mpart,target_body,current_recursion+1);
+        } else if (current_body->bd_type==MAILIMAP_BODY_1PART){
+            RecPart currentPart;
+            fillSinglePart(currentPart,current_body->bd_data.bd_body_1part);
+            target_body.addPart(currentPart);
+        }
+    }
+    if (current_recursion==0) {
+        
+    }
+}
+
+void IMAPwrapper::fillSinglePart(RecPart&target_part,mailimap_body_type_1part*Description)
+{
+    if (!Description) {
+        return;
+    }
+    switch (Description->bd_type) {
+        case MAILIMAP_BODY_TYPE_1PART_TEXT:
+            target_part.setType("text");
+            fillSingleTextPart(target_part,Description->bd_data.bd_type_text);
+            break;
+        case MAILIMAP_BODY_TYPE_1PART_BASIC:
+            fillSingleBasicPart(target_part,Description->bd_data.bd_type_basic);
+            break;
+        default:
+            break;
+    }
+}
+
+void IMAPwrapper::fillSingleTextPart(RecPart&target_part,mailimap_body_type_text*which)
+{
+    if (!which) {
+        return;
+    }
+    QString sub;
+    sub = which->bd_media_text;
+    target_part.setSubtype(sub.lower());
+    target_part.setLines(which->bd_lines);
+    fillBodyFields(target_part,which->bd_fields);
+}
+
+void IMAPwrapper::fillSingleBasicPart(RecPart&target_part,mailimap_body_type_basic*which)
+{
+    if (!which) {
+        return;
+    }
+    QString type,sub;
+    switch (which->bd_media_basic->med_type) {
+    case MAILIMAP_MEDIA_BASIC_APPLICATION:
+        type = "application";
+        break;
+    case MAILIMAP_MEDIA_BASIC_AUDIO:
+        type = "audio";
+        break;
+    case MAILIMAP_MEDIA_BASIC_IMAGE:
+        type = "image";
+        break;
+    case MAILIMAP_MEDIA_BASIC_MESSAGE:
+        type = "message";
+        break;
+    case MAILIMAP_MEDIA_BASIC_VIDEO:
+        type = "video";
+        break;
+    case MAILIMAP_MEDIA_BASIC_OTHER:
+    default:
+        if (which->bd_media_basic->med_basic_type) {
+            type = which->bd_media_basic->med_basic_type;
+        } else {
+            type = "";
+        }
+        break;
+    }
+    if (which->bd_media_basic->med_subtype) {
+        sub = which->bd_media_basic->med_subtype;
+    } else {
+        sub = "";
+    }
+    qDebug("Type = %s/%s",type.latin1(),sub.latin1());
+    target_part.setType(type.lower());
+    target_part.setSubtype(sub.lower());
+    fillBodyFields(target_part,which->bd_fields);
+}
+
+void IMAPwrapper::fillBodyFields(RecPart&target_part,mailimap_body_fields*which)
+{
+    if (!which) return;
+    if (which->bd_id) {
+        qDebug("Part ID = %s",which->bd_id);
+        target_part.setIdentifier(which->bd_id);
+    } else {
+        qDebug("ID empty");
+        target_part.setIdentifier("");
+    }
+
+    clistcell*cur;
+    mailimap_single_body_fld_param*param;
+    for (cur = clist_begin(which->bd_parameter->pa_list);cur!=NULL;cur=clist_next(cur)) {
+        param = (mailimap_single_body_fld_param*)cur->data;
+    }
+}
diff --git a/noncore/net/mail/libmailwrapper/imapwrapper.h b/noncore/net/mail/libmailwrapper/imapwrapper.h
index 90f3004..f809edb 100644
--- a/noncore/net/mail/libmailwrapper/imapwrapper.h
+++ b/noncore/net/mail/libmailwrapper/imapwrapper.h
@@ -8,7 +8,11 @@ struct mailimap;
 struct mailimap_body_type_1part;
 struct mailimap_body_type_text;
+struct mailimap_body_type_basic;
+struct mailimap_body_type_mpart;
+struct mailimap_body_fields;
 struct mailimap_msg_att;
 class RecMail;
 class RecBody;
+class RecPart;
 
 class IMAPwrapper : public QObject
@@ -28,6 +32,15 @@ protected:
     void login();
     void logout();
+    
     void searchBodyText(const RecMail&mail,mailimap_body_type_1part*mailDescription,RecBody&target_body);
-    void fillPlainBody(const RecMail&mail,RecBody&target_body, mailimap_body_type_text * text_body);
+    void searchBodyText(const RecMail&mail,mailimap_body_type_mpart*mailDescription,RecBody&target_body,int current_recursion);
+
+    void fillPlainBody(const RecMail&mail,RecBody&target_body);
+    void fillSinglePart(RecPart&target_part,mailimap_body_type_1part*Description);
+    void fillSingleTextPart(RecPart&target_part,mailimap_body_type_text*which);
+    void fillSingleBasicPart(RecPart&target_part,mailimap_body_type_basic*which);
+    
+    /* just helpers */
+    static void fillBodyFields(RecPart&target_part,mailimap_body_fields*which);
     static QStringList address_list_to_stringlist(clist*list);
 
diff --git a/noncore/net/mail/libmailwrapper/mailtypes.cpp b/noncore/net/mail/libmailwrapper/mailtypes.cpp
index f9e5794..0e3174d 100644
--- a/noncore/net/mail/libmailwrapper/mailtypes.cpp
+++ b/noncore/net/mail/libmailwrapper/mailtypes.cpp
@@ -3,5 +3,5 @@
 
 RecMail::RecMail()
-    :subject(""),date(""),from(""),mbox(""),msg_id(""),msg_number(0),msg_flags(7)
+    :subject(""),date(""),from(""),mbox(""),msg_id(""),msg_number(0),msg_size(0),msg_flags(7)
 {
     init();
@@ -22,4 +22,5 @@ void RecMail::copy_old(const RecMail&old)
     mbox = old.mbox;
     msg_id = old.msg_id;
+    msg_size = old.msg_size;
     msg_number = old.msg_number;
     from = old.from;
@@ -68,10 +69,10 @@ const QStringList& RecMail::Bcc()const
 
 RecPart::RecPart()
-    : m_type(""),m_subtype(""),m_identifier(""),m_encoding("")
+    : m_type(""),m_subtype(""),m_identifier(""),m_encoding(""),m_lines(0)
 {
 }
 
-RecPart::RecPart(const QString&identifier,const QString&type,const QString&subtype,const QString&encoding)
-    : m_type(type),m_subtype(subtype),m_identifier(identifier),m_encoding(encoding)
+RecPart::RecPart(const QString&identifier,const QString&type,const QString&subtype,const QString&encoding,unsigned int lines)
+    : m_type(type),m_subtype(subtype),m_identifier(identifier),m_encoding(encoding),m_lines(lines)
 {
 }
@@ -81,4 +82,14 @@ RecPart::~RecPart()
 }
 
+void RecPart::setLines(unsigned int lines)
+{
+    m_lines = lines;
+}
+    
+const unsigned int RecPart::Lines()const
+{
+    return m_lines;
+}
+
 const QString& RecPart::Type()const
 {
@@ -157,2 +168,22 @@ void RecBody::addPart(const RecPart& part)
     m_PartsList.append(p);
 }
+
+void RecBody::setType(const QString&type)
+{
+    m_type = type;
+}
+
+const QString& RecBody::Type()const
+{
+    return m_type;
+}
+
+void RecBody::setSubtype(const QString&type)
+{
+    m_subtype = type;
+}
+
+const QString& RecBody::Subtype()const
+{
+    return m_subtype;
+}
diff --git a/noncore/net/mail/libmailwrapper/mailtypes.h b/noncore/net/mail/libmailwrapper/mailtypes.h
index 6d6b080..900f10a 100644
--- a/noncore/net/mail/libmailwrapper/mailtypes.h
+++ b/noncore/net/mail/libmailwrapper/mailtypes.h
@@ -47,4 +47,6 @@ public:
     void setReplyto(const QString&reply){replyto=reply;}
     const QString&Replyto()const{return replyto;}
+    void setMsgsize(int size){msg_size = size;}
+    const int Msgsize()const{return msg_size;}
 
 
@@ -60,5 +62,5 @@ public:
 protected:
     QString subject,date,from,mbox,msg_id,replyto;
-    int msg_number;
+    int msg_number,msg_size;
     QBitArray msg_flags;
     QStringList to,cc,bcc;
@@ -71,7 +73,8 @@ class RecPart
 protected:
     QString m_type,m_subtype,m_identifier,m_encoding;
+    unsigned int m_lines;
 public:
     RecPart();
-    RecPart(const QString&identifier,const QString&type="",const QString&subtype="",const QString&encoding="BASE64");
+    RecPart(const QString&identifier,const QString&type="",const QString&subtype="",const QString&encoding="BASE64",unsigned int lines=0);
     virtual ~RecPart();
     
@@ -84,4 +87,6 @@ public:
     const QString&Encoding()const;
     void setEncoding(const QString&encoding);
+    void setLines(unsigned int lines);
+    const unsigned int Lines()const;
 };
 
@@ -89,5 +94,5 @@ class RecBody
 {
 protected:
-    QString m_BodyText;
+    QString m_BodyText,m_type,m_subtype;
     QList<RecPart> m_PartsList;
 
@@ -97,4 +102,9 @@ public:
     void setBodytext(const QString&);
     const QString& Bodytext()const;
+    void setType(const QString&);
+    const QString&Type()const;
+    void setSubtype(const QString&);
+    const QString&Subtype()const;
+
     
     void setParts(const QList<RecPart>&parts);
diff --git a/noncore/net/mail/mailtypes.cpp b/noncore/net/mail/mailtypes.cpp
index f9e5794..0e3174d 100644
--- a/noncore/net/mail/mailtypes.cpp
+++ b/noncore/net/mail/mailtypes.cpp
@@ -3,5 +3,5 @@
 
 RecMail::RecMail()
-    :subject(""),date(""),from(""),mbox(""),msg_id(""),msg_number(0),msg_flags(7)
+    :subject(""),date(""),from(""),mbox(""),msg_id(""),msg_number(0),msg_size(0),msg_flags(7)
 {
     init();
@@ -22,4 +22,5 @@ void RecMail::copy_old(const RecMail&old)
     mbox = old.mbox;
     msg_id = old.msg_id;
+    msg_size = old.msg_size;
     msg_number = old.msg_number;
     from = old.from;
@@ -68,10 +69,10 @@ const QStringList& RecMail::Bcc()const
 
 RecPart::RecPart()
-    : m_type(""),m_subtype(""),m_identifier(""),m_encoding("")
+    : m_type(""),m_subtype(""),m_identifier(""),m_encoding(""),m_lines(0)
 {
 }
 
-RecPart::RecPart(const QString&identifier,const QString&type,const QString&subtype,const QString&encoding)
-    : m_type(type),m_subtype(subtype),m_identifier(identifier),m_encoding(encoding)
+RecPart::RecPart(const QString&identifier,const QString&type,const QString&subtype,const QString&encoding,unsigned int lines)
+    : m_type(type),m_subtype(subtype),m_identifier(identifier),m_encoding(encoding),m_lines(lines)
 {
 }
@@ -81,4 +82,14 @@ RecPart::~RecPart()
 }
 
+void RecPart::setLines(unsigned int lines)
+{
+    m_lines = lines;
+}
+    
+const unsigned int RecPart::Lines()const
+{
+    return m_lines;
+}
+
 const QString& RecPart::Type()const
 {
@@ -157,2 +168,22 @@ void RecBody::addPart(const RecPart& part)
     m_PartsList.append(p);
 }
+
+void RecBody::setType(const QString&type)
+{
+    m_type = type;
+}
+
+const QString& RecBody::Type()const
+{
+    return m_type;
+}
+
+void RecBody::setSubtype(const QString&type)
+{
+    m_subtype = type;
+}
+
+const QString& RecBody::Subtype()const
+{
+    return m_subtype;
+}
diff --git a/noncore/net/mail/mailtypes.h b/noncore/net/mail/mailtypes.h
index 6d6b080..900f10a 100644
--- a/noncore/net/mail/mailtypes.h
+++ b/noncore/net/mail/mailtypes.h
@@ -47,4 +47,6 @@ public:
     void setReplyto(const QString&reply){replyto=reply;}
     const QString&Replyto()const{return replyto;}
+    void setMsgsize(int size){msg_size = size;}
+    const int Msgsize()const{return msg_size;}
 
 
@@ -60,5 +62,5 @@ public:
 protected:
     QString subject,date,from,mbox,msg_id,replyto;
-    int msg_number;
+    int msg_number,msg_size;
     QBitArray msg_flags;
     QStringList to,cc,bcc;
@@ -71,7 +73,8 @@ class RecPart
 protected:
     QString m_type,m_subtype,m_identifier,m_encoding;
+    unsigned int m_lines;
 public:
     RecPart();
-    RecPart(const QString&identifier,const QString&type="",const QString&subtype="",const QString&encoding="BASE64");
+    RecPart(const QString&identifier,const QString&type="",const QString&subtype="",const QString&encoding="BASE64",unsigned int lines=0);
     virtual ~RecPart();
     
@@ -84,4 +87,6 @@ public:
     const QString&Encoding()const;
     void setEncoding(const QString&encoding);
+    void setLines(unsigned int lines);
+    const unsigned int Lines()const;
 };
 
@@ -89,5 +94,5 @@ class RecBody
 {
 protected:
-    QString m_BodyText;
+    QString m_BodyText,m_type,m_subtype;
     QList<RecPart> m_PartsList;
 
@@ -97,4 +102,9 @@ public:
     void setBodytext(const QString&);
     const QString& Bodytext()const;
+    void setType(const QString&);
+    const QString&Type()const;
+    void setSubtype(const QString&);
+    const QString&Subtype()const;
+
     
     void setParts(const QList<RecPart>&parts);