[asterisk-bugs] [Asterisk 0015185]: [patch] Patch to allow tone-list as argument to ReadExten

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15185 
====================================================================== 
Reported By:                jcovert
Assigned To:                qwell
====================================================================== 
Project:                    Asterisk
Issue ID:                   15185
Category:                   Applications/app_readexten
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     closed
Target Version:             1.8
Asterisk Version:           SVN 
JIRA:                       SWP-413 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!):  
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-05-22 10:32 CDT
Last Modified:              2009-12-14 15:33 CST
====================================================================== 
Summary:                    [patch] Patch to allow tone-list as argument to
ReadExten
Description: 
It is desirable to allow ReadExten to prompt with tones that are specific
to the use of ReadExten.  This patch allows support of tones (a) not in
indications.conf and/or (b) independent of the tone-zone.

With no flag, the "filename" argument causes ReadExten to play the file in
the background.

There is an "i" flag, which causes it to look the "filename" argument up
in indications.conf and play the tone sequence instead.

Current behaviour if the "indication" isn't found is to fall back to
trying to treat it as a filename (even though "i" is given).  That will
probably fail.

The attached patch allows a tone sequence to be specified instead of an
indication.  Effectively, if the "i" option is specified, the filename
argument is treated exactly like an argument to the "Playtones"
application.  Since the Playtones application accepts "|" as a separator,
even a multiple tone sequence may be specified.

example:

exten => s,n,ReadExten(exten,390+460/400|330+400,unrestricteddisa,i,20)

This plays 390+440 for 400 ms and then 330+400 from then on.
====================================================================== 

---------------------------------------------------------------------- 
 (0115232) svnbot (reporter) - 2009-12-14 15:33
 https://issues.asterisk.org/view.php?id=15185#c115232 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 234779

_U  branches/1.6.2/

------------------------------------------------------------------------
r234779 | qwell | 2009-12-14 15:33:09 -0600 (Mon, 14 Dec 2009) | 18 lines

Blocked revisions 234776 via svnmerge

........
  r234776 | qwell | 2009-12-14 15:32:03 -0600 (Mon, 14 Dec 2009) | 12
lines
  
  Allow tonelist as argument to ReadExten.
  
  ReadExten already supported playing a tonezone from indications.conf.
  It now has the ability to use a tonelist like 440+480/2000|0/4000
  
  (closes issue https://issues.asterisk.org/view.php?id=15185)
  Reported by: jcovert
  Patches:
        app_readexten.c.patch uploaded by jcovert (license 551)
  Tested by: qwell
  Patch modified by me, to maintain backwards compatibility.
........

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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-12-14 15:33 svnbot         Checkin                                      
2009-12-14 15:33 svnbot         Note Added: 0115232                          
======================================================================