[asterisk-commits] rmudgett: branch rmudgett/mwi2 r307395 - /team/rmudgett/mwi2/channels/sig_pri.c

[SVN commits to the Asterisk project]

Author: rmudgett
Date: Wed Feb  9 17:36:39 2011
New Revision: 307395

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=307395
Log:
Add warning message if voicemail number is empty and made a debug message more useful.

Modified:
    team/rmudgett/mwi2/channels/sig_pri.c

Modified: team/rmudgett/mwi2/channels/sig_pri.c
URL: http://svnview.digium.com/svn/asterisk/team/rmudgett/mwi2/channels/sig_pri.c?view=diff&rev=307395&r1=307394&r2=307395
==============================================================================
--- team/rmudgett/mwi2/channels/sig_pri.c (original)
+++ team/rmudgett/mwi2/channels/sig_pri.c Wed Feb  9 17:36:39 2011
@@ -7906,8 +7906,8 @@
 	struct pri_party_id voicemail;
 	struct pri_party_id mailbox;
 
-	ast_debug(1, "Send MWI indication for %s@%s num_messages:%d\n", mbox_number,
-		mbox_context, num_messages);
+	ast_debug(1, "Send MWI indication for %s@%s vm_number:%s num_messages:%d\n",
+		mbox_number, mbox_context, S_OR(vm_number, "<not-present>"), num_messages);
 
 	memset(&mailbox, 0, sizeof(mailbox));
 	mailbox.number.valid = 1;
@@ -8180,6 +8180,12 @@
 			ast_log(LOG_ERROR, "%s span %d could not subscribe to MWI events for %s@%s.",
 				sig_pri_cc_type_name, pri->span, mbox_number, mbox_context);
 		}
+#if defined(HAVE_PRI_MWI_V2)
+		if (ast_strlen_zero(pri->mbox[i].vm_number)) {
+			ast_log(LOG_WARNING, "%s span %d MWI voicemail number for %s@%s is empty.\n",
+				sig_pri_cc_type_name, pri->span, mbox_number, mbox_context);
+		}
+#endif	/* defined(HAVE_PRI_MWI_V2) */
 	}
 #endif	/* defined(HAVE_PRI_MWI) */