[asterisk-commits] irroot: branch 10 r335014 - in /branches/10: ./ apps/ funcs/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Fri Sep 9 02:23:58 CDT 2011
Author: irroot
Date: Fri Sep 9 02:23:53 2011
New Revision: 335014
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=335014
Log:
Move code for VALID_EXTEN from app_readexten to func_dialplan
Mark VALID_EXTEN deprecated.
Review: https://reviewboard.asterisk.org/r/1396/
Modified:
branches/10/CHANGES
branches/10/apps/app_readexten.c
branches/10/funcs/func_dialplan.c
Modified: branches/10/CHANGES
URL: http://svnview.digium.com/svn/asterisk/branches/10/CHANGES?view=diff&rev=335014&r1=335013&r2=335014
==============================================================================
--- branches/10/CHANGES (original)
+++ branches/10/CHANGES Fri Sep 9 02:23:53 2011
@@ -121,6 +121,7 @@
for a given string to replace with another string as many times as the
user specifies or just throughout the whole string.
* Added option to CHANNEL(pickupgroup) allow reading and setting the pickupgroup of channel.
+ * Mark VALID_EXTEN() deprecated in favor of DIALPLAN_EXISTS()
libpri channel driver (chan_dahdi) DAHDI changes
--------------------------
Modified: branches/10/apps/app_readexten.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/apps/app_readexten.c?view=diff&rev=335014&r1=335013&r2=335014
==============================================================================
--- branches/10/apps/app_readexten.c (original)
+++ branches/10/apps/app_readexten.c Fri Sep 9 02:23:53 2011
@@ -96,24 +96,6 @@
</variablelist>
</description>
</application>
- <function name="VALID_EXTEN" language="en_US">
- <synopsis>
- Determine whether an extension exists or not.
- </synopsis>
- <syntax>
- <parameter name="context">
- <para>Defaults to the current context</para>
- </parameter>
- <parameter name="extension" required="true" />
- <parameter name="priority">
- <para>Priority defaults to <literal>1</literal>.</para>
- </parameter>
- </syntax>
- <description>
- <para>Returns a true value if the indicated <replaceable>context</replaceable>,
- <replaceable>extension</replaceable>, and <replaceable>priority</replaceable> exist.</para>
- </description>
- </function>
***/
enum readexten_option_flags {
@@ -280,57 +262,15 @@
return status[0] == 'H' ? -1 : 0;
}
-static int acf_isexten_exec(struct ast_channel *chan, const char *cmd, char *parse, char *buffer, size_t buflen)
-{
- int priority_int;
- AST_DECLARE_APP_ARGS(args,
- AST_APP_ARG(context);
- AST_APP_ARG(extension);
- AST_APP_ARG(priority);
- );
-
- AST_STANDARD_APP_ARGS(args, parse);
-
- if (ast_strlen_zero(args.context))
- args.context = chan->context;
-
- if (ast_strlen_zero(args.extension)) {
- ast_log(LOG_WARNING, "Syntax: VALID_EXTEN([<context>],<extension>[,<priority>]) - missing argument <extension>!\n");
- return -1;
- }
-
- if (ast_strlen_zero(args.priority))
- priority_int = 1;
- else
- priority_int = atoi(args.priority);
-
- if (ast_exists_extension(chan, args.context, args.extension, priority_int,
- S_COR(chan->caller.id.number.valid, chan->caller.id.number.str, NULL))) {
- ast_copy_string(buffer, "1", buflen);
- } else {
- ast_copy_string(buffer, "0", buflen);
- }
-
- return 0;
-}
-
-static struct ast_custom_function acf_isexten = {
- .name = "VALID_EXTEN",
- .read = acf_isexten_exec,
-};
-
static int unload_module(void)
{
int res = ast_unregister_application(app);
- res |= ast_custom_function_unregister(&acf_isexten);
-
- return res;
+ return res;
}
static int load_module(void)
{
int res = ast_register_application_xml(app, readexten_exec);
- res |= ast_custom_function_register(&acf_isexten);
return res;
}
Modified: branches/10/funcs/func_dialplan.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/funcs/func_dialplan.c?view=diff&rev=335014&r1=335013&r2=335014
==============================================================================
--- branches/10/funcs/func_dialplan.c (original)
+++ branches/10/funcs/func_dialplan.c Fri Sep 9 02:23:53 2011
@@ -51,11 +51,29 @@
<para>This function returns <literal>1</literal> if the target exits. Otherwise, it returns <literal>0</literal>.</para>
</description>
</function>
-
+ <function name="VALID_EXTEN" language="en_US">
+ <synopsis>
+ Determine whether an extension exists or not.
+ </synopsis>
+ <syntax>
+ <parameter name="context">
+ <para>Defaults to the current context</para>
+ </parameter>
+ <parameter name="extension" required="true" />
+ <parameter name="priority">
+ <para>Priority defaults to <literal>1</literal>.</para>
+ </parameter>
+ </syntax>
+ <description>
+ <para>Returns a true value if the indicated <replaceable>context</replaceable>,
+ <replaceable>extension</replaceable>, and <replaceable>priority</replaceable> exist.</para>
+ <warning><para>This function has been deprecated in favor of the <literal>DIALPLAN_EXISTS()</literal> function</para></warning>
+ </description>
+ </function>
***/
-static int isexten_function_read(struct ast_channel *chan, const char *cmd, char *data,
- char *buf, size_t len)
+static int isexten_function_read(struct ast_channel *chan, const char *cmd, char *data,
+ char *buf, size_t len)
{
char *parse;
AST_DECLARE_APP_ARGS(args,
@@ -102,7 +120,43 @@
ast_log(LOG_ERROR, "Invalid arguments provided to DIALPLAN_EXISTS\n");
return -1;
}
-
+
+ return 0;
+}
+
+static int acf_isexten_exec(struct ast_channel *chan, const char *cmd, char *parse, char *buffer, size_t buflen)
+{
+ int priority_int;
+ AST_DECLARE_APP_ARGS(args,
+ AST_APP_ARG(context);
+ AST_APP_ARG(extension);
+ AST_APP_ARG(priority);
+ );
+
+ AST_STANDARD_APP_ARGS(args, parse);
+
+ if (ast_strlen_zero(args.context)) {
+ args.context = chan->context;
+ }
+
+ if (ast_strlen_zero(args.extension)) {
+ ast_log(LOG_WARNING, "Syntax: VALID_EXTEN([<context>],<extension>[,<priority>]) - missing argument <extension>!\n");
+ return -1;
+ }
+
+ if (ast_strlen_zero(args.priority)) {
+ priority_int = 1;
+ } else {
+ priority_int = atoi(args.priority);
+ }
+
+ if (ast_exists_extension(chan, args.context, args.extension, priority_int,
+ S_COR(chan->caller.id.number.valid, chan->caller.id.number.str, NULL))) {
+ ast_copy_string(buffer, "1", buflen);
+ } else {
+ ast_copy_string(buffer, "0", buflen);
+ }
+
return 0;
}
@@ -112,14 +166,23 @@
.read_max = 2,
};
+static struct ast_custom_function acf_isexten = {
+ .name = "VALID_EXTEN",
+ .read = acf_isexten_exec,
+};
+
static int unload_module(void)
{
- return ast_custom_function_unregister(&isexten_function);
+ int res = ast_custom_function_unregister(&isexten_function);
+ res |= ast_custom_function_unregister(&acf_isexten);
+ return res;
}
static int load_module(void)
{
- return ast_custom_function_register(&isexten_function);
+ int res = ast_custom_function_register(&isexten_function);
+ res |= ast_custom_function_register(&acf_isexten);
+ return res;
}
AST_MODULE_INFO_STANDARD(ASTERISK_GPL_KEY, "Dialplan Context/Extension/Priority Checking Functions");
More information about the asterisk-commits
mailing list