Summary:ASTERISK-04204: [patch] Old style of app,appdata not parsed correctly when appdata contains a (
Reporter:Brian West (bkw918)Labels:
Date Opened:2005-05-17 10:36:16Date Closed:2011-06-07 14:10:50
Versions:Frequency of
Environment:Attachments:( 0) 20050517__fix_4306.diff.txt
Description:exten => 578,1,NoOp(${DB(TEST/blah)})
exten => 578,2,NoOp,${DB(TEST/blah)}

priority 1 works fine.
priority 2 results in:

No application 'NoOp,${DB' for extension (default, 578, 2)

Tiny parse error I say in pbx_config.c  I'll see if I have time to find it.. if not its here for documentation.

Either we fix this or depricate the abilitiy to not use () on app's
Comments:By: Tilghman Lesher (tilghman) 2005-05-17 10:48:41

Easy fix.  The problem is that parsing of parentheses took precedence over commas, instead of checking which came first.

By: Russell Bryant (russell) 2005-05-18 21:14:54

fixed in cvs head, thanks!

By: Digium Subversion (svnbot) 2008-01-15 15:35:23.000-0600

Repository: asterisk
Revision: 5721

U   trunk/pbx/pbx_config.c

r5721 | russell | 2008-01-15 15:35:22 -0600 (Tue, 15 Jan 2008) | 3 lines

check to see if a comma or an open paren came first when splitting the application
from the application arguments (bug ASTERISK-4204)