Summary: | ASTERISK-04585: [patch] The flag "transfertobusy" doesn't work. | ||
Reporter: | Thomas (thomas andrews) | Labels: | |
Date Opened: | 2005-07-14 03:27:44 | Date Closed: | 2008-01-15 15:41:31.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) dial_busy.patch | |
Description: | chan_zap.c revision 1.462 was patched similarly to this: if( p->owner->_state == AST_STATE_BUSY ) Unfortunately however, p->owner->_state never gets set to AST_STATE_BUSY, thus making these sections of the code inoperable. This patch changes the state to AST_STATE_BUSY in two places: 1. When the "Busy" built-in function is used, and 2. In app_dial.c where "Everyone is busy/congested" | ||
Comments: | By: Kevin P. Fleming (kpfleming) 2005-07-15 11:56:16 Fixed in CVS HEAD, although I did not modify app_dial, since deciding whether the channel is busy or not would then fall to the dialplan writer. I also added AST_STATE_BUSY setting to pbx_builtin_congestion, since it would also apply. Thanks for the detective work! By: Digium Subversion (svnbot) 2008-01-15 15:41:30.000-0600 Repository: asterisk Revision: 6136 U trunk/pbx.c ------------------------------------------------------------------------ r6136 | kpfleming | 2008-01-15 15:41:29 -0600 (Tue, 15 Jan 2008) | 2 lines ensure that channels that are busy/congested are marked AST_STATE_BUSY (bug ASTERISK-4585) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=6136 By: Digium Subversion (svnbot) 2008-01-15 15:41:31.000-0600 Repository: asterisk Revision: 6137 U trunk/channels/chan_zap.c ------------------------------------------------------------------------ r6137 | kpfleming | 2008-01-15 15:41:30 -0600 (Tue, 15 Jan 2008) | 2 lines various code formatting changes and cleanups related to bug ASTERISK-4585 ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=6137 |