[asterisk-bugs] [Asterisk 0013249]: AEL does not translate quoted strings correctly in 1.6

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13249 
====================================================================== 
Reported By:                dimas
Assigned To:                murf
====================================================================== 
Project:                    Asterisk
Issue ID:                   13249
Category:                   PBX/pbx_ael
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     acknowledged
Asterisk Version:           1.6.0-beta9 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-08-06 09:48 CDT
Last Modified:              2008-08-18 16:39 CDT
====================================================================== 
Summary:                    AEL does not translate quoted strings correctly in
1.6
Description: 
I used to use constructs like 

    a = "test here";

in the Asterisk AEL scripts. These were working fine in 1.4 (translating
to)

    Set(a=$[ "test here"])

This does not work for 1.6 anymore although translates to exactly the same
dialplan. After executing this line the value of a variable on 1.6 contains
quotes.

This results in:
1. The following construct:

    b = "${a}";
does not work at all giving parse error [Aug  6 18:01:24] WARNING[29514]:
ast_expr2.fl:437 ast_yyerror: ast_yyerror():  syntax error: syntax error,
unexpected '<token>', expecting $end; Input:

2. when the variable being assigned is CALLERID(number) - this results in
sending 

    From: ""test here"" <sip:xxxxxxxxxxx at 1.2.3.4>;tag=as1539ede1

in the SIP INVITE header. Note the double quotes. As result, the Aastra
57i phone rejects the invite with 400 Bad Request.
====================================================================== 

---------------------------------------------------------------------- 
 (0091527) murf (administrator) - 2008-08-18 16:39
 http://bugs.digium.com/view.php?id=13249#c91527 
---------------------------------------------------------------------- 
OK, I've been pondering what I **could** do in AEL to make your burden
lighter, and it sounds like all I have to do is look for a set to a quoted
string (in 1.6 and trunk only), and dequote the string before I put in the
Set() call. This would only be done for <val>="<something>";  -- if it
generates Set(<val>=<something>);
instead of Set(<val>="<something>"); 

Would this kind of update be useful? I've been trying to think of issues
that it might not solve... 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-08-18 16:39 murf           Note Added: 0091527                          
======================================================================