[asterisk-bugs] [Asterisk 0012755]: [patch] parking space number is announced to the parkee instead of the parker

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12755 
====================================================================== 
Reported By:                nreinartz
Assigned To:                jpeeler
====================================================================== 
Project:                    Asterisk
Issue ID:                   12755
Category:                   Resources/res_features
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Asterisk Version:           1.4.20.1 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-05-29 10:37 CDT
Last Modified:              2008-08-29 12:58 CDT
====================================================================== 
Summary:                    [patch] parking space number is announced to the
parkee instead of the parker
Description: 
If call parking is done by res/res_features.c, park_call_exec() the parking
space number is announced to the parkee but it should be announced to the
parker.
Revision 97529 (
http://svn.digium.com/view/asterisk/branches/1.4/res/res_features.c?view=log#rev97529
   ) introduced a fix and stated out "Fix saying the parking space number
to the caller doing the parking ..." but its not working as described.
Until revision 97529 the parking space wasn't announced. Now its
announced, but it's announced to the parkee not to the parker.

Most of the channel drivers do parking directly by ast_park_call() if the
number matches with ast_parking_ext(). If the number doesn't match,
ast_async_goto() is called which goes through the dial plan.
ast_async_goto() calls the registered application park_call_exec().
Unfortunately this function doesn't include the peer channel as parameter.
Thus it's not possible to announce the parking space number to the parker.
A design with both channels as parameters would be great.

I tested with some snom phones using the park orbit function. Usually
parking is done by ast_park_call(), I had to use a special configuration,
so that parking is done by park_call_exec(). See attached file
"configuration.txt".

The attached patch doesn't solve the whole problem, but with this patch
the parker is at least called back after parking timeout. 
I tested this for chan_sip, it works for me. The patch also undoes the fix
of revision 97529.
====================================================================== 

---------------------------------------------------------------------- 
 (0091920) jpeeler (administrator) - 2008-08-29 12:58
 http://bugs.digium.com/view.php?id=12755#c91920 
---------------------------------------------------------------------- 
I've looked at trying to make this work and have determined it is just not
going to be possible. Park was not really designed to be used directly in
the dialplan with a blind transfer. If you really want to do a blind
transfer to park the call, use the registered parking extension (parkext)
from features.conf which is handled appropriately.

As a partial compromise, this bug inspired the addition of a new argument
to the Park application. The new option 's' will silence playing the
parking number. It'll be in 1.6.1 once it's released. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-08-29 12:58 jpeeler        Note Added: 0091920                          
======================================================================