[asterisk-bugs] [Asterisk 0016586]: [patch] features.conf auto-included park function sometimes included junk in Park() command

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=16586 
====================================================================== 
Reported By:                DLNoah
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   16586
Category:                   Applications/app_parkandannounce
Reproducibility:            random
Severity:                   major
Priority:                   normal
Status:                     acknowledged
Asterisk Version:           1.6.2.0 
JIRA:                       SWP-707 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-01-12 09:20 CST
Last Modified:              2010-01-19 11:20 CST
====================================================================== 
Summary:                    [patch] features.conf auto-included park function
sometimes included junk in Park() command
Description: 
When using the included parking context from features.conf, or using an
Aastra phone with their built-in park softkey (which appears to call the
same built-in park, from what I can tell) in any version of 1.6.x that I've
used (including 1.6.0.9, 1.6.0.15, 1.6.1.0, 1.6.2.0), the Park() command
will sometimes but inconsistently include extra information--most commonly
a channel that was dialed by the parking phone.  In 1.6.0.x, the extra
information is simply ignored as Park() doesn't take parameters.  In
1.6.1.x and 1.6.2.x, Park interprets this information numerically (usually
as 0, since they generally start with a letter), which causes the call to
timeout immediately.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0016406 Calls transfered to parking lot immedia...
====================================================================== 

---------------------------------------------------------------------- 
 (0116918) DLNoah (reporter) - 2010-01-19 11:20
 https://issues.asterisk.org/view.php?id=16586#c116918 
---------------------------------------------------------------------- 
I applied the two patches above to 1.6.0.9, and placed about 30 calls to
our parking extension, including: direct calls while I wasn't on a call and
hadn't received a call in a while, direct calls while I wasn't on a call
and had just received a call, and actual attended transfers of a live call.
 In all of these calls, Park worked as expected, e.g.:
    -- Executing [170 at default:1] Park("SIP/100-b7dce1f8", "") in new
stack

Before, the extra stuff in the Park() application would usually show up
within 3-5 calls, especially if I had just received a call.  I haven't
tested against 1.6.1.x or 1.6.2.x yet, but this seems to fix the behavior I
reported. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-01-19 11:20 DLNoah         Note Added: 0116918                          
======================================================================