[asterisk-bugs] [Asterisk 0016954]: [patch] [regression] local channel with '/bn' modifier does not optimize itself on built-in transfer

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16954 
====================================================================== 
Reported By:                frawd
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   16954
Category:                   Channels/chan_local
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     acknowledged
Asterisk Version:           1.6.2.2 
JIRA:                       SWP-1005 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.2 
SVN Revision (number only!): 250253 
Request Review:              
====================================================================== 
Date Submitted:             2010-03-03 12:03 CST
Last Modified:              2010-03-05 12:16 CST
====================================================================== 
Summary:                    [patch] [regression] local channel with '/bn'
modifier does not optimize itself on built-in transfer
Description: 
The 'b' modifier supposedly removes the Local channel out of the path on
transfer situations.

It works perfectly with SIP-style transfers (both attended and blind), but
does not work with the built-in atxfer and blindxfer (features.conf).

As a result, a "Local/" queue agent will still be marked as "In use" after
a builtin transfer for as long as the original caller does not hangup. I
believe it was why the 'b' modifier was added in the first place.
====================================================================== 

---------------------------------------------------------------------- 
 (0119032) frawd (reporter) - 2010-03-05 12:16
 https://issues.asterisk.org/view.php?id=16954#c119032 
---------------------------------------------------------------------- 
Attached an old patch I used in 1.4 (that didn't have the /b parameter).

It adds parameter /t (optimize_on_transfer) to Local channels and detects
GOTOs and MASQUERADES directly from chan_local.c in a very simple manner.

Worked very well in 1.4, also works in 1.6 (patch is for 1.6.2 branch).

I attach it to give an additional solution to the real devs that are going
to fix this bug! 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-03-05 12:16 frawd          Note Added: 0119032                          
======================================================================