ipsservices/ipssosplugin/src/ipsplgcreateforwardmessageoperation.cpp
changeset 23 2dc6caa42ec3
parent 20 ecc8def7944a
--- a/ipsservices/ipssosplugin/src/ipsplgcreateforwardmessageoperation.cpp	Mon May 03 12:23:15 2010 +0300
+++ b/ipsservices/ipssosplugin/src/ipsplgcreateforwardmessageoperation.cpp	Fri May 14 15:41:10 2010 +0300
@@ -18,11 +18,10 @@
 // <qmail>
 
 // INCLUDE FILES
-
 #include "emailtrace.h"
 #include "ipsplgheaders.h"
 
-// LOCAL CONSTANTS AND MACROS
+// <qmail> removed comment
 
 // ================= MEMBER FUNCTIONS =======================
 
@@ -30,8 +29,10 @@
 // CIpsPlgCreateForwardMessageOperation::CIpsPlgCreateForwardMessageOperation
 // ----------------------------------------------------------------------------
 //
+//<qmail> aSmtpService changed to reference
+//<qmail> KMsvNullIndexEntry para removed
 CIpsPlgCreateForwardMessageOperation::CIpsPlgCreateForwardMessageOperation(
-    CIpsPlgSmtpService* aSmtpService,
+    CIpsPlgSmtpService& aSmtpService,
     CMsvSession& aMsvSession,
     TRequestStatus& aObserverRequestStatus,
     TMsvPartList aPartList,
@@ -44,7 +45,6 @@
         aSmtpService,
         aMsvSession,
         aObserverRequestStatus,
-        KMsvNullIndexEntryId,
         aPartList,
         aMailBoxId,
         aOperationObserver,
@@ -53,14 +53,16 @@
     {
     FUNC_LOG;
     }
+//</qmail>	
 
 
 // ----------------------------------------------------------------------------
 // CIpsPlgCreateForwardMessageOperation::NewL
 // ----------------------------------------------------------------------------
 //
+//<qmail> aSmtpService changed to reference
 CIpsPlgCreateForwardMessageOperation* CIpsPlgCreateForwardMessageOperation::NewL(
-    CIpsPlgSmtpService* aSmtpService,
+    CIpsPlgSmtpService& aSmtpService,
     CMsvSession& aMsvSession,
     TRequestStatus& aObserverRequestStatus,
     TMsvPartList aPartList,
@@ -68,6 +70,7 @@
     TMsvId aOriginalMessageId, 
     MFSMailRequestObserver& aOperationObserver,
     TInt aRequestId )
+//</qmail>	
     {
     FUNC_LOG;
     CIpsPlgCreateForwardMessageOperation* self =
@@ -81,7 +84,9 @@
             aOperationObserver, 
             aRequestId );
     CleanupStack::PushL( self );
-    self->ConstructL();
+    // <qmail>
+    self->ConstructL(); // Use base class constructor
+    // </qmail>
     CleanupStack::Pop( self ); 
     return self;
     }
@@ -106,16 +111,14 @@
         {
         // new message creation has finished so make an FS type message
         CFSMailMessage* newMessage = NULL;
+        //<qmail> TRAP removed
+        TMsvId msgId;
         
-        TMsvId msgId = TMsvId();
-        TRAPD( err, msgId = GetIdFromProgressL( iOperation->FinalProgress() ) );
+        msgId = GetIdFromProgressL( iOperation->FinalProgress() );
             
-        if( err == KErrNone )
-            {
-            newMessage = iSmtpService->CreateFSMessageAndSetFlagsL( 
-                    msgId, iOriginalMessageId, iFSMailboxId.Id(), ETrue );
-            }
-        
+        newMessage = iSmtpService.CreateFSMessageAndSetFlagsL( 
+                msgId, iOriginalMessageId, iFSMailboxId.Id(), ETrue );
+        //</qmail>
         // relay the created message (observer takes ownership)
         SignalFSObserver( iStatus.Int(), newMessage );        
         }
@@ -134,7 +137,7 @@
     FUNC_LOG;
     delete iOperation;
     iOperation = NULL;
-    
+    //<qmail> commented parameter removed
     // Start a new operation, execution continues in RunL 
     // once the operation has finished.
     iOperation = CImEmailOperation::CreateForwardL(
@@ -145,8 +148,8 @@
             iPartList, 
             KIpsPlgForwardSubjectFormat,
             KMsvEmailTypeListMHTMLMessage,
-            //0,
             KUidMsgTypeSMTP);
+    //</qmail>
     }
 
 //  End of File