STORM-519 ( "Delete" is enabled in the context menu for folders which contain worn items)
Review Request #396 - Created July 14, 2011 and submitted
| Paul ProductEngine | Reviewers | ||
| viewer | |||
| storm-519 | |||
| None | viewer-development | ||
- Disable "Delete" menu item in case selected folder contains non-removable items.
Expand AllStraightforward fix.
Posted (July 14, 2011, 9:03 a.m.)
-
indra/newview/llinventorybridge.cpp (Diff revision 1) -
Why store this in a variable when the result is only used once and the call is this simple? (I assume the result of this call wouldn't change between here where the value is stored and the below where it's used.)
-
indra/newview/llinventorybridge.cpp (Diff revision 1) -
We could just use if (!isItemRemovable()) here, couldn't we?
Review request changed
Updated (July 19, 2011, 11:11 a.m.)
-
- added Diff r2
Other reviews