| 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 |
| Priority: | Major | Regression? | No |
| Status: | Closed/Complete | Components: | Functions/General |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| 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. | ||