[asterisk-bugs] [Asterisk 0014267]: macro compatibility

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=14267 
====================================================================== 
Reported By:                dimas
Assigned To:                tilghman
====================================================================== 
Project:                    Asterisk
Issue ID:                   14267
Category:                   PBX/pbx_ael
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     feedback
Asterisk Version:           SVN 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 169153 
Request Review:              
====================================================================== 
Date Submitted:             2009-01-18 16:27 CST
Last Modified:              2010-10-18 15:23 CDT
====================================================================== 
Summary:                    macro compatibility
Description: 
In 1.4 AEL used to generate macro- prefix for all macro contexts and
transform &m1() calls to Macro(m1). Now 1.6 Macro is deprcated and AEL
generates Gosub instead and does NOT generate macro- prefix.

The problem is that if your dialplan calls AEL macros from regular
dialplan - it will not work because AEL-generated macros do NOT have macro-
prefix while Macro application will add it. For the same reason features
like

myfeature => #,self,Macro,feature1

will not work anymore if feature1 is defined in the AEL.
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
related to          0016646 [patch] Its not possible to pass more t...
====================================================================== 

---------------------------------------------------------------------- 
 (0128172) dimas (reporter) - 2010-10-18 15:23
 https://issues.asterisk.org/view.php?id=14267#c128172 
---------------------------------------------------------------------- 
Tilghman, well to be honest I was not aware of new syntax because UPGRADE
document I checked didn't mention it. However the funny thing is that even
not knowing the syntax, I tried it :) "just in case". And it did not work
for me.

Right now I'm repeating my test and it looks like that I just
misunderstood it not working completely with not working as I expected. The
fact is that neither Goto nor Gosub worked. However now I see a section in
the sample features.conf 

;
; IMPORTANT NOTE: The applicationmap is not intended to be used for all
Asterisk
;   applications. When applications are used in extensions.conf, they are
executed
;   by the PBX core. In this case, these applications are executed outside
of the
;   PBX core, so it does *not* make sense to use any application which has
any
;   concept of dialplan flow. Examples of this would be things like Macro,
Goto,
;   Background, WaitExten, and many more.


which does not really make sense to me because unlike Goto/Gosub the Macro
application in fact WORKS. The only trick is that it MUST be regular
dialpplan macro, not an AEL one. And that is the main point in the original
reports - since AEL macro changed to Gosub/return syntax internally, it
become impossible to call AEL macro using Macro application. Which in turn
made them unusable from features.conf where they worked fine in 1.4.


So let me summarize once again:
1. Gosub does not work in applicationmap
2. Macro does work but Macro cannot be used to call AEL macro - Gosub
should be used instead. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-10-18 15:23 dimas          Note Added: 0128172                          
======================================================================