From 47ba9f946d652b6f8f7f2e18d0207180fbbd86a6 Mon Sep 17 00:00:00 2001 From: Mark Michelson Date: Wed, 11 Feb 2009 23:11:33 +0000 Subject: [PATCH] Merged revisions 174948 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r174948 | mmichelson | 2009-02-11 17:03:08 -0600 (Wed, 11 Feb 2009) | 20 lines Fix odd "thank you" sound playing behavior in app_queue.c If someone has configured the queue to play an position or holdtime announcement, then it is odd and potentially unexpected to hear a "Thank you for your patience" sound when no position or holdtime was actually announced. This fixes the announcement so that the "thanks" sound is only played in the case that a position or holdtime was actually announced. There is a way that the "thank you" sound can be played without a position or holdtime, and that is to set announce-frequency to a value but keep announce-position and announce-holdtime both turned off. (closes issue #14227) Reported by: caspy Patches: 14227_v3.patch uploaded by putnopvut (license 60) Tested by: caspy ........ git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.1@174950 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- apps/app_queue.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/apps/app_queue.c b/apps/app_queue.c index 1d02b04a3d..2932813324 100644 --- a/apps/app_queue.c +++ b/apps/app_queue.c @@ -1834,6 +1834,7 @@ static int valid_exit(struct queue_ent *qe, char digit) static int say_position(struct queue_ent *qe, int ringing) { int res = 0, avgholdmins, avgholdsecs, announceposition = 0; + int say_thanks = 1; time_t now; /* Let minannouncefrequency seconds pass between the start of each position announcement */ @@ -1942,11 +1943,12 @@ static int say_position(struct queue_ent *qe, int ringing) if (res) goto playout; } - + } else if (qe->parent->announceholdtime && !qe->parent->announceposition) { + say_thanks = 0; } posout: - if (announceposition == 1){ + if (announceposition == 1 || say_thanks) { if (qe->parent->announceposition) { ast_verb(3, "Told %s in %s their queue position (which was %d)\n", qe->chan->name, qe->parent->name, qe->pos);