[asterisk-bugs] [Asterisk 0014014]: [patch] DAHDI group dials/members broken with AddQueueMember / queue member add

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14014 
====================================================================== 
Reported By:                kebl0155
Assigned To:                putnopvut
====================================================================== 
Project:                    Asterisk
Issue ID:                   14014
Category:                   Applications/app_queue
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     closed
Target Version:             1.6.0.4
Asterisk Version:           1.6.0.1 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2008-12-03 09:44 CST
Last Modified:              2009-01-20 18:34 CST
====================================================================== 
Summary:                    [patch] DAHDI group dials/members broken with
AddQueueMember / queue member add
Description: 
We have Queues, and we dial the agent's phones directly over ZAP (now
DAHDI).  The agents are dynamic, and we used to add and remove them to/from
the queue using:

add queue member ZAP/r2/0008 to CustServ

Having upgraded to DAHDI and Asterisk 1.6.0.1, we now use:

add queue member DAHDI/r2/0008 to CustServ

This does add the member to the queue, but after a single call is placed
to the agent, the agent becomes invalid and cannot receive further calls.

Furthermore, the member cannot be removed from the queue without restart.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
duplicate of        0014020 Asterisk cuts a piece of the dynamic me...
has duplicate       0013949 IAX2 Channel queue members lose their e...
has duplicate       0014124 queues fail to recognize already logged...
has duplicate       0014280 Queue member location is altered
====================================================================== 

---------------------------------------------------------------------- 
 (0098291) svnbot (reporter) - 2009-01-20 18:34
 http://bugs.digium.com/view.php?id=14014#c98291 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 169612

_U  branches/1.6.0/
U   branches/1.6.0/apps/app_queue.c

------------------------------------------------------------------------
r169612 | mmichelson | 2009-01-20 18:34:48 -0600 (Tue, 20 Jan 2009) | 30
lines

Merged revisions 169611 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

........
r169611 | mmichelson | 2009-01-20 18:33:32 -0600 (Tue, 20 Jan 2009) | 22
lines

Fix device state parsing issues for channel names with multiple slashes

The fix being applied is a bit different for trunk and the 1.6.X branches.
For trunk, we only wish to strip off the characters beyond the second
slash
if the channel is a Local channel (i.e. we are removing the /n from the
device
name). Other channel technologies with multiple slashes (e.g. DAHDI) need
the
information after the second slash in order to get the proper device state
information.

In addition to this fix, the 1.6.X branches are receiving a much more
important
fix as well. The problem in 1.6.X is that the member's device name was
being directly
changed instead of having a copy changed. This meant that we would strip
off the
second slash and trailing characters and then leave the member's device
name like
that permanently thereafter.

(closes issue http://bugs.digium.com/view.php?id=14014)
Reported by: kebl0155
Patches:
      14014_number2.patch uploaded by putnopvut (license 60)
Tested by: kebl0155


........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=169612 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-20 18:34 svnbot         Checkin                                      
2009-01-20 18:34 svnbot         Note Added: 0098291                          
======================================================================