[asterisk-bugs] [Asterisk 0015557]: [patch] Gosub() dequotes once more than Macro()

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15557 
====================================================================== 
Reported By:                rain
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   15557
Category:                   Applications/app_stack
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           Older 1.6.0 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2009-07-22 15:33 CDT
Last Modified:              2009-08-04 16:16 CDT
====================================================================== 
Summary:                    [patch] Gosub() dequotes once more than Macro()
Description: 
Since Gosub started accepting "macro" arguments (before 1.6.0-beta1), it
has been dequoting arguments one more time than Macro does.

Since Gosub() is meant to supplant Macro() in 1.6 and is already used by
AEL to implement macros, this breaks compatibility with 1.4.  This behavior
also makes it impossible to fetch arguments without quotes being
interpreted, which IS possible in a Macro() call through ${ARGn}.  It also
requires an obnoxious level of quoting if you want quotes to survive all
the way to a named macro argument in AEL -- e.g.
&macro(${QUOTE(${QUOTE(${CALLERID(all)})})});

Behind the scenes, Gosub() uses AST_STANDARD_APP_ARGS, which causes
ast_app_separate_args() to eat the quotes; Macro() does its own argument
parsing.  I think the best solution is probably to make Gosub() use
non-standard argument parsing as well, but I haven't written a patch yet
since I think this warrants more discussion.

I verified that this problem exists in 1.6.0-beta1, 1.6.0.10, and 1.6.11. 
It does not exist in 1.4 because Gosub() doesn't accept extra arguments.

====================================================================== 

---------------------------------------------------------------------- 
 (0108618) rain (reporter) - 2009-08-04 16:16
 https://issues.asterisk.org/view.php?id=15557#c108618 
---------------------------------------------------------------------- 
With this patch, macros in AEL pass any ',' in an argument to Set() or
MSet() (depending on the compatibility setting) wholesale when setting
named argument variables.  Because the version of Set is always chosen to
be the version compatible with 1.4 behavior, the ',' is interpreted by Set
as a request to set multiple variables and causes an error.  Would you like
me to open a new bug for this, or do you want to handle it in this bug?

Example:
macro testmacro(arg) { ...anything... }
&testmacro("Hello, world!");

Generates (with app_set = 1.6):
MSet(LOCAL(arg)=Hello, world!) 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-08-04 16:16 rain           Note Added: 0108618                          
======================================================================