[asterisk-dev] [Code Review] Allow to set the wrapuptime per queue member, and disable the wrapuptime for the next call or increase its value as requested.

Eliel Sardañons eliels at gmail.com
Mon Apr 20 08:19:21 CDT 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/222/
-----------------------------------------------------------

(Updated 2009-04-20 08:19:21.773273)


Review request for Asterisk Developers.


Changes
-------

- Fix doxygen documentation.
- Add manager_1_1 documentation.


Summary
-------

The idea is to allow to configure the wrapuptime for each member of a queue, and allow to disable the wrapuptime or increase it for the next call.
There is a manager action (the documentation is missing) called QueueWrapuptime (I would like to call it QueueMemberWrapuptime, but penalty is also called QueuePenalty [??]), the action takes four params:
Interface: <required>
Queue: <optional>
Static: <required>
{Offset | Absolute}: <required>

If a queue is passed, only the member in that queue is updated, if it is not specified, all the members with interface 'Interface' will be updated.
You can use 'Offset' or 'Absolute' to change the wrapuptime. Offset will increase or decrease the wrapuptime instead of replacing it with your value.
Example (the agent wants more time):
Action: QueueWrapuptime
Interface: Agent/1000
Static: 0
Offset: 60

Or The user wants to disable the wrapuptime for the next call:
Action: QueueWrapuptime
Interface: Agent/1000
Static: 0
Absolute: 0

The 'Static' parameter will tell the action to persist the new wrapuptime in the member configuration (in memory) and will be used for all the future calls. If 'Static' is 0 the wrapuptime will be changed only for the next call.


There is a CLI command 'queue set wrapuptime <value> on <member> [in <queue>] [static]'.


Diffs (updated)
-----

  /trunk/CHANGES 189275 
  /trunk/apps/app_queue.c 189275 
  /trunk/doc/manager_1_1.txt 189275 
  /trunk/include/asterisk/cli.h 189275 
  /trunk/main/cli.c 189275 

Diff: http://reviewboard.digium.com/r/222/diff


Testing
-------

Minor developer testing.

More testing will be needed before commit.


Thanks,

Eliel




More information about the asterisk-dev mailing list