[svn-commits] pabelanger: branch 1.6.2 r270333 - /branches/1.6.2/res/res_musiconhold.c

[SVN commits to the Digium repositories]

Author: pabelanger
Date: Mon Jun 14 16:35:52 2010
New Revision: 270333

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=270333
Log:
Merged revisions 270332 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
  r270332 | pabelanger | 2010-06-14 17:33:55 -0400 (Mon, 14 Jun 2010) | 21 lines
  
  Merged revisions 270331 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r270331 | pabelanger | 2010-06-14 17:31:59 -0400 (Mon, 14 Jun 2010) | 14 lines
    
    Properly play first file in sort list.
    
    When using sort=alpha we would always skip the first file
    in the list first time through.  We now check for that
    properly. 
    
    (closes issue #17470)
    Reported by: pabelanger
    Patches:
          sort.aplha.patch uploaded by pabelanger (license 224)
    Tested by: lmadsen
    
    Review: https://reviewboard.asterisk.org/r/703/
  ........
................

Modified:
    branches/1.6.2/res/res_musiconhold.c

Modified: branches/1.6.2/res/res_musiconhold.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/res/res_musiconhold.c?view=diff&rev=270333&r1=270332&r2=270333
==============================================================================
--- branches/1.6.2/res/res_musiconhold.c (original)
+++ branches/1.6.2/res/res_musiconhold.c Mon Jun 14 16:35:52 2010
@@ -269,16 +269,20 @@
 		return -1;
 	}
 
-	/* If a specific file has been saved confirm it still exists and that it is still valid */
-	if (state->save_pos >= 0 && state->save_pos < state->class->total_files && state->class->filearray[state->save_pos] == state->save_pos_filename) {
+	if (state->pos == 0 && state->save_pos_filename == NULL) {
+		/* First time so lets play the file. */
+		state->save_pos = -1;
+	} else if (state->save_pos >= 0 && state->save_pos < state->class->total_files && state->class->filearray[state->save_pos] == state->save_pos_filename) {
+		/* If a specific file has been saved confirm it still exists and that it is still valid */
 		state->pos = state->save_pos;
 		state->save_pos = -1;
 	} else if (ast_test_flag(state->class, MOH_RANDOMIZE)) {
 		/* Get a random file and ensure we can open it */
 		for (tries = 0; tries < 20; tries++) {
 			state->pos = ast_random() % state->class->total_files;
-			if (ast_fileexists(state->class->filearray[state->pos], NULL, NULL) > 0)
+			if (ast_fileexists(state->class->filearray[state->pos], NULL, NULL) > 0) {
 				break;
+			}
 		}
 		state->save_pos = -1;
 		state->samples = 0;
@@ -302,8 +306,9 @@
 
 	ast_debug(1, "%s Opened file %d '%s'\n", chan->name, state->pos, state->class->filearray[state->pos]);
 
-	if (state->samples)
+	if (state->samples) {
 		ast_seekstream(chan->stream, state->samples, SEEK_SET);
+	}
 
 	return 0;
 }