[asterisk-bugs] [Asterisk 0018650]: comebacktoorigin=no returns to flattened channel, not extensions s, for parked call timeout

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18650 
====================================================================== 
Reported By:                davidw
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18650
Category:                   Features/Parking
Reproducibility:            have not tried
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.2 
SVN Revision (number only!): 302693 
Request Review:              
====================================================================== 
Date Submitted:             2011-01-20 12:12 CST
Last Modified:              2011-01-20 12:21 CST
====================================================================== 
Summary:                    comebacktoorigin=no returns to flattened channel,
not extensions s, for parked call timeout
Description: 
There appears to be a cut and paste error in the parked call timeout code,
when combacktoorigin=no and there is a peer channel.  Instead of returning
to (parkedcallstimeout, s, 1), as indicated by a warning, output in 1.6.2
branch, it actually returns to (parkedcallstimeout, <peername_flat>, 1).

This may work most of the time because of fallback processing in the PBX,
but is clearly a cut and paste error from the comebacktoorigin=yes branch.
====================================================================== 

---------------------------------------------------------------------- 
 (0130806) davidw (reporter) - 2011-01-20 12:21
 https://issues.asterisk.org/view.php?id=18650#c130806 
---------------------------------------------------------------------- 
It looks like the trunk .conf.sample describes the de facto behaviour, even
though it isn't clear why using the flattened peer name would be any use to
anyone.

However, the 1.6.2 branch .conf.sample describes it as returning it to s,
1 - which I suspect was always the real intent.

Whilst I think the right change here is to roll back the .conf.sample text
in trunk and fix the extension in the code in both trunk and 1.6.2, at a
minimum, one needs to back port the trunk .conf.sample change and remove
the warning in 1.6.2. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-01-20 12:21 davidw         Note Added: 0130806                          
======================================================================