Update MeetMe p and X option documentation when interacting with the s option.

ASTERISK-12175 changed the p and X options to not interfere with the s
option when they are used together.  It makes more sense for the s option
to have priority for the DTMF '*' key since it cannot change its
activation code.  Otherwise, you could not use option s with the p or X
options.

JIRA AST-671


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@340470 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Richard Mudgett
2011-10-12 17:49:19 +00:00
parent 35fcb785af
commit 4051609b5c

View File

@@ -142,10 +142,13 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
</option>
<option name="p" hasparams="optional">
<para>Allow user to exit the conference by pressing <literal>#</literal> (default)
or any of the defined keys. If keys contain <literal>*</literal> this will override
option <literal>s</literal>. The key used is set to channel variable
or any of the defined keys. The key used is set to channel variable
<variable>MEETME_EXIT_KEY</variable>.</para>
<argument name="keys" required="true" />
<note>
<para>Option <literal>s</literal> has priority for <literal>*</literal>
since it cannot change its activation code.</para>
</note>
</option>
<option name="P">
<para>Always prompt for the pin even if it is specified.</para>
@@ -180,6 +183,10 @@ ASTERISK_FILE_VERSION(__FILE__, "$Revision$")
<para>Allow user to exit the conference by entering a valid single digit
extension <variable>MEETME_EXIT_CONTEXT</variable> or the current context
if that variable is not defined.</para>
<note>
<para>Option <literal>s</literal> has priority for <literal>*</literal>
since it cannot change its activation code.</para>
</note>
</option>
<option name="1">
<para>Do not play message when first person enters</para>
@@ -3535,7 +3542,10 @@ static int conf_run(struct ast_channel *chan, struct ast_conference *conf, struc
}
conf_flush(fd, chan);
/* Since this option could absorb DTMF meant for the previous (menu), we have to check this one last */
/*
* Since options using DTMF could absorb DTMF meant for the
* conference menu, we have to check them after the menu.
*/
} else if ((f->frametype == AST_FRAME_DTMF) && ast_test_flag64(confflags, CONFFLAG_EXIT_CONTEXT) && ast_exists_extension(chan, exitcontext, dtmfstr, 1, "")) {
if (ast_test_flag64(confflags, CONFFLAG_PASS_DTMF)) {
conf_queue_dtmf(conf, user, f);