[asterisk-commits] mjordan: branch 11 r382617 - /branches/11/apps/app_voicemail.c

[SVN commits to the Asterisk project]

Author: mjordan
Date: Thu Mar  7 11:57:08 2013
New Revision: 382617

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=382617
Log:
Let vm_mailbox_snapshot combine "Urgent" when no folder is specified

r381835 fixed a bug in vm_mailbox_snapshot where combining INBOX and Old forgot
that Urgent also "counts" as new messages. This fixed the problem when any of
the three folders was specified and the combine option was used.

It missed the case where the folder isn't specified and we build a snapshot of
all folders. This patch corrects that.


Modified:
    branches/11/apps/app_voicemail.c

Modified: branches/11/apps/app_voicemail.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/apps/app_voicemail.c?view=diff&rev=382617&r1=382616&r2=382617
==============================================================================
--- branches/11/apps/app_voicemail.c (original)
+++ branches/11/apps/app_voicemail.c Thu Mar  7 11:57:08 2013
@@ -15021,10 +15021,12 @@
 		int combining_old = 0;
 		/* Assume we are combining folders if:
 		 *  - The current index is the old folder index OR
-		 *  - The current index is urgent and we were looking for INBOX OR
-		 *  - The current index is INBOX and we were looking for Urgent
+		 *  - The current index is urgent and we were looking for INBOX or all folders OR
+		 *  - The current index is INBOX and we were looking for Urgent or all folders
 		 */
-		if ((i == old_index || (i == urgent_index && this_index_only == inbox_index) || (i == inbox_index && this_index_only == urgent_index)) && (combine_INBOX_and_OLD)) {
+		if ((i == old_index ||
+			(i == urgent_index && (this_index_only == inbox_index || this_index_only == -1)) ||
+			(i == inbox_index && (this_index_only == urgent_index || this_index_only == -1))) && (combine_INBOX_and_OLD)) {
 			combining_old = 1;
 		}