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.

Dragging the "Contents" folder from object inventory to agent inventory shows the "prohibited" cursor

Review Request #26 - Created Dec. 16, 2010 and submitted

Kitty Barnett Reviewers
viewer
VWR-24217
None viewer-development 
The "Contents" folder is an LLInventoryObject instance, and failed the dynamic cast to LLInventoryItem.

The permission logic didn't apply since it's a category (which has no permissions) and was changed to use the same function that will do the actual copying so that drop behaviour should always match the "can drop" result.

 

 










  

 


















 

  




 

 

  
Ship it!

  

  
Merov Linden

  
Posted  (Dec. 16, 2010, 10:13 a.m.)

 

 

   
I think this is correct though I haven't tried the code. Good use of move_inv_category_world_to_agent() that does all the right perm checking.