fetching mail body and attachments is working.

4 files changed, 141 insertions, 95 deletions

diff --git a/noncore/net/mail/libmailwrapper/imapwrapper.cpp b/noncore/net/mail/libmailwrapper/imapwrapper.cpp
index fa967da..da5974c 100644
--- a/noncore/net/mail/libmailwrapper/imapwrapper.cpp
+++ b/noncore/net/mail/libmailwrapper/imapwrapper.cpp
@@ -375,9 +375,12 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mai
     }
-    QString sub;
+    QString sub,body_text;
+    RecPart singlePart;
+    QValueList<int> path;
+    fillSinglePart(singlePart,mailDescription);
     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);
+        path.append(1);
+        body_text = fetchPart(mail,path,true);
+        target_body.setBodytext(body_text);
+        target_body.setDescription(singlePart);
         break;
@@ -385,6 +388,11 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mai
         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);
+        path.append(1);
+        body_text = fetchPart(mail,path,true);
+        target_body.setBodytext(body_text);
+        target_body.setDescription(singlePart);
+        break;
+    case MAILIMAP_BODY_TYPE_1PART_BASIC:
+        qDebug("Single attachment");
+        target_body.setBodytext("");
+        target_body.addPart(singlePart);
         break;
@@ -393,57 +401,3 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_1part*mai
     }
-    return;
-}
-
-void IMAPwrapper::fillPlainBody(const RecMail&mail,RecBody&target_body)
-{
-    const char *mb;
-    QString body="";
-    int err = MAILIMAP_NO_ERROR;
-    clist *result;
-    clistcell *current,*cur;
-    mailimap_fetch_att *fetchAtt;
-    mailimap_fetch_type *fetchType;
-    mailimap_set *set;
-
-    mb = mail.getMbox().latin1();
-
-    if (!m_imap) {
-        return;
-    }
-
-    result = clist_new();
-    set =  set = mailimap_set_new_single(mail.getNumber());
-    mailimap_section * section = mailimap_section_new_text();
-    fetchAtt = mailimap_fetch_att_new_body_peek_section(section);
-    fetchType = mailimap_fetch_type_new_fetch_att(fetchAtt);
-    err = mailimap_fetch( m_imap, set, fetchType, &result );
-    mailimap_set_free( set );
-    mailimap_fetch_type_free( fetchType );
-
     
-    if (err == MAILIMAP_NO_ERROR && (current=clist_begin(result)) ) {
-        mailimap_msg_att * msg_att;
-        msg_att = (mailimap_msg_att*)current->data;
-        mailimap_msg_att_item*msg_att_item;
-        for(cur = clist_begin(msg_att->att_list) ; cur != NULL ; cur = clist_next(cur)) {
-            msg_att_item = (mailimap_msg_att_item*)clist_content(cur);
-            if (msg_att_item->att_type == MAILIMAP_MSG_ATT_ITEM_STATIC) {
-                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;
-                    msg_att_item->att_data.att_static->att_data.att_body_section->sec_body_part = 0L;
-                    if (text) {
-                        body = QString(text);
-                        free(text);
-                    } else {
-                        body = "";
-                    }
-                }
-            }
-        }
-         
-    } else {
-        qDebug("error fetching text: %s",m_imap->imap_response);
-    }
-    mailimap_fetch_list_free(result);
-    target_body.setBodytext(body);
     return;
@@ -492,3 +446,3 @@ QStringList IMAPwrapper::address_list_to_stringlist(clist*list)
 
-QString IMAPwrapper::fetchPart(const RecMail&mail,QValueList<int>&path,bool internal_call)
+QString IMAPwrapper::fetchPart(const RecMail&mail,const QValueList<int>&path,bool internal_call)
 {
@@ -586,5 +540,4 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_mpart*mai
                 QString body_text = fetchPart(mail,clist,true);
+                target_body.setDescription(currentPart);
                 target_body.setBodytext(body_text);
-                target_body.setType(currentPart.Type());
-                target_body.setSubtype(currentPart.Subtype());
             } else {
@@ -592,3 +545,3 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_mpart*mai
                 for (unsigned int j = 0; j < clist.count();++j) {
-                    id+=(j>0?".":"");
+                    id+=(j>0?" ":"");
                     id+=QString("%1").arg(clist[j]);
@@ -597,2 +550,3 @@ void IMAPwrapper::searchBodyText(const RecMail&mail,mailimap_body_type_mpart*mai
                 currentPart.setIdentifier(id);
+                currentPart.setPositionlist(clist);
                 target_body.addPart(currentPart);
@@ -616,2 +570,5 @@ void IMAPwrapper::fillSinglePart(RecPart&target_part,mailimap_body_type_1part*De
             break;
+        case MAILIMAP_BODY_TYPE_1PART_MSG:
+            fillSingleMsgPart(target_part,Description->bd_data.bd_type_msg);
+            break;
         default:
@@ -633,2 +590,18 @@ void IMAPwrapper::fillSingleTextPart(RecPart&target_part,mailimap_body_type_text
 
+void IMAPwrapper::fillSingleMsgPart(RecPart&target_part,mailimap_body_type_msg*which)
+{
+    if (!which) {
+        return;
+    }
+//    QString sub;
+//    sub = which->bd_media_text;
+//    target_part.setSubtype(sub.lower());
+    qDebug("Message part");
+    /* we set this type to text/plain */
+    target_part.setType("text");
+    target_part.setSubtype("plain");
+    target_part.setLines(which->bd_lines);
+    fillBodyFields(target_part,which->bd_fields);    
+}
+
 void IMAPwrapper::fillSingleBasicPart(RecPart&target_part,mailimap_body_type_basic*which)
@@ -682,3 +655,39 @@ void IMAPwrapper::fillBodyFields(RecPart&target_part,mailimap_body_fields*which)
         param = (mailimap_single_body_fld_param*)cur->data;
+        if (param) {
+            target_part.addParameter(QString(param->pa_name).lower(),QString(param->pa_value));
+        }
     }
+    mailimap_body_fld_enc*enc = which->bd_encoding;
+    QString encoding("");
+    switch (enc->enc_type) {
+    case MAILIMAP_BODY_FLD_ENC_7BIT:
+        encoding = "7bit";
+        break;
+    case MAILIMAP_BODY_FLD_ENC_8BIT:
+        encoding = "8bit";
+        break;
+    case MAILIMAP_BODY_FLD_ENC_BINARY:
+        encoding="binary";
+        break;
+    case MAILIMAP_BODY_FLD_ENC_BASE64:
+        encoding="base64";
+        break;
+    case MAILIMAP_BODY_FLD_ENC_QUOTED_PRINTABLE:
+        encoding="quoted-printable";
+        break;
+    case MAILIMAP_BODY_FLD_ENC_OTHER:
+    default:
+        if (enc->enc_value) {
+            char*t=enc->enc_value;
+            encoding=QString(enc->enc_value);
+            enc->enc_value=0L;
+            free(t);
+        }
+    }
+    target_part.setEncoding(encoding);
+}
+
+QString IMAPwrapper::fetchPart(const RecMail&mail,const RecPart&part)
+{
+    return fetchPart(mail,part.Positionlist(),false);
 }
diff --git a/noncore/net/mail/libmailwrapper/imapwrapper.h b/noncore/net/mail/libmailwrapper/imapwrapper.h
index 5ea45f3..95de215 100644
--- a/noncore/net/mail/libmailwrapper/imapwrapper.h
+++ b/noncore/net/mail/libmailwrapper/imapwrapper.h
@@ -10,2 +10,3 @@ struct mailimap_body_type_text;
 struct mailimap_body_type_basic;
+struct mailimap_body_type_msg;
 struct mailimap_body_type_mpart;
@@ -27,3 +28,4 @@ public:
     RecBody fetchBody(const RecMail&mail);
-    QString fetchPart(const RecMail&mail,QValueList<int>&path,bool internal_call=false);
+    QString fetchPart(const RecMail&mail,const QValueList<int>&path,bool internal_call=false);
+    QString fetchPart(const RecMail&mail,const RecPart&part);
     static void imap_progress( size_t current, size_t maximum );
@@ -38,3 +40,2 @@ protected:
 
-    void fillPlainBody(const RecMail&mail,RecBody&target_body);
     void fillSinglePart(RecPart&target_part,mailimap_body_type_1part*Description);
@@ -42,2 +43,3 @@ protected:
     void fillSingleBasicPart(RecPart&target_part,mailimap_body_type_basic*which);
+    void fillSingleMsgPart(RecPart&target_part,mailimap_body_type_msg*which);
     
diff --git a/noncore/net/mail/libmailwrapper/mailtypes.cpp b/noncore/net/mail/libmailwrapper/mailtypes.cpp
index 6a66113..f56bb63 100644
--- a/noncore/net/mail/libmailwrapper/mailtypes.cpp
+++ b/noncore/net/mail/libmailwrapper/mailtypes.cpp
@@ -72,7 +72,4 @@ RecPart::RecPart()
 {
-}
-
-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)
-{
+    m_Parameters.clear();
+    m_poslist.clear();
 }
@@ -133,4 +130,39 @@ void RecPart::setEncoding(const QString&encoding)
 
+void RecPart::setParameters(const part_plist_t&list)
+{
+    m_Parameters = list;
+}
+
+const part_plist_t& RecPart::Parameters()const
+{
+    return m_Parameters;
+}
+
+void RecPart::addParameter(const QString&key,const QString&value)
+{
+    m_Parameters[key]=value;
+}
+
+const QString RecPart::searchParamter(const QString&key)const
+{
+    QString value("");
+    part_plist_t::ConstIterator it = m_Parameters.find(key);
+    if (it != m_Parameters.end()) {
+        value = it.data();
+    }
+    return value;
+}
+
+void RecPart::setPositionlist(const QValueList<int>&poslist)
+{
+    m_poslist = poslist;
+}
+
+const QValueList<int>& RecPart::Positionlist()const
+{
+    return m_poslist;
+}
+
 RecBody::RecBody()
-    : m_BodyText(),m_PartsList()
+    : m_BodyText(),m_PartsList(),m_description()
 {
@@ -170,20 +202,10 @@ void RecBody::addPart(const RecPart& part)
 
-void RecBody::setType(const QString&type)
+void RecBody::setDescription(const RecPart&des)
 {
-    m_type = type;
+    m_description = des;
 }
 
-const QString& RecBody::Type()const
+const RecPart& RecBody::Description()const
 {
-    return m_type;
-}
-
-void RecBody::setSubtype(const QString&type)
-{
-    m_subtype = type;
-}
-
-const QString& RecBody::Subtype()const
-{
-    return m_subtype;
+    return m_description;
 }
diff --git a/noncore/net/mail/libmailwrapper/mailtypes.h b/noncore/net/mail/libmailwrapper/mailtypes.h
index 900f10a..60db527 100644
--- a/noncore/net/mail/libmailwrapper/mailtypes.h
+++ b/noncore/net/mail/libmailwrapper/mailtypes.h
@@ -14,2 +14,4 @@
 #include <qstringlist.h>
+#include <qmap.h>
+#include <qvaluelist.h>
 
@@ -70,2 +72,4 @@ protected:
 
+typedef QMap<QString,QString> part_plist_t;
+
 class RecPart
@@ -75,5 +79,8 @@ protected:
     unsigned int m_lines;
+    part_plist_t m_Parameters;
+    /* describes the position in the mail */
+    QValueList<int> m_poslist;
+
 public:
     RecPart();
-    RecPart(const QString&identifier,const QString&type="",const QString&subtype="",const QString&encoding="BASE64",unsigned int lines=0);
     virtual ~RecPart();
@@ -90,2 +97,9 @@ public:
     const unsigned int Lines()const;
+    
+    void setParameters(const part_plist_t&list);
+    const part_plist_t&Parameters()const;
+    void addParameter(const QString&key,const QString&value);
+    const QString searchParamter(const QString&key)const;
+    void setPositionlist(const QValueList<int>&poslist);
+    const QValueList<int>& Positionlist()const;
 };
@@ -95,4 +109,5 @@ class RecBody
 protected:
-    QString m_BodyText,m_type,m_subtype;
+    QString m_BodyText;
     QList<RecPart> m_PartsList;
+    RecPart m_description;
 
@@ -103,7 +118,5 @@ public:
     const QString& Bodytext()const;
-    void setType(const QString&);
-    const QString&Type()const;
-    void setSubtype(const QString&);
-    const QString&Subtype()const;
 
+    void setDescription(const RecPart&des);
+    const RecPart& Description()const;