Summary:ASTERISK-04178: [patch] convert app_chanisavail to a dialplan function
Reporter:Russell Bryant (russell)Labels:
Date Opened:2005-05-15 13:40:21Date Closed:2011-06-07 14:05:18
Versions:Frequency of
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.


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.