From bdd6e89dcd29c580fe6fd9c3ccea4e46ee5bfa7a Mon Sep 17 00:00:00 2001 From: Russell Bryant Date: Fri, 6 Apr 2007 18:59:42 +0000 Subject: [PATCH] Merged revisions 60521 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r60521 | russell | 2007-04-06 13:58:46 -0500 (Fri, 06 Apr 2007) | 16 lines Fix a few problems with SLA. (issue #9459, reported by francesco_r, fixed by me) * The original behavior was that if one station put a call on hold, another one picked it up, and then hung up, the code would still consider the call on hold by the first station, so the trunk would not be hung up. However, to better comply with what most people seem to expect it to behave, it will now hang up the trunk. * Fix a problem with "barge=no". This was only intended to prevent people from joining calls that are in progress. However, it also prevented other people from picking up a call that was on hold. This has been fixed. * When there are no active stations on a trunk and it is on hold, the code now indicates the HOLD and UNHOLD conditions to the trunk channel. This allows music on hold to be played to the trunk when it is on hold. ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@60522 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- apps/app_meetme.c | 33 +++++++++++++++++++++++++++------ 1 file changed, 27 insertions(+), 6 deletions(-) diff --git a/apps/app_meetme.c b/apps/app_meetme.c index f22772572a..16fb31b277 100644 --- a/apps/app_meetme.c +++ b/apps/app_meetme.c @@ -426,6 +426,9 @@ struct sla_trunk { /*! This option uses the values in the sla_hold_access enum and sets the * access control type for hold on this trunk. */ unsigned int hold_access:1; + /*! Whether this trunk is currently on hold, meaning that once a station + * connects to it, the trunk channel needs to have UNHOLD indicated to it. */ + unsigned int on_hold:1; }; struct sla_trunk_ref { @@ -3141,12 +3144,14 @@ static struct sla_trunk_ref *sla_find_trunk_ref_byname(const struct sla_station continue; if ( (trunk_ref->trunk->barge_disabled - && trunk_ref->state != SLA_TRUNK_STATE_IDLE) || + && trunk_ref->state == SLA_TRUNK_STATE_UP) || (trunk_ref->trunk->hold_stations && trunk_ref->trunk->hold_access == SLA_HOLD_PRIVATE && trunk_ref->state != SLA_TRUNK_STATE_ONHOLD_BYME) || - sla_check_station_hold_access(trunk_ref->trunk, station) ) + sla_check_station_hold_access(trunk_ref->trunk, station) ) + { trunk_ref = NULL; + } break; } @@ -3235,9 +3240,10 @@ static void *run_station(void *data) } trunk_ref->chan = NULL; if (ast_atomic_dec_and_test((int *) &trunk_ref->trunk->active_stations) && - !trunk_ref->trunk->hold_stations) { + trunk_ref->state != SLA_TRUNK_STATE_ONHOLD_BYME) { strncat(conf_name, "|K", sizeof(conf_name) - strlen(conf_name) - 1); admin_exec(NULL, conf_name); + trunk_ref->trunk->hold_stations = 0; sla_change_trunk_state(trunk_ref->trunk, SLA_TRUNK_STATE_IDLE, ALL_TRUNK_REFS, NULL); } @@ -3691,7 +3697,14 @@ static void sla_handle_hold_event(struct sla_event *event) event->station->name, event->trunk_ref->trunk->name); sla_change_trunk_state(event->trunk_ref->trunk, SLA_TRUNK_STATE_ONHOLD, INACTIVE_TRUNK_REFS, event->trunk_ref); - + + if (event->trunk_ref->trunk->active_stations == 1) { + /* The station putting it on hold is the only one on the call, so start + * Music on hold to the trunk. */ + event->trunk_ref->trunk->on_hold = 1; + ast_indicate(event->trunk_ref->trunk->chan, AST_CONTROL_HOLD); + } + ast_softhangup(event->trunk_ref->chan, AST_CAUSE_NORMAL); event->trunk_ref->chan = NULL; } @@ -4053,6 +4066,7 @@ static void *dial_trunk(void *data) sla_change_trunk_state(trunk_ref->trunk, SLA_TRUNK_STATE_IDLE, ALL_TRUNK_REFS, NULL); trunk_ref->trunk->chan = NULL; + trunk_ref->trunk->on_hold = 0; ast_dial_join(dial); ast_dial_destroy(dial); @@ -4174,7 +4188,12 @@ static int sla_station_exec(struct ast_channel *chan, void *data) } } - ast_atomic_fetchadd_int((int *) &trunk_ref->trunk->active_stations, 1); + if (ast_atomic_fetchadd_int((int *) &trunk_ref->trunk->active_stations, 1) == 0 && + trunk_ref->trunk->on_hold) { + trunk_ref->trunk->on_hold = 0; + ast_indicate(trunk_ref->trunk->chan, AST_CONTROL_UNHOLD); + } + snprintf(conf_name, sizeof(conf_name), "SLA_%s", trunk_ref->trunk->name); ast_set_flag(&conf_flags, CONFFLAG_QUIET | CONFFLAG_MARKEDEXIT | CONFFLAG_PASS_DTMF | CONFFLAG_SLA_STATION); @@ -4187,9 +4206,10 @@ static int sla_station_exec(struct ast_channel *chan, void *data) } trunk_ref->chan = NULL; if (ast_atomic_dec_and_test((int *) &trunk_ref->trunk->active_stations) && - !trunk_ref->trunk->hold_stations) { + trunk_ref->state != SLA_TRUNK_STATE_ONHOLD_BYME) { strncat(conf_name, "|K", sizeof(conf_name) - strlen(conf_name) - 1); admin_exec(NULL, conf_name); + trunk_ref->trunk->hold_stations = 0; sla_change_trunk_state(trunk_ref->trunk, SLA_TRUNK_STATE_IDLE, ALL_TRUNK_REFS, NULL); } @@ -4276,6 +4296,7 @@ static int sla_trunk_exec(struct ast_channel *chan, void *data) dispose_conf(conf); conf = NULL; trunk->chan = NULL; + trunk->on_hold = 0; pbx_builtin_setvar_helper(chan, "SLATRUNK_STATUS", "SUCCESS");