[asterisk-dev] [Code Review] app_queue per member ringinuse [ignorebusy] / set chan group on answer / change behaviour of neg penalty

irroot reviewboard at asterisk.org
Sat Feb 19 14:01:23 CST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1119/
-----------------------------------------------------------

(Updated 2011-02-19 14:01:22.967652)


Review request for Asterisk Developers.


Changes
-------

Made the negative penalty change a global config option with the default existing behavior.

also added my autopause delay patch to delay autopause as per a request from a customer see it as 
a member "grace" period.


Summary (updated)
-------

This patch is a combination of 3 changes id like to submit.

1)allow a per member ringinuse setting called ignorebusy this was a requirement of a customer and and has been most useful
2)when a call is answered set a channel group this can be used to count active calls as well as be used to not send calls to users flaged above in the dialplan
3)when a member penalty is negative see this as a logged off agent this is a behavior change that allows "removing" realtime agents as well as making IMHO more logic sense if the penalty is neg its not valid 

NB: this is a global config option now as not to change behavior.

4)allow a "grace" period for autopause queue option "autopausedelay" [as requested by a customer we support]

Enjoy the fish


This addresses bugs 18794, 18825 and 18826.
    https://issues.asterisk.org/view.php?id=18794
    https://issues.asterisk.org/view.php?id=18825
    https://issues.asterisk.org/view.php?id=18826


Diffs (updated)
-----

  /trunk/apps/app_queue.c 308283 
  /trunk/configs/queues.conf.sample 308283 

Diff: https://reviewboard.asterisk.org/r/1119/diff


Testing
-------


Thanks,

irroot

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110219/fc6edb8f/attachment.htm>


More information about the asterisk-dev mailing list