-----------------------------------------------------------
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.


Changes
-------

Applied the fix to translations and to the minimal (basic viewer) skin.


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 (updated)
-----

  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

Reply via email to