[opensource-dev] Review Request: storm-1037: remove the 'hide url' checkboxes from parcel management

Oz Linden oz at lindenlab.com
Thu Mar 3 11:52:40 PST 2011


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

Review request for Viewer.


Summary
-------

This patch removes the two "Hide URL" checkboxes in the parcel management screens.
The corresponding flags are not removed from the messages exchanged with server in order to preserve backward compatibility; instead, they are always sent as 'false' (show the url), and are ignored when received.


This addresses bug storm-1037.
    http://jira.secondlife.com/browse/storm-1037


Diffs
-----

  indra/llinventory/llparcel.h 8596ccc522d9 
  indra/llinventory/llparcel.cpp 8596ccc522d9 
  indra/newview/llfloaterauction.cpp 8596ccc522d9 
  indra/newview/llpanellandaudio.cpp 8596ccc522d9 
  indra/newview/llpanellandmedia.cpp 8596ccc522d9 
  indra/newview/llpanelnearbymedia.cpp 8596ccc522d9 
  indra/newview/skins/default/xui/da/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/de/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/en/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/es/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/fr/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/it/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/ja/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/nl/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/pl/floater_about_land.xml 8596ccc522d9 
  indra/newview/skins/default/xui/pt/floater_about_land.xml 8596ccc522d9 

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


Testing
-------

Verified that the checkboxes are no longer present, and that media can be set using either dialog.


Thanks,

Oz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110303/0b1fa88f/attachment.htm 


More information about the opensource-dev mailing list