[opensource-dev] Review Request: STORM-1804 Details... button on PERMISSION_DEBIT dialog triggers run_time_permissions() with a deny action

[Jonathan Yap]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/544/
-----------------------------------------------------------

Review request for Viewer.


Description
-------

When an object requests DEBIT permissions a large, yellow dialog appears. This dialog has three buttons: "Grant", "Deny", and "Details..." When you select "Details..." Additional information about DEBIT permission is displayed.

The bug is that when "Details..." is selected PERMISSION_DEBIT is denied and the run_time_permissions() event handler is triggered. There is also a system message in the Local Chat pane of the communicate window that informs the user that permission has been denied. Further, if "Grant" or "Deny" are pressed AFTER details the run_time_permissions() event handler is NOT triggered, yet there is a system message saying that permissions have been granted or denied.

What should happen when "Details..." is pressed is no run_time_permissions event should be triggered and the "Grant" and "Deny" buttons should operate normally and trigger the run_time_permissions event afterwards.


This addresses bug STORM-1804.
    http://jira.secondlife.com/browse/STORM-1804


Diffs
-----

  doc/contributions.txt 3f2162768b29 
  indra/newview/llviewermessage.cpp 3f2162768b29 

Diff: http://codereview.secondlife.com/r/544/diff/diff


Testing
-------

See test script in jira.

Click on Details: additional information is shown, click on Ok to dismiss that, then you can go back and click on Grant or Deny.


Thanks,

Jonathan Yap

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20120125/58a9f44e/attachment.htm