[opensource-dev] Review Request: (STORM-941) IM log naming should go by SL name, not DN.

[Seth ProductEngine]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/249/
-----------------------------------------------------------

(Updated April 7, 2011, 4:16 p.m.)


Review request for Viewer.


Changes
-------

Fixed resolving IM session participants names prior to writing messages to the log file to avoid temporary log files which could possibly appear if user names were not resolved before the first message had been written to the log.


Summary
-------

Fixed IM history to use the resident's user name for the log file name.
Added conversions from legacy names or SLURLs with avatar id to the user names in cases of logging P2P sessions and inventory offers.


This addresses bug STORM-941.
    http://jira.secondlife.com/browse/STORM-941


Diffs (updated)
-----

  indra/newview/llgiveinventory.cpp d30636c2a83a 
  indra/newview/llimview.h d30636c2a83a 
  indra/newview/llimview.cpp d30636c2a83a 
  indra/newview/llnotificationhandler.h d30636c2a83a 
  indra/newview/llnotificationhandlerutil.cpp d30636c2a83a 

Diff: http://codereview.secondlife.com/r/249/diff


Testing
-------


Thanks,

Seth

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110407/043563e2/attachment.htm