[Asterisk-bugs] [Asterisk 0010185]: [patch]: Added automixmonitor feature / ability to use mixmonitor to record queue conversations on demand.

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=10185 
====================================================================== 
Reported By:                xmarksthespot
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   10185
Category:                   Resources/res_features
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     new
Asterisk Version:            1.4.6 
SVN Branch (only for SVN checkouts, not tarball releases): N/A  
SVN Revision (number only!):  
Disclaimer on File?:        Yes 
Request Review:              
====================================================================== 
Date Submitted:             07-11-2007 16:21 CDT
Last Modified:              07-12-2007 11:36 CDT
====================================================================== 
Summary:                    [patch]: Added automixmonitor feature / ability to
use mixmonitor to record queue conversations on demand.
Description: 
I needed to record queue conversations again on demand only. I found out
about the option 'w' in app_queue, but found out it recorded using monitor
and produced two sound files, as everyone knows.

There was also the automon (*1 by default) feature, but that didn't help
either, as I need it to work on a queue and produce only one file.

I set out to find another way to do this, which led me to code this. What
this patch does is add a new feature: automixmon, which is auto mix
monitor. Now you can specify a new option in app_queue, which is called
'x', which lets the callee dial a string to get the conversation recorded
by MixMonitor instead of monitor, and thus get only one file.

I also added the 'X' function, just for the sake of being able to do all
that Monitor does, and finally do away with it.
====================================================================== 

---------------------------------------------------------------------- 
 xmarksthespot - 07-12-07 11:36  
---------------------------------------------------------------------- 
Todo:

<STRIKE>-Add new channel variables like TOUCH_MONITOR and
TOUCH_MONITOR_FORMAT to separate its function from Monitor. I don't know
where to add new channel variables yet, so if you do, let me know. Probably
they should be called TOUCH_MIXMONITOR and TOUCH_MIXMONITOR_FORMAT. For the
time being, TOUCH_MONITOR and TOUCH_MONITOR_FORMAT apply the same way to
automon as they do to the new automixmon.</STRIKE> Done! It now uses
TOUCH_MIXMONITOR and TOUCH_MIXMONITOR_FORMAT and some other variable for
the output! We're moving forward!

-Verify if MixMonitor is already running at the beginning of the
builtin_automixmonitor()!!! Really important, otherwise you can sort of
"recursively" MixMonitor a channel!
MixMonitor(MixMonitor(MixMonitor(MixMonitor(... ...)))). I definitely don't
want that.

-Enable a keypress to stop the MixMonitor. Maybe the same when pressed a
second time. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
07-12-07 11:36  xmarksthespot  Note Added: 0067314                          
======================================================================