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:16 | Date Closed: | 2011-06-07 14:10:50 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
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 /b | ||
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) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=5721 |