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.

STORM-1041 Removed clothes come back to haunt me

Review Request #441 - Created Aug. 15, 2011 and submitted

Vadim ProductEngine Reviewers
viewer
STORM-1041 nyx.linden
None viewer-development 
The "Remove all clothes" item of the avatar menu didn't actually remove all clothes.

I haven't investigated what the problem was, I've just rewritten the (ancient?) removal code
in the way we take off items in other places, i.e. by removing them from the Current Outfit forder.

What I'm not sure about is whether we need to call updateAppearanceFromCOF() afterwards. Nyx?

 

 










  

 


















 

  




 

 

  
Ship it!

  

  
Nyx Linden

  
Posted  (Aug. 17, 2011, 8:48 a.m.)

 

 

   
I was skeptical, but after reviewing some of the code from those systems (that I hadn't touched in months), the proposed code appears to be correct. Also tested the build and it appears functional. Ship it!