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

Wolfpup Lowenhar wolfpup67 at earthlink.net
Tue Apr 5 10:23:49 PDT 2011



> On April 5, 2011, 9:02 a.m., Wolfpup Lowenhar wrote:
> > indra/llui/llurlaction.h, line 81
> > <http://codereview.secondlife.com/r/249/diff/1/?file=1396#file1396line81>
> >
> >     Having this here looks to be the best way to prevent generating a Legacy named P2P system message during said conversation that has been started by a person that DOSE NOT have Display Names turned on as there is one SYSTEM message that would seem to be coming from no where and this looks like it is its source.
> 
> Seth ProductEngine wrote:
>     I guess Vadim is right about moving the method. Seems that just stripping a UUID part of a given SLURL doesn't really belong here, but moving it won't affect the patch functionality.
>     
>     Wolfpup, do you have some objections against moving the method or some suggestions about the functionality changes?

Not at all.


- Wolfpup


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


On April 4, 2011, 4:30 p.m., Seth ProductEngine wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/249/
> -----------------------------------------------------------
> 
> (Updated April 4, 2011, 4:30 p.m.)
> 
> 
> Review request for Viewer.
> 
> 
> 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
> -----
> 
>   indra/llui/llurlaction.h d30636c2a83a 
>   indra/llui/llurlaction.cpp d30636c2a83a 
>   indra/newview/llgiveinventory.cpp 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/20110405/f3d3ba57/attachment.htm 


More information about the opensource-dev mailing list