|Summary:||ASTERISK-04178: [patch] convert app_chanisavail to a dialplan function|
|Reporter:||Russell Bryant (russell)||Labels:|
|Date Opened:||2005-05-15 13:40:21||Date Closed:||2011-06-07 14:05:18|
|Environment:||Attachments:||( 0) func_chanisavail.rev1.txt|
|Description:||app_chanisavail sets 3 variables, AVAILCHAN, AVAILORIGCHAN, and AVAILSTATUS.|
The changes I have here so far are taking out the jumps to n+101 and instead of setting AVAILSTATUS, that is the return value for the function.
There is an option for app_chanisavail to consider the channel not available if it is in use at all, even if it can take another call. In app_chanisavail, this just made it so AVAILCHAN and AVAILORIGCHAN were not set. However, for the function, I have made it so the returned AVAILSTATUS is 0, instead of whatever it got from the channel driver, as was the case in app_chanisavail.
****** ADDITIONAL INFORMATION ******
disclaimer is on file
|Comments:||By: Russell Bryant (russell) 2005-05-15 14:06:24|
rev2 uploaded ...
- removed some extra words from the description
- removed the pointless double call to strsep. This is also in app_chanisavail, but I don't know why.
By: Russell Bryant (russell) 2005-05-15 14:08:22
Kevin mentioned on IRC that maybe we should have this return all of the information instead of setting channel variables.
Any opinions out there?
By: Kevin P. Fleming (kpfleming) 2005-05-15 18:13:24
My suggestion was for the result of this function to be:
<status>|<channel name>|<original channel name>
corresponding to the three variables that ChanIsAvail() sets.
By: Russell Bryant (russell) 2005-05-23 15:37:43
I'm not going to be a jerk and leave this up if I'm not working on it. I'll re-open if I get something done that I am happy with.