diff -r 91c2fb4b78df -r ba54057fe027 phoneengine/loghandling/src/cpelogevent.cpp --- a/phoneengine/loghandling/src/cpelogevent.cpp Tue Apr 27 16:37:10 2010 +0300 +++ b/phoneengine/loghandling/src/cpelogevent.cpp Tue May 11 16:13:03 2010 +0300 @@ -658,7 +658,12 @@ { if ( KNullDesC() != aLogInfo.Name() ) { - aEvent.SetRemoteParty( aLogInfo.Name() ); + // This if clause fixes ou1cimx1#320365 Wrong address shown in received log entry + if ( aLogInfo.EventType() != CPELogInfo::EPEVoIPEvent || + KNullDesC() == aEvent.RemoteParty() ) + { + aEvent.SetRemoteParty( aLogInfo.Name() ); + } } } @@ -670,22 +675,18 @@ void CPELogEvent::SetRemoteContact( CLogEvent& aEvent, const CPELogInfo& aLogInfo ) { - if ( KNullDesC() != aLogInfo.PhoneNumber() ) + if ( KNullDesC() != aLogInfo.PhoneNumber() ) { - aEvent.SetNumber( aLogInfo.PhoneNumber() ); - - if ( CPELogInfo::EPEVoIPEvent == aLogInfo.EventType() ) + if ( aLogInfo.EventType() != CPELogInfo::EPEVoIPEvent || + ( KNullDesC() == aEvent.Number() && + KErrNotFound == iEvent->Data().Find( KLogsDataFldTag_URL ) ) ) { - TRAPD( error, RemoveTagFromDataFieldL( KLogsDataFldTag_URL ) ); - if( error ) - { - TEFLOGSTRING2( KTAERROR, - "LOG CPELogEvent::SetRemoteContact()>RemoveTagFromDataFieldL(), error=%d", error ) - } + aEvent.SetNumber( aLogInfo.PhoneNumber() ); } } - if ( KNullDesC() != aLogInfo.VoipAddress() ) + else if ( KNullDesC() != aLogInfo.VoipAddress() && + KNullDesC() == aEvent.Number()) { TRAPD( error, SetDataFieldL( KLogsDataFldTag_URL, aLogInfo.VoipAddress() ) ); if ( error ) @@ -693,38 +694,6 @@ TEFLOGSTRING2( KTAERROR, "LOG CPELogEvent::SetRemoteContact()>SetDataFieldL(), error=%d", error ) } - aEvent.SetNumber( KNullDesC() ); - } - } - -// ----------------------------------------------------------------------------- -// CPELogEvent::RemoveTagFromDataFieldL -// ----------------------------------------------------------------------------- -// -void CPELogEvent::RemoveTagFromDataFieldL( const TDesC8& aTag ) - { - __ASSERT_ALWAYS( aTag.Length() != 0 , - User::Leave( KErrArgument ) ); - - TInt index = iEvent->Data().Find( aTag ); - if ( KErrNotFound != index ) - { - HBufC8* eventData = iEvent->Data().AllocLC(); - TPtrC8 urlData = eventData->Mid( index ); - TInt otherTag = urlData.Find( KLogsDataFldNameDelimiter ); - - const TInt dlLen = KLogsDataFldNameDelimiter().Length(); - if ( KErrNotFound == otherTag ) - { - eventData->Des().Delete( index - dlLen, urlData.Length() + dlLen ); - } - else - { - eventData->Des().Delete( index - dlLen, otherTag ); - } - iEvent->SetDataL( *eventData ); - - CleanupStack::PopAndDestroy( eventData ); } }