All submissions to this site are governed by the Second Life Viewer Contribution Agreement. By submitting patches and other information using this site, you acknowledge that you have read, understood, and agreed to those terms.

Review Board 1.6.11

Welcome to the Second Life Viewer Code Review tool.
See the documentation on our wiki for how to use this site.

Bug Fix for STORM-776 and STORM-288

Review Request #21 - Created Dec. 15, 2010 and submitted

Wolfpup Lowenhar Reviewers
viewer
STORM-776
None viewer-development 
This is actually Kitty's patch for STORM-288 but it also corrects the issue listed in STORM-776 as well.
I have built and tested this locally and was able to fully modify the next person permissions on object that i had full permissions also tested to see if I could set item in inventory for sale and I could.

Test Viewers for this issue are available here:
http://automated-builds-secondlife-com.s3.amazonaws.com/hg/repo/oz_project-2/rev/216992/index.html

please test this on mac and linux to verify fix works on all platforms.
Review request changed
Updated (Dec. 15, 2010, 7:59 p.m.)

  • I have built and tested this locally and was able to fully modify the next person permissions on object that i had full permissions also tested to see if I could set item in inventory for sale and I could.

    I have built and tested this locally and was able to fully modify the next person permissions on object that i had full permissions also tested to see if I could set item in inventory for sale and I could.

Test Viewers for this issue are available here:
http://automated-builds-secondlife-com.s3.amazonaws.com/hg/repo/oz_project-2/rev/216992/index.html

please test this on mac and linux to verify fix works on all platforms.
add link to TC builds Oz did of online repository so others can test this fix.
Ship it!
Vadim ProductEngine
Posted (Dec. 16, 2010, 5:56 a.m.)
Looks plausible and works for me.