diff options
author | Michael L. Young <elgueromexicano@gmail.com> | 2014-07-22 20:01:42 +0000 |
---|---|---|
committer | Michael L. Young <elgueromexicano@gmail.com> | 2014-07-22 20:01:42 +0000 |
commit | 20cb961b3ed3d5890a88a0cac6c9ba5dbda6570e (patch) | |
tree | 565409faffe32253a6a771f567066f1ad8828a72 | |
parent | b4a681684d2fee800add540ede399bf45915fe49 (diff) |
apps/app_mixmonitor: Add Options To Play Beep At Start Or Stop
We have a new periodic beep feature but sometimes a user needs some sort of
feedback, without the need to have a periodic beep during the recording, to let
them know that MixMonitor started recording or ended the recording. The use
case where this patch is being used is when using Dynamic Features to start and
end MixMonitor.
This patch adds an option to play a beep when MixMonitor starts and an option to
play a beep when MixMonitor ends.
ASTERISK-24051 #close
Reported by: Michael L. Young
patches:
mixmonitor-play-beep-start-stop.diff
uploaded by Michael L. Young (license 5026)
Review: https://reviewboard.asterisk.org/r/3820/
git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@419238 65c4cc65-6c06-0410-ace0-fbb531ad65f3
-rw-r--r-- | CHANGES | 8 | ||||
-rw-r--r-- | apps/app_mixmonitor.c | 21 |
2 files changed, 28 insertions, 1 deletions
@@ -182,6 +182,13 @@ res_config_pgsql for res_config_pgsql via the dbappname configuration setting in res_pgsql.conf. +MixMonitor +------------------ + * New options to play a beep when starting a recording and stopping a recording + have been added. The option "p" will play a beep to the channel that starts + the recording. The option "P" will play a beep to the channel that stops the + recording. + ------------------------------------------------------------------------------ --- Functionality changes from Asterisk 12.4.0 to Asterisk 12.5.0 ------------ ------------------------------------------------------------------------------ @@ -367,7 +374,6 @@ MixMonitor * A new option, B(), has been added that will turn on a periodic beep while the call is being recorded. - Channel Drivers ------------------------- diff --git a/apps/app_mixmonitor.c b/apps/app_mixmonitor.c index 1e7833cea..52128942a 100644 --- a/apps/app_mixmonitor.c +++ b/apps/app_mixmonitor.c @@ -120,6 +120,12 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$") <argument name="chanvar" required="true" /> <para>Stores the MixMonitor's ID on this channel variable.</para> </option> + <option name="p"> + <para>Play a beep on the channel that starts the recording.</para> + </option> + <option name="P"> + <para>Play a beep on the channel that stops the recording.</para> + </option> <option name="m"> <argument name="mailbox" required="true" /> <para>Create a copy of the recording as a voicemail in the indicated <emphasis>mailbox</emphasis>(es) @@ -326,6 +332,8 @@ enum mixmonitor_flags { MUXFLAG_UID = (1 << 9), MUXFLAG_VMRECIPIENTS = (1 << 10), MUXFLAG_BEEP = (1 << 11), + MUXFLAG_BEEP_START = (1 << 12), + MUXFLAG_BEEP_STOP = (1 << 13) }; enum mixmonitor_args { @@ -344,6 +352,8 @@ AST_APP_OPTIONS(mixmonitor_opts, { AST_APP_OPTION('a', MUXFLAG_APPEND), AST_APP_OPTION('b', MUXFLAG_BRIDGED), AST_APP_OPTION_ARG('B', MUXFLAG_BEEP, OPT_ARG_BEEP_INTERVAL), + AST_APP_OPTION('p', MUXFLAG_BEEP_START), + AST_APP_OPTION('P', MUXFLAG_BEEP_STOP), AST_APP_OPTION_ARG('v', MUXFLAG_READVOLUME, OPT_ARG_READVOLUME), AST_APP_OPTION_ARG('V', MUXFLAG_WRITEVOLUME, OPT_ARG_WRITEVOLUME), AST_APP_OPTION_ARG('W', MUXFLAG_VOLUME, OPT_ARG_VOLUME), @@ -733,6 +743,11 @@ static void *mixmonitor_thread(void *obj) "File: %s\r\n", ast_channel_name(mixmonitor->autochan->chan), mixmonitor->filename); + ast_channel_lock(mixmonitor->autochan->chan); + if (ast_test_flag(mixmonitor, MUXFLAG_BEEP_STOP)) { + ast_stream_and_wait(mixmonitor->autochan->chan, "beep", ""); + } + ast_channel_unlock(mixmonitor->autochan->chan); ast_audiohook_unlock(&mixmonitor->audiohook); @@ -805,6 +820,11 @@ static int setup_mixmonitor_ds(struct mixmonitor *mixmonitor, struct ast_channel return -1; } + ast_channel_lock(mixmonitor->autochan->chan); + if (ast_test_flag(mixmonitor, MUXFLAG_BEEP_START)) { + ast_stream_and_wait(mixmonitor->autochan->chan, "beep", ""); + } + ast_channel_unlock(mixmonitor->autochan->chan); mixmonitor_ds->samp_rate = 8000; mixmonitor_ds->audiohook = &mixmonitor->audiohook; @@ -1164,6 +1184,7 @@ static int stop_mixmonitor_full(struct ast_channel *chan, const char *data) if (!ast_channel_datastore_remove(chan, datastore)) { ast_datastore_free(datastore); } + ast_channel_unlock(chan); if (!ast_strlen_zero(beep_id)) { |