#493: AEL parse error on valid file: Digium issue 14543 (Fixed)

Back to ticket list | See last change
Apr 14 2009 * 02:22
Reported by: Release: 1.2
Priority: Normal Milestone: 2.0
Component: core Assigned to: mjagdis
Attachment: extensions.ael  [application/octet-stream]

Issue 14543 crops up for me using r5359. I don’t believe it to be patched as I did a search for 14543 and nothing came up.

The attached extensions.ael results in a parse error at the expansion of 61${CALLERID}.

Cheers,

Michael

Changelog:

Modified by Jun 05 2010 * 13:02
  • Component: changed from other to core
  • Assigned user: set to mjagdis
  • Status: changed from Open to Fixed
  • Milestone: set to 2.0

Removed the calls to cw_expr() from pbx_ael.c in trunk. These are used to “syntax check” expression. But this is just plain wrong. Expressions are subject to substitution BEFORE evaluation by cw_expr() so it’s doing the wrong thing to start with. To compound the silliness you don’t want to substitute or do anything else that might have side effects when simply loading dialplan.

AEL is a pretty dumb idea. All it does is transform to Real dialplan on loading. Avoid it.