[asterisk-bugs] [Asterisk 0010896]: 2 features, ring expiry and periodic announce firstplay

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=10896 
====================================================================== 
Reported By:                kwakwaversal
Assigned To:                putnopvut
====================================================================== 
Project:                    Asterisk
Issue ID:                   10896
Category:                   Applications/app_queue
Reproducibility:            N/A
Severity:                   feature
Priority:                   normal
Status:                     feedback
Asterisk Version:            SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 84744 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             10-05-2007 09:05 CDT
Last Modified:              12-07-2007 14:48 CST
====================================================================== 
Summary:                    2 features, ring expiry and periodic announce
firstplay
Description: 
Posted both these together because they work hand in hand (but will also
work separately).

1st feature; ringexpiry

; When the 'r' option is passed to the Queue() you can expire the ringing
; then start music on hold.  If you use this with
periodic-announce-frequency
; and set it to the same value, when the ring expires it will play the 
; periodic announce then start the music on hold.
;
; ringexpiry = 10

2nd feature; periodic-announce-firstplay

; When to play the first periodic announcement.  This allows you to 
; play the first announcement at the beginning of the call then revert
; back to a much higher frequency (0=off)
;
;periodic-announce-firstplay=10

====================================================================== 

---------------------------------------------------------------------- 
 Corydon76 - 12-07-07 14:48  
---------------------------------------------------------------------- 
I wonder if the firstplay option would be better served as a second
argument to the periodic-announce-freqency, something like:

periodic-announce-frequency=10:120

where the first number is optional (in other words, parsed right-to-left).
 That would eliminate the extra option, while making it more clear as to
what the option is referring to.

Second, in terms of the ringexpiry, I would prefer if you would use the
optional argument to the r option directly in the Queue arguments, to
specify a ring timeout.  That is:  "r" or "r(10)" (where the default is of
course 0, to indicate no timeout). 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
12-07-07 14:48  Corydon76      Note Added: 0075056                          
======================================================================