| Summary: | ASTERISK-08618: [patch] Add option to revert old ChanIsAvail() with 's' option behavior | ||
| Reporter: | Corey Frang (gnarf) | Labels: | |
| Date Opened: | 2007-01-20 04:23:11.000-0600 | Date Closed: | 2007-01-24 14:37:48.000-0600 |
| Priority: | Major | Regression? | No |
| Status: | Closed/Complete | Components: | Applications/app_chanisavail |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) app_chanisavail.c.patch | |
| Description: | The old chanisavail behaved slighly differently when checking channel status with the 's' option in use. The channel state was determined if a string match was made versus the in use channels in asterisk 1.2. This seemed to be the only difference between the functions ast_parse_device_state and ast_device_state. I brought back the old behavior with a new option 't' that doesn't call channel drivers to determine "in use" state. This may also solve http://bugs.digium.com/view.php?id=8301 and is related to http://bugs.digium.com/view.php?id=7433 . | ||
| Comments: | By: Corey Frang (gnarf) 2007-01-20 18:57:07.000-0600 I've also thought that the use of ChanIsAvail to perform this purpose is probally not very accurate naming. Perhaps "ChanExists()" or some other application name is the better place to put this code. There will probally always be a use for this type of check in a dialplan. By: Steve Murphy (murf) 2007-01-24 14:37:48.000-0600 Merged into trunk via r.52086. Added an entry in the UPGRADE.txt file for this new feature. Many thanks to gnarf for this code! | ||