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 /apps | |
| 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
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/app_mixmonitor.c | 21 |
1 files changed, 21 insertions, 0 deletions
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)) { |