[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    aegypten_branch: kdepim/kmail
From:       Marc Mutz <Marc.Mutz () uni-bielefeld ! de>
Date:       2004-05-29 15:10:24
Message-ID: 20040529151024.40FF4126EB () office ! kde ! org
[Download RAW message or body]

CVS commit by mutz: 

Merge from HEAD: CVS commit by khz: 

fixing bug #33653 (equals kolab issue #96) according to RFC 2822, 3.4 allowing to \
have an EMPTY group of mailboxes in the TO header field.

(the patch to messagecomposer.cpp a) doesn't apply anymore and b) doesn't apply \
anymore.


  M +13 -13    kmcomposewin.cpp   1.765.2.38
  M +19 -1     kmsender.cpp   1.195.2.5


--- kdepim/kmail/kmcomposewin.cpp  #1.765.2.37:1.765.2.38
@@ -2810,10 +2810,10 @@ void KMComposeWin::doSend(int aSendNow, 
   if (!saveInDrafts)
   {
-     if (to().isEmpty())
+    if (to().isEmpty() && cc().isEmpty() && bcc().isEmpty())
      {
         mEdtTo->setFocus();
         KMessageBox::information( this,
-                                  i18n("You must specify at least one "
-                                       "receiver in the To: field.") );
+                                i18n("You must specify at least one receiver,"
+                                     "either in the To: field or as CC or as BCC.") \
);  return;
      }

--- kdepim/kmail/kmsender.cpp  #1.195.2.4:1.195.2.5
@@ -124,5 +124,22 @@ bool KMSender::send(KMMessage* aMsg, sho
     // Note that the "Bcc" field may be empty, while the "To" field is required to
     // have at least one address.
-    return FALSE;
+    //
+    // however:
+    //
+    // The following string is accepted according to RFC 2822,
+    // section 3.4 "Address Specification" where they say:
+    //
+    //     "An address may either be an individual mailbox,
+    //      or a group of mailboxes."
+    // and:
+    //     "group   +   display-name ":" [mailbox-list / CFWS] ";"
+    //      [CFWS]"
+    //
+    // In this syntax our "undisclosed-recipients: ;"
+    // just specifies an empty group.
+    //
+    // In further explanations RFC 2822 states that it *is*
+    // allowed to have a ZERO number of mailboxes in the "mailbox-list".
+    aMsg->setTo("Undisclosed.Recipients: ;");
   }
 


[prev in list] [next in list] [prev in thread] [next in thread]