[asterisk-bugs] [Asterisk 0012230]: Instead of just going to the failed extension, add a couple more for OutgoingSpoolFailed

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12230 
====================================================================== 
Reported By:                ZX81
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   12230
Category:                   Core/*
Reproducibility:            always
Severity:                   feature
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.4.18 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             03-16-2008 15:06 CDT
Last Modified:              03-16-2008 22:34 CDT
====================================================================== 
Summary:                    Instead of just going to the failed extension, add a
couple more for OutgoingSpoolFailed
Description: 
This is a quick tweak that I am applying on everyone's machines at the
moment.  I tried to paste it to someone on the mailing list but wrapping
broke the patch so I'm uploading it here.  Mainly just for posterity, but
if you want to commit it that would make my life easier - pretty trivial
patch. 
====================================================================== 

---------------------------------------------------------------------- 
 Corydon76 - 03-16-08 22:34  
---------------------------------------------------------------------- 
And, in fact, the current code lets you emulate exactly what your patch
does, simply by adding some dialplan logic, which is far more versatile:

exten => failed,1,GotoIf($[${REASON} = 34]?congestion,1)
exten => failed,n,GotoIf($[${REASON} = 0]?timeout,1)
exten => failed,n,GotoIf($[${REASON} = 17]?busy,1)
etc.

I don't see any reason whatsoever to commit this patch, as it hardcodes
behavior that should properly be done in the dialplan. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
03-16-08 22:34  Corydon76      Note Added: 0084042                          
======================================================================