Summary: | ASTERISK-21306: set FEATURE(parkingtime) is not inherited by child channels | ||
Reporter: | David Cunningham (dcunningham) | Labels: | |
Date Opened: | 2013-03-21 01:51:37 | Date Closed: | 2013-07-31 16:26:20 |
Priority: | Major | Regression? | |
Status: | Closed/Complete | Components: | Applications/app_parkandannounce |
Versions: | 11.2.1 | Frequency of Occurrence | |
Related Issues: | |||
Environment: | CentOS release 5.8 32 bit | Attachments: | |
Description: | If you set FEATURE(parkingtime), it doesn't appear to be inherited by child channels.
It you try using "__FEATURE(parkingtime)" it gives an error that "Function __FEATURE not registered". Please let us know if you need more information. | ||
Comments: | By: Russell Bryant (russell) 2013-03-21 10:18:37.100-0500 This is indeed the way this feature was written. I don't think we would want it to *always* be inherited. Optional inheritance turned on by a flag or something seems like a reasonable enhancement, though. By: Richard Mudgett (rmudgett) 2013-03-21 11:01:12.268-0500 Functions are *not* channel variables. By: Russell Bryant (russell) 2013-03-21 11:40:23.120-0500 Sure, but this is implemented as a datastore underneath, which could be inherited if that was the desired behavior. By: David Cunningham (dcunningham) 2013-03-21 21:48:07.442-0500 This would be a good enhancement. For now we will work around it in extensions.conf. Thanks. By: Russell Bryant (russell) 2013-03-25 21:41:45.639-0500 Patch in progress here: https://reviewboard.asterisk.org/r/2415/ By: David Cunningham (dcunningham) 2013-03-25 22:26:50.972-0500 Hi Russell, That looks great. What version of Asterisk will it go in? By: Matt Jordan (mjordan) 2013-03-26 07:18:38.225-0500 Just curious, why not use a pre-dial routine to set the FEATURE on the child channel at the time of its creation? https://wiki.asterisk.org/wiki/display/AST/Pre-Dial+Handlers By: Russell Bryant (russell) 2013-03-26 08:11:00.367-0500 Indeed, that works, at least for the simpler cases. Some downsides .. 1) pre-dial routines result in a more complex dialplan 2) pre-dial routines only work for Dial() and FollowMe() This is just a more brute force single-line addition to an extension that says "inherit this stuff forever". |