[asterisk-bugs] [Asterisk 0015168]: [patch] Add ability to use extension state as well as device state when adding queue memebers

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15168 
====================================================================== 
Reported By:                p_lindheimer
Assigned To:                mmichelson
====================================================================== 
Project:                    Asterisk
Issue ID:                   15168
Category:                   Applications/app_queue
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-05-20 15:43 CDT
Last Modified:              2009-05-25 13:50 CDT
====================================================================== 
Summary:                    [patch] Add ability to use extension state as well
as device state when adding queue memebers
Description: 
The new ability to add a device state associated with a Queue member when
configuring a queue or adding a member with AddQueueMembers solves a lot of
problems. However, it would be EXTREMELY valuable to use extension state
information as well (e.g. hints).

hints are extensively used in may dialplans to track extension state of a
system. This ability becomes even more important when you have a pbx that
supports 'hot-desking' where a user can log into and out of one or multiple
devices.

It would be extremely valuable, and from what I understand, relatively
straight forward, to add this ability such that conceptually you could an
entry such as:

member => Local/3000 at default,0,John Smith,HINT:3000 at default

and the queue would parse that this is a hint instead of a device, and use
ast_extension_state() instead of ast_device_state()


====================================================================== 

---------------------------------------------------------------------- 
 (0105374) p_lindheimer (reporter) - 2009-05-25 13:50
 https://issues.asterisk.org/view.php?id=15168#c105374 
---------------------------------------------------------------------- 
oops - forgot to check the box, it is a code submission (but I've signed
the waiver)

The following patch, queue_extenstate_1.4.svn.patch provides my first pass
at modifying the code so that it can included hints as well as devices. It
is applied against r196621 of the 1.4 branch.

It expects HINT:nnn at context format if you want to use a HINT instead of a
device. This patch also includes a fix in the handle_statechange() function
that monitors state change. That function currently ignores state changes
for Custom DEVSTATESs. Note that the bug fix is not complete, since it will
also ignore state changes for MeetMe:nnn devstates and probably others. But
that is a separate bug that should be filed separately. (note - it may be
fixed in trunk, but given that it was chosen to backport this new feature
to 1.4 and given the widespread use of func_devstate.c backport to 1.4, it
would appear a little questionable not to address this shortcoming in 1.4). 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-05-25 13:50 p_lindheimer   Note Added: 0105374                          
======================================================================