[opensource-dev] Review Request: STORM-793 Offered URL-like named object is displayed as a HTTP URL in the llGiveInventory

[Vadim ProductEngine]

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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110817/165979b7/attachment.htm