author | harlekin <harlekin> | 2003-12-12 13:50:54 (UTC) |
---|---|---|
committer | harlekin <harlekin> | 2003-12-12 13:50:54 (UTC) |
commit | c7549de5f69c98c6b15b6e1a6f4d73ac3f04669e (patch) (side-by-side diff) | |
tree | b836cf4336a2e8b13636298bb12d4d794cdb3bff | |
parent | 77299749e119436aab7af82e065357fca0cead96 (diff) | |
download | opie-c7549de5f69c98c6b15b6e1a6f4d73ac3f04669e.zip opie-c7549de5f69c98c6b15b6e1a6f4d73ac3f04669e.tar.gz opie-c7549de5f69c98c6b15b6e1a6f4d73ac3f04669e.tar.bz2 |
code cleanups and output beautification
-rw-r--r-- | noncore/net/mail/mainwindow.cpp | 5 | ||||
-rw-r--r-- | noncore/net/mail/viewmail.cpp | 95 | ||||
-rw-r--r-- | noncore/net/mail/viewmail.h | 27 |
3 files changed, 43 insertions, 84 deletions
diff --git a/noncore/net/mail/mainwindow.cpp b/noncore/net/mail/mainwindow.cpp index 6d13a8b..1d6e901 100644 --- a/noncore/net/mail/mainwindow.cpp +++ b/noncore/net/mail/mainwindow.cpp @@ -153,3 +153,2 @@ void MainWindow::displayMail(QListViewItem*item) if (!item) return; - qDebug("View mail"); RecMail mail = ((MailListViewItem*)item)->data(); @@ -158,4 +157,4 @@ void MainWindow::displayMail(QListViewItem*item) ViewMail readMail( this ); - - readMail.setMailInfo( mail.getFrom(), mail.To(), mail.getSubject(), mail.CC(), mail.Bcc(), mail.getDate(), body.Bodytext(), mail.Msgid() ); + readMail.setBody( body ); + readMail.setMail( mail ); readMail.showMaximized(); diff --git a/noncore/net/mail/viewmail.cpp b/noncore/net/mail/viewmail.cpp index 7969235..7b8494d 100644 --- a/noncore/net/mail/viewmail.cpp +++ b/noncore/net/mail/viewmail.cpp @@ -25,12 +25,19 @@ AttachItem::AttachItem(QListViewItem *parent, AttachItemStore &attachItemStore) -void ViewMail::setMailInfo( const QString & from, const QStringList & to, const QString & subject, const QStringList & cc, const QStringList & bcc, const QString & date, const QString & bodytext, const QString & messageID ) { -m_mail[0] = from; -m_mail2[0] = to; -m_mail[1] = subject; -m_mail2[1] = cc; -m_mail2[2] = bcc; -m_mail[2] = bodytext; -m_mail[3] = date; -m_mail[4] = messageID; +void ViewMail::setBody( RecBody body ) { + +m_mail[2] = body.Bodytext(); + +} + +void ViewMail::setMail( RecMail mail ) { + +m_mail[0] = mail.getFrom(); +m_mail[1] = mail.getSubject(); +m_mail[3] = mail.getDate(); +m_mail[4] = mail.Msgid(); + +m_mail2[0] = mail.To(); +m_mail2[1] = mail.CC(); +m_mail2[2] = mail.Bcc(); @@ -40,2 +47,3 @@ setText(); + ViewMail::ViewMail( QWidget *parent, const char *name, WFlags fl) @@ -43,3 +51,3 @@ ViewMail::ViewMail( QWidget *parent, const char *name, WFlags fl) { - _gotBody = false; + m_gotBody = false; @@ -48,6 +56,3 @@ ViewMail::ViewMail( QWidget *parent, const char *name, WFlags fl) - attachments->setEnabled(_gotBody); - -// _handler->iUid("FETCH", QString("%1 (BODY[1])").arg(mail.uid())); -// connect(_handler, SIGNAL(gotResponse(IMAPResponse &)), SLOT(slotIMAPUid(IMAPResponse &))); + attachments->setEnabled(m_gotBody); } @@ -73,18 +78,15 @@ void ViewMail::setText() - _mailHtml = tr( - "<html><body>" - "<div align=center><b><font color=#0000FF>%1</b></font></div>" - "<b>From:</b><font color=#6C86C0> %2</font><br>" - "<b>To:</b><font color=#6C86C0> %3</font><br>" - "%4" - "<b>Date:</b> %5<hr>" - "<font face=fixed>") - .arg( deHtml( m_mail[1] ) ) - .arg( deHtml( m_mail[0] ) ) - .arg( deHtml( toString ) ) - .arg( tr("<b>Cc:</b> %1<br>").arg( deHtml( ccString ) ) ) - .arg( m_mail[3] ); - browser->setText( QString(_mailHtml) + deHtml( m_mail[2] ) + "</font>" ); + m_mailHtml = "<html><body>" + "<table width=\"100%\" border=\"0\"><tr bgcolor=\"#FFDD76\"><td>" + "<div align=left><b>" + deHtml( m_mail[1] ) + "</b></div>" + "</td></tr><tr bgcolor=\"#EEEEE6\"><td>" + "<b>" + tr( "From" ) + ": </b><font color=#6C86C0>" + deHtml( m_mail[0] ) + "</font><br>" + "<b>" + tr( "To" ) + ": </b><font color=#6C86C0>" + deHtml( toString ) + "</font><br><b>" + + tr( "Cc" ) + ": </b>" + deHtml( ccString ) + "<br>" + "<b>" + tr( "Date" ) + ": </b> " + m_mail[3] + + "</td></tr></table><font face=fixed>"; + + browser->setText( QString( m_mailHtml) + deHtml( m_mail[2] ) + "</font></html>" ); // remove later in favor of a real handling - _gotBody = true; + m_gotBody = true; } @@ -92,3 +94,2 @@ void ViewMail::setText() - ViewMail::~ViewMail() @@ -130,3 +131,3 @@ void ViewMail::slotReply() { - if (!_gotBody) { + if (!m_gotBody) { QMessageBox::information(this, tr("Error"), tr("<p>The mail body is not yet downloaded, so you cannot reply yet."), tr("Ok")); @@ -160,3 +161,2 @@ void ViewMail::slotReply() - qDebug ( rtext ); } @@ -165,3 +165,3 @@ void ViewMail::slotForward() { - if (!_gotBody) { + if (!m_gotBody) { QMessageBox::information(this, tr("Error"), tr("<p>The mail body is not yet downloaded, so you cannot forward yet."), tr("Ok")); @@ -179,8 +179,2 @@ void ViewMail::slotForward() .arg( m_mail[0] ); - //if (!_mail.envelope().to().toString().isNull()) - // ftext += QString("To: %1\n") - // .arg(_mail.envelope().to().toString()); - //if (!_mail.envelope().cc().toString().isNull()) - // ftext += QString("Cc: %1\n") - // .arg(_mail.envelope().cc().toString()); if (!m_mail[1].isNull()) @@ -194,5 +188,2 @@ void ViewMail::slotForward() - qDebug( ftext ); - - Settings *settings = new Settings(); @@ -205,21 +196 @@ void ViewMail::slotForward() -/* -void ViewMail::slotIMAPUid(IMAPResponse &response) -{ - disconnect(_handler, SIGNAL(gotResponse(IMAPResponse &)), this, SLOT(slotIMAPUid(IMAPResponse &))); - - if (response.statusResponse().status() == IMAPResponseEnums::OK) { - QValueList<IMAPResponseBodyPart> bodyParts; - bodyParts.append(response.FETCH()[0].bodyPart(0)); - _mail.setBodyParts(bodyParts); - - browser->setText(QString(_mailHtml).arg(deHtml(response.FETCH()[0].bodyPart(0).data()))); - -// fillList(response.FETCH()[0].bodyStructure()); - - _gotBody = true; - } else { - QMessageBox::warning(this, tr("Error"), tr("<p>I was unable to retrieve the mail from the server. You can try again later or give up.</p>"), tr("Ok")); - } -} -*/ diff --git a/noncore/net/mail/viewmail.h b/noncore/net/mail/viewmail.h index 258c09b..61baa1d 100644 --- a/noncore/net/mail/viewmail.h +++ b/noncore/net/mail/viewmail.h @@ -8,4 +8,3 @@ #include "viewmailbase.h" -//#include "imapresponse.h" -//#include "mailtable.h" +#include "mailtypes.h" @@ -51,7 +50,6 @@ public: void exec(); - static QString appName() { return QString::fromLatin1("mail"); } - void setMailInfo( const QString & from, const QStringList & to, const QString & subject, const QStringList & cc, const QStringList & bcc,const QString & date, const QString & bodytext, const QString & messageID ); + void setMail( RecMail mail ); + void setBody( RecBody body ); protected: -// void fillList(IMAPResponseBODYSTRUCTURE &structure); QString deHtml(const QString &string); @@ -63,19 +61,10 @@ protected slots: -// void slotIMAPUid(IMAPResponse &response); - private: bool _inLoop; -// IMAPResponseFETCH _mail; -// IMAPHandler *_handler; - QString _mailHtml; - bool _gotBody; - - // 0 from - // 1 subject - // 2 bodytext - // 3 date + QString m_mailHtml; + bool m_gotBody; + + // 0 from 1 subject 2 bodytext 3 date QMap <int,QString> m_mail; - // 0 to - // 1 cc - // 2 bcc + // 0 to 1 cc 2 bcc QMap <int,QStringList> m_mail2; |