[opensource-dev] Review Request: Dragging the "Contents" folder from object inventory to agent inventory shows the "prohibited" cursor
Merov Linden
merov at lindenlab.com
Thu Dec 16 10:13:29 PST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/26/#review41
-----------------------------------------------------------
Ship it!
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.
- Merov
On 2010-12-16 03:04:28, Kitty Barnett wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://codereview.secondlife.com/r/26/
> -----------------------------------------------------------
>
> (Updated 2010-12-16 03:04:28)
>
>
> Review request for Viewer.
>
>
> Summary
> -------
>
> 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.
>
>
> This addresses bug VWR-24217.
> http://jira.secondlife.com/browse/VWR-24217
>
>
> Diffs
> -----
>
> indra/newview/llpanelobjectinventory.cpp UNKNOWN
>
> Diff: http://codereview.secondlife.com/r/26/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> Kitty
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20101216/462b0693/attachment.htm
More information about the opensource-dev
mailing list