[asterisk-bugs] [Asterisk 0011564]: [PATCH] Removing the need for the second app in ExecIf

[noreply at bugs.digium.com]

The following issue has been ASSIGNED. 
====================================================================== 
http://bugs.digium.com/view.php?id=11564 
====================================================================== 
Reported By:                pnlarsson
Assigned To:                qwell
====================================================================== 
Project:                    Asterisk
Issue ID:                   11564
Category:                   Applications/app_exec
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     assigned
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 93152 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             12-15-2007 09:48 CST
Last Modified:              12-26-2007 16:10 CST
====================================================================== 
Summary:                    [PATCH] Removing the need for the second app in
ExecIf
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.
====================================================================== 

---------------------------------------------------------------------- 
 svnbot - 12-26-07 16:10  
---------------------------------------------------------------------- 
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 http://bugs.digium.com/view.php?id=11564, patch by pnlarsson (with
some extra cleanup by me).

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

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

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
12-26-07 16:10  svnbot         Note Added: 0075982                          
12-26-07 16:10  svnbot         Status                   new => assigned     
12-26-07 16:10  svnbot         Assigned To               => qwell           
======================================================================