[Home]

Summary:ASTERISK-05613: [patch] equal sign in application argument confuses AEL
Reporter:edgreenberg (edgreenberg)Labels:
Date Opened:2005-11-15 15:44:14.000-0600Date Closed:2008-01-15 15:56:19.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:PBX/pbx_ael
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) ael-assignment.diff
Description:I wrote this:
context edg {
       s => {
               NoOp(foo = ${FOO});
       };
} ;

and I got this:

[ Context 'edg' created by 'pbx_ael' ]
 's' =>            1. Set(NoOp(foo=$[ ${FOO}) ])                 [pbx_ael]

Looks like the = sign caused AEL to try and construct a Set command



****** ADDITIONAL INFORMATION ******

To prove that this wasn't related to NoOp, I also wrote:
context edg {
       s => {
               Dial(foo = ${FOO});
       };
} ;

and got:
[ Context 'edg' created by 'pbx_ael' ]
 's' =>            1. Set(Dial(foo=$[ ${FOO}) ])                 [pbx_ael]
Comments:By: Kevin P. Fleming (kpfleming) 2005-11-15 16:08:43.000-0600

Experimental features can't have 'MAJOR' bugs :-)
By: edgreenberg (edgreenberg) 2005-11-15 16:52:54.000-0600

Sorry. It would have been major by definition, had it not been experimental :)
By: Sergey Okhapkin (sokhapkin) 2005-11-15 17:06:20.000-0600

Seems like a bug to me:-) The workaround is to remove spaces around "=" character. I.e.

NoOp(foo=${FOO});

instead of

NoOp(foo = ${FOO});
By: Sergey Okhapkin (sokhapkin) 2005-11-15 17:38:06.000-0600

Bug fixed, patch uploaded.
By: Tilghman Lesher (tilghman) 2005-11-15 19:26:38.000-0600

edgreenberg: please confirm that the uploaded patch fixes the issue for you.
By: edgreenberg (edgreenberg) 2005-11-15 22:27:04.000-0600

Patch works for me. Thank you.
By: Sergey Okhapkin (sokhapkin) 2005-11-16 05:54:02.000-0600

Corydon76: yes, on file:-)
By: Kevin P. Fleming (kpfleming) 2005-11-16 12:26:38.000-0600

Committed to CVS HEAD, thanks!
By: Digium Subversion (svnbot) 2008-01-15 15:56:19.000-0600

Repository: asterisk
Revision: 7113

U   trunk/ChangeLog
U   trunk/pbx/pbx_ael.c

------------------------------------------------------------------------
r7113 | kpfleming | 2008-01-15 15:56:19 -0600 (Tue, 15 Jan 2008) | 2 lines

issue ASTERISK-5613

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=7113