diff -r 16ed8d08d0b1 -r 478bc57ad291 emailservices/emailcommon/src/CFSMailMessageBase.cpp --- a/emailservices/emailcommon/src/CFSMailMessageBase.cpp Wed Aug 18 09:37:47 2010 +0300 +++ b/emailservices/emailcommon/src/CFSMailMessageBase.cpp Thu Sep 02 20:15:00 2010 +0300 @@ -1,5 +1,5 @@ /* -* Copyright (c) 2007-2009 Nokia Corporation and/or its subsidiary(-ies). +* Copyright (c) 2007-2010 Nokia Corporation and/or its subsidiary(-ies). * All rights reserved. * This component and the accompanying materials are made available * under the terms of "Eclipse Public License v1.0" @@ -63,8 +63,8 @@ NM_FUNCTION; // unnecessary member initialization removed: iSubject, iFlags - // set request handler pointer - iRequestHandler = static_cast(Dll::Tls()); + // set request handler pointer + iRequestHandler = static_cast(Dll::Tls()); } // ----------------------------------------------------------------------------- @@ -78,13 +78,13 @@ // Construction of shared data object iNmPrivateMessageEnvelope = new NmMessageEnvelopePrivate(); - // typedef long int TInt32 -> typedef unsigned int quint32 + // typedef long int TInt32 -> typedef unsigned int quint32 iNmPrivateMessageEnvelope->mMessageId.setId32((quint32)aMessageId.Id() ); - // typedef unsigned int TUint -> typedef unsigned int quint32 + // typedef unsigned int TUint -> typedef unsigned int quint32 iNmPrivateMessageEnvelope->mMessageId.setPluginId32((quint32)aMessageId.PluginId().iUid); - // construct the CFSMailAddress object and connect it with NmAddress private data + // construct the CFSMailAddress object and connect it with NmAddress private data iSender = CFSMailAddress::NewL(iNmPrivateMessageEnvelope->mSender); // } @@ -109,11 +109,11 @@ { NM_FUNCTION; - if(iSender) - { - delete iSender; - iSender = NULL; - } + if(iSender) + { + delete iSender; + iSender = NULL; + } // unnecessary member destruction removed: iSubject, iToRecipients, iCcRecipients, iBccRecipients } @@ -132,7 +132,7 @@ if(id.IsNullId()){ id = TFSMailMsgId(iNmPrivateMessageEnvelope->mFolderId); } - + return id; // } @@ -210,13 +210,20 @@ NM_FUNCTION; // - // store sender - if (iSender) - { - delete iSender; + // store sender + if (iSender) + { + delete iSender; iSender = NULL; - } - iSender = aSender; + } + if (!aSender) + { + TRAP_IGNORE(iSender = CFSMailAddress::NewL()); + } + else + { + iSender = aSender; + } iNmPrivateMessageEnvelope->mSender = iSender->GetNmAddress(); // } @@ -228,7 +235,7 @@ { NM_FUNCTION; - return iSender; + return iSender; } // ----------------------------------------------------------------------------- @@ -278,8 +285,10 @@ NM_FUNCTION; // - iNmPrivateMessageEnvelope->mToRecipients.append( - aRecipient->GetNmAddress()); + if (aRecipient) + { + iNmPrivateMessageEnvelope->mToRecipients.append(aRecipient->GetNmAddress()); + } // } @@ -291,8 +300,11 @@ NM_FUNCTION; // - iNmPrivateMessageEnvelope->mCcRecipients.append( - aRecipient->GetNmAddress()); + if (aRecipient) + { + iNmPrivateMessageEnvelope->mCcRecipients.append( + aRecipient->GetNmAddress()); + } // } @@ -304,8 +316,11 @@ NM_FUNCTION; // - iNmPrivateMessageEnvelope->mBccRecipients.append( - aRecipient->GetNmAddress()); + if (aRecipient) + { + iNmPrivateMessageEnvelope->mBccRecipients.append( + aRecipient->GetNmAddress()); + } // } @@ -313,37 +328,37 @@ // CFSMailMessageBase::ClearToRecipients // ----------------------------------------------------------------------------- EXPORT_C void CFSMailMessageBase::ClearToRecipients( ) - { +{ NM_FUNCTION; // iNmPrivateMessageEnvelope->mToRecipients.clear(); // - } +} // ----------------------------------------------------------------------------- // CFSMailMessageBase::ClearCcRecipients // ----------------------------------------------------------------------------- EXPORT_C void CFSMailMessageBase::ClearCcRecipients( ) - { +{ NM_FUNCTION; // iNmPrivateMessageEnvelope->mCcRecipients.clear(); // - } +} // ----------------------------------------------------------------------------- // CFSMailMessageBase::ClearBccRecipients // ----------------------------------------------------------------------------- EXPORT_C void CFSMailMessageBase::ClearBccRecipients( ) - { +{ NM_FUNCTION; // iNmPrivateMessageEnvelope->mBccRecipients.clear(); // - } +} // // ----------------------------------------------------------------------------- @@ -451,38 +466,18 @@ } // ----------------------------------------------------------------------------- -// CFSMailMessageBase::IsRelatedTo -// ----------------------------------------------------------------------------- -EXPORT_C TFSMailMsgId CFSMailMessageBase::IsRelatedTo() const - { - NM_FUNCTION; - - return iRelatedTo; - } - -// ----------------------------------------------------------------------------- -// CFSMailMessageBase::SetRelatedTo -// ----------------------------------------------------------------------------- -EXPORT_C void CFSMailMessageBase::SetRelatedTo( const TFSMailMsgId aMessageId ) - { - NM_FUNCTION; - - iRelatedTo = aMessageId; - } - -// ----------------------------------------------------------------------------- // CFSMailMessageBase::SetReplyToAddress // ----------------------------------------------------------------------------- EXPORT_C void CFSMailMessageBase::SetReplyToAddress(CFSMailAddress* aReplyToAddress) { NM_FUNCTION; - // store sender - if (iReplyTo) - { - delete iReplyTo; - } - iReplyTo = aReplyToAddress; + // store sender + if (iReplyTo) + { + delete iReplyTo; + } + iReplyTo = aReplyToAddress; } @@ -493,14 +488,14 @@ { NM_FUNCTION; - return *iReplyTo; + return *iReplyTo; } EXPORT_C CFSMailRequestHandler& CFSMailMessageBase::RequestHandler( ) { NM_FUNCTION; - return *iRequestHandler; + return *iRequestHandler; } //