[svn-commits] irroot: branch 10 r335014 - in /branches/10: ./ apps/ funcs/

SVN commits to the Digium repositories svn-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 svn-commits mailing list