[asterisk-commits] tilghman: trunk r188210 - /trunk/main/pbx.c

[SVN commits to the Asterisk project]

Author: tilghman
Date: Tue Apr 14 00:45:13 2009
New Revision: 188210

URL: http://svn.digium.com/svn-view/asterisk?view=rev&rev=188210
Log:
As suggested by Russell, warn users when their dialplan arguments contain pipes, but not commas.

Modified:
    trunk/main/pbx.c

Modified: trunk/main/pbx.c
URL: http://svn.digium.com/svn-view/asterisk/trunk/main/pbx.c?view=diff&rev=188210&r1=188209&r2=188210
==============================================================================
--- trunk/main/pbx.c (original)
+++ trunk/main/pbx.c Tue Apr 14 00:45:13 2009
@@ -1339,6 +1339,11 @@
 	c->data = data;
 	if (app->module)
 		u = __ast_module_user_add(app->module, c);
+	if (!ast_strlen_zero(data) && strchr(data, '|') && !strchr(data, ',')) {
+		ast_log(LOG_WARNING, "The application delimiter is now the comma, not "
+			"the pipe.  Did you forget to convert your dialplan?  (%s(%s))\n",
+			app->name, (char *) data);
+	}
 	res = app->execute(c, S_OR(data, ""));
 	if (app->module && u)
 		__ast_module_user_remove(app->module, u);