Fixed a crash

1 files changed, 20 insertions, 8 deletions

diff --git a/kmicromail/composemail.cpp b/kmicromail/composemail.cpp
index c1b58a4..9060364 100644
--- a/kmicromail/composemail.cpp
+++ b/kmicromail/composemail.cpp
@@ -72,8 +72,16 @@ ComposeMail::ComposeMail( Settings *s, QWidget *parent, const char *name, bool m
            smtpAccountBox->insertItem( smtp->getAccountName() );
            smtpAccounts.append( smtp );
         }
     }
+    if ( smtpAccounts.count() > 0 ) {
+        fillValues( smtpAccountBox->currentItem() );
+    } else {
+        QMessageBox::information( 0, tr( "Problem" ),
+                                  tr( "Please create an SMTP account first.\nThe SMTP is needed for sending mail.\n" ),
+                                  tr( "Ok" ) );
+        return;
+    }
     connect( smtpAccountBox, SIGNAL( activated(int) ), SLOT( fillValues(int) ) );
     connect( toButton, SIGNAL( clicked() ), SLOT( pickAddressTo() ) );
     connect( ccButton, SIGNAL( clicked() ), SLOT( pickAddressCC() ) );
     connect( bccButton, SIGNAL( clicked() ), SLOT( pickAddressBCC() ) );
@@ -82,16 +90,9 @@ ComposeMail::ComposeMail( Settings *s, QWidget *parent, const char *name, bool m
     connect( deleteButton, SIGNAL( clicked() ), SLOT( removeAttachment() ) );
     connect( SaveButton, SIGNAL( clicked() ), SLOT( saveAsDraft()) );
     mMail = 0;
     warnAttach = true;
-    if ( smtpAccounts.count() > 0 ) {
-        fillValues( smtpAccountBox->currentItem() );
-    } else {
-        QMessageBox::information( 0, tr( "Problem" ),
-                                  tr( "Please create an SMTP account first.\nThe SMTP is needed for sending mail.\n" ),
-                                  tr( "Ok" ) );
-        return;
-    }
+   
 }
 void ComposeMail::saveAsDraft()
 {
   
@@ -255,8 +256,14 @@ void ComposeMail::removeAttachment()
 }
 
 void ComposeMail::accept()
 {
+    if ( smtpAccountBox->count() == 0 ) { 
+       
+        reject();
+        return;
+    }
+
     if (! checkBoxLater->isChecked() ) {
         int yesno = QMessageBox::warning(0,tr("Stop editing message"),
                                          tr("Send this message?"),
                                          tr("Yes"),
@@ -326,8 +333,13 @@ void ComposeMail::reject()
                                      tr("No"));
 
     //qDebug("button %d ", yesno);
     if (yesno == 0) {
+        if ( toLine->text().isEmpty() ) {
+            QMessageBox::warning(0,tr("Sending mail"),
+                                 tr("No Receiver spezified" ) );
+            return;
+        }  
         saveAsDraft();
     }
     if (yesno == 2) {
         qDebug("return ");