[asterisk-bugs] [Asterisk 0017807]: Music on hold doesn't recover very cleanly when it can't play a file

[Asterisk Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
https://issues.asterisk.org/view.php?id=17807 
====================================================================== 
Reported By:                kshumard
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   17807
Category:                   Resources/res_musiconhold
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.8.0-beta2 
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-08-06 11:35 CDT
Last Modified:              2010-08-06 11:35 CDT
====================================================================== 
Summary:                    Music on hold doesn't recover very cleanly when it
can't play a file
Description: 
I found that MOH using default files misbehaved in 1.8.0-beta1 because of a
dot in the filenames of the LICENSE, CHANGES, and CREDITS files. I see that
this has been fixed in 1.8.0-beta2 by renaming those files so they don't
include a dot. But this uncovered something else that it wouldn't hurt to
address.

When res_musiconhold can't play a file in one of its classes, it stops
musiconhold and just waits indefinitely.

I suggest that a cleaner way to handle this would be to skip the file it
can't play (and perhaps remove it from the musicclass) and continue on to
try to play the next file in the class. As it is, MOH stops and there's
just dead air on the channel until it's hungup or taken off hold or
whatever. Logging a warning and playing another file is IMO a saner/cleaner
way to fail.
====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-08-06 11:35 kshumard       New Issue                                    
2010-08-06 11:35 kshumard       Asterisk Version          => 1.8.0-beta2     
2010-08-06 11:35 kshumard       Regression                => No              
2010-08-06 11:35 kshumard       SVN Branch (only for SVN checkouts, not tarball
releases) => N/A             
======================================================================