[Home]

Summary:ASTERISK-11050: [PATCH] Removing the need for the second app in ExecIf
Reporter:Niklas Larsson (pnlarsson)Labels:
Date Opened:2007-12-15 09:48:57.000-0600Date Closed:2007-12-26 16:10:58.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_exec
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) execif_patch_2.diff
Description:When using ExecIf without the second application argument it would return -1:

exten s,1,ExecIf($[$["${ARG3}" != ""]]?Authenticate(${ARG3}))

log:
app_exec.c: Could not find application! ((null))

Attached patch fixes this.
Comments:By: Niklas Larsson (pnlarsson) 2007-12-15 13:26:34.000-0600

The change between 1.4 and trunk for ExecIf is quit big. Should this not be mentioned in UPGRADE.txt or so?
By: Niklas Larsson (pnlarsson) 2007-12-15 16:06:37.000-0600

Well, if somebody could remove the first patch i would be happy and less ashamed...
By: Digium Subversion (svnbot) 2007-12-26 16:10:57.000-0600

Repository: asterisk
Revision: 94814

U   trunk/apps/app_exec.c

------------------------------------------------------------------------
r94814 | qwell | 2007-12-26 16:10:55 -0600 (Wed, 26 Dec 2007) | 4 lines

Make 'else' argument to ExecIf optional.  Clean up the description and usage text a bit.

Closes issue ASTERISK-11050, patch by pnlarsson (with some extra cleanup by me).

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

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