Monotone-Parent: 3c4b94054ca0b56c5e59f42bb1293b5f1040611c

Monotone-Revision: e2c92d491c4f1a4138cd77034e3c0a712bf09003

Monotone-Author: wsourdeau@inverse.ca
Monotone-Date: 2009-10-01T14:47:51
Monotone-Branch: ca.inverse.sogo

ChangeLog

@@ -1,3 +1,10 @@
+2009-10-01  Cyril Robert  <crobert@inverse.ca>
+
+	* SoObjects/Mailer/SOGoMailFolder.m
+	(-appendMessage:inContext:usingId:): we no longer instantiate the
+	message to check whether it exists since we trust the fact that we
+	would receive an error from NGImap if the creation failed.
+
 2009-10-01  Wolfgang Sourdeau  <wsourdeau@inverse.ca>
 
 	* Tests/test-maildav.py (DAVMailCollectionTest.testPUT): fixed to

SoObjects/Mailer/SOGoMailFolder.m

@@ -1133,32 +1133,18 @@ static NSString *spoolFolder = nil;
   NSException *error;
   WOResponse *response;
   NSString *location;
-  id msg;
 
   error = [self _appendMessageData: message
                            usingId: imap4id];
-
   if (error)
     response = (WOResponse *) error;
   else
     {
       response = [_ctx response];
       [response setStatus: 201];
-      msg = [SOGoMailObject objectWithName: 
-             [NSString stringWithFormat: @"%d", *imap4id] 
-                               inContainer: self];
-      if (msg && [msg doesMailExist])
-        {
-          location = [NSString stringWithFormat: @"%@%d.eml",
-                      [self davURL], *imap4id];
-          [response setHeader: location forKey: @"location"];
-        }
-      else
-        {
-          response = (WOResponse *) 
-            [NSException exceptionWithHTTPStatus: 500
-                                          reason: @"Failed to store message"];
-        }
+      location = [NSString stringWithFormat: @"%@%d.eml",
+                           [self davURL], *imap4id];
+      [response setHeader: location forKey: @"location"];
     }
 
   return response;