[opensource-dev] Review Request: STORM-1543 During busy mode inventory offers get silently deleted instead of thrown into trash
Vadim ProductEngine
vsavchuk at productengine.com
Thu Aug 25 14:23:14 PDT 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/444/
-----------------------------------------------------------
(Updated Aug. 25, 2011, 2:23 p.m.)
Review request for Viewer and Richard Nelson.
Changes
-------
In response to Richard's feedback, removed invalid IOR_BUSY response option.
Summary
-------
In busy mode offered inventory items are now moved to trash.
There were two bugs:
1. When auto-discarding inventory offers we looked up missing Busy button
(i.e. a button having index=3) in the inventory offer notification dialog
template. Failure to find the button resulted in ignoring inventory offers.
Fixed that by "auto-clicking" the existing Discard button.
2. It turned out impossible to properly remove an inventory item
from within LLDiscardAgentOffer::done(), because that would lead to
nested LLInventoryModel::notifyObservers() calls.
Fixed that by deferring removal until the next LLAppViewer::idle() iteration.
This addresses bug STORM-1543.
http://jira.secondlife.com/browse/STORM-1543
Diffs (updated)
-----
indra/newview/llappviewer.h 7dbd8eaefaec
indra/newview/llappviewer.cpp 7dbd8eaefaec
indra/newview/llviewermessage.h 7dbd8eaefaec
indra/newview/llviewermessage.cpp 7dbd8eaefaec
Diff: http://codereview.secondlife.com/r/444/diff
Testing
-------
Thanks,
Vadim
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110825/2d8f0042/attachment.htm
More information about the opensource-dev
mailing list