[asterisk-bugs] [Asterisk 0017498]: [patch] Bug in reporting queue hold time

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17498 
====================================================================== 
Reported By:                corruptor
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   17498
Category:                   Applications/app_queue
Reproducibility:            always
Severity:                   trivial
Priority:                   normal
Status:                     feedback
Asterisk Version:           1.6.2.9-rc3 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Request Review:              
====================================================================== 
Date Submitted:             2010-06-11 03:52 CDT
Last Modified:              2010-06-15 07:07 CDT
====================================================================== 
Summary:                    [patch] Bug in reporting queue hold time
Description: 
I've noticed that asterisk incorrectly reports estimate hold time to
callers if the hold time between more than 1 minute and less then 2
minutes.
Asterisk correctly calculates avg minutes and avg seconds but play
incorrect time.
Example:
-- Hold time for 6 is 1 minute(s) 30 seconds    -- <SIP/1110-000000a8>
Playing 'queue-holdtime.slin' (language 'ru')
    -- Nobody picked up in 10000 ms
    -- Stopped music on hold on SIP/1108-000000a6
    -- <SIP/1108-000000a6> Playing 'queue-youarenext.slin' (language
'ru')
    -- <SIP/1110-000000a8> Playing 'digits/1.slin' (language 'ru')
    -- <SIP/1110-000000a8> Playing 'queue-minute.slin' (language 'ru')
    -- <SIP/1110-000000a8> Playing 'digits/90.slin' (language 'ru')
    -- <SIP/1110-000000a8> Playing 'queue-seconds.slin' (language 'ru')

We hear 1 minute 90 seconds which is wrong
====================================================================== 

---------------------------------------------------------------------- 
 (0123413) davidw (reporter) - 2010-06-15 07:07
 https://issues.asterisk.org/view.php?id=17498#c123413 
---------------------------------------------------------------------- 
The order of precedence of multiple entries in configuration files is not
well defined.  Some parts of the code scan the file and act on each
variable, in which case you get the last setting, but some parts of the
code go through a list of variables and search the file for each one in
turn, in which case you get the first occurrence.  This is considered a
feature; you should not rely on one behaviour or the other. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-06-15 07:07 davidw         Note Added: 0123413                          
======================================================================