[asterisk-bugs] [Asterisk 0018817]: MONITOR_FILENAME should be MIXMONITOR_FILENAME in documentation of MONITOR_EXEC

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=18817 
====================================================================== 
Reported By:                davidw
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   18817
Category:                   Documentation
Reproducibility:            N/A
Severity:                   text
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 307791 
Request Review:              
====================================================================== 
Date Submitted:             2011-02-15 11:23 CST
Last Modified:              2011-02-21 09:01 CST
====================================================================== 
Summary:                    MONITOR_FILENAME should be MIXMONITOR_FILENAME in
documentation of MONITOR_EXEC
Description: 
In queues.conf.sample, both the variable used to force the filename for
MixMonitor and the variable that gets set to the actual filename are shown
as MONITOR_FILENAME.  However, the latter is actually MIXMONITOR_FILENAME,
as set by this line in app_mixmonitor.c:

apps/app_mixmonitor.c:  pbx_builtin_setvar_helper(chan,
"MIXMONITOR_FILENAME", args.filename);

and as indicated in the API meta-data:

                                <variable name="MIXMONITOR_FILENAME">
                                        <para>Will contain the filename
used to record.</para>
                                </variable>

====================================================================== 

---------------------------------------------------------------------- 
 (0132244) elguero (reporter) - 2011-02-21 09:01
 https://issues.asterisk.org/view.php?id=18817#c132244 
---------------------------------------------------------------------- 
Not to be dense here.  Technically, it would appear that you are right. 
But, remember the user can choose to use Monitor or MixMonitor.  The
documentation actually is less confusing to the user because the variable
MONITOR_FILENAME can be used for both cases, unless I am missing something.
 Therefore, it actually provides for more flexibility since the user can
switch between Monitor or MixMonitor without having to change their dial
plan.

IMHO, the documentation is fine the way it is.  I hope someone else will
comment on this. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2011-02-21 09:01 elguero        Note Added: 0132244                          
======================================================================