----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://codereview.secondlife.com/r/442/#review982 -----------------------------------------------------------
Ship it! Looks good now. - Boroondas On Aug. 17, 2011, 3:21 a.m., Vadim ProductEngine wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://codereview.secondlife.com/r/442/ > ----------------------------------------------------------- > > (Updated Aug. 17, 2011, 3:21 a.m.) > > > Review request for Viewer. > > > Summary > ------- > > Disallowed parsing HTTP URLs in llGiveInventory notifications (one more case). > > The fix may look weird because we seem to wrap a SLURL with > <nolink>...</nolink> tags. > But actually there is no SLURL, there's just an object name. > Investigating that goes beyond the bug scope. > > > This addresses bug STORM-793. > http://jira.secondlife.com/browse/STORM-793 > > > Diffs > ----- > > indra/newview/skins/default/xui/da/notifications.xml UNKNOWN > indra/newview/skins/default/xui/de/notifications.xml UNKNOWN > indra/newview/skins/default/xui/en/notifications.xml UNKNOWN > indra/newview/skins/default/xui/es/notifications.xml UNKNOWN > indra/newview/skins/default/xui/fr/notifications.xml UNKNOWN > indra/newview/skins/default/xui/it/notifications.xml UNKNOWN > indra/newview/skins/default/xui/ja/notifications.xml UNKNOWN > indra/newview/skins/default/xui/nl/notifications.xml UNKNOWN > indra/newview/skins/default/xui/pl/notifications.xml UNKNOWN > indra/newview/skins/default/xui/pt/notifications.xml UNKNOWN > indra/newview/skins/default/xui/zh/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/da/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/de/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/en/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/es/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/fr/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/it/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/ja/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/pl/notifications.xml UNKNOWN > indra/newview/skins/minimal/xui/pt/notifications.xml UNKNOWN > > Diff: http://codereview.secondlife.com/r/442/diff > > > Testing > ------- > > > Thanks, > > Vadim > >
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
