[opensource-dev] Review Request: VWR-21522: Prevent unintended 10L sale
Kadah Coba
kadah.coba at gmail.com
Thu Jun 30 10:33:42 PDT 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/365/
-----------------------------------------------------------
(Updated June 30, 2011, 10:33 a.m.)
Review request for Viewer.
Changes
-------
Requested CS changes.
Would you like a clean fork with this as one changeset? The current fork got too messy for my standards.
Summary
-------
This change makes it so setting an object for sale or modifying the price or sale type requires explicitly applying the changes. Default behavior before this patch is all sale info is committed to sim on change. After this patch the sale info will only get set or modified after hitting a "mark for sale" button. There is no change to removing an object from sale, unchecking "For sale" will be committed to sim without further user interation.
https://bitbucket.org/Kadah_Coba/vwr-21522
Main Patch: https://bitbucket.org/Kadah_Coba/vwr-21522/changeset/45df3e1e9f6f
German translation: https://bitbucket.org/Kadah_Coba/vwr-21522/changeset/01dc58391588
This addresses bug VWR-21522.
http://jira.secondlife.com/browse/VWR-21522
Diffs (updated)
-----
doc/contributions.txt UNKNOWN
indra/newview/llpanelpermissions.h UNKNOWN
indra/newview/llpanelpermissions.cpp UNKNOWN
indra/newview/skins/default/xui/de/floater_tools.xml UNKNOWN
indra/newview/skins/default/xui/en/floater_tools.xml UNKNOWN
Diff: http://codereview.secondlife.com/r/365/diff
Testing
-------
(I had stuff here but Review Board kept deleting it every time I tried to save.)
Thanks,
Kadah
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110630/9b0a5591/attachment.htm
More information about the opensource-dev
mailing list