[Asterisk-code-review] func_vmcount: Add support for multiple mailboxes (asterisk[master])

[N A]

N A has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/16490 )


Change subject: func_vmcount: Add support for multiple mailboxes
......................................................................

func_vmcount: Add support for multiple mailboxes

Allows multiple mailboxes to be specified for VMCOUNT
instead of just one.

ASTERISK-29661 #close

Change-Id: I9108528300795fd5b607efa9d4dd7b74be031813
---
A doc/CHANGES-staging/func_vmcount.txt
M funcs/func_vmcount.c
2 files changed, 15 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/90/16490/1

diff --git a/doc/CHANGES-staging/func_vmcount.txt b/doc/CHANGES-staging/func_vmcount.txt
new file mode 100644
index 0000000..ba2a0a1
--- /dev/null
+++ b/doc/CHANGES-staging/func_vmcount.txt
@@ -0,0 +1,3 @@
+Subject: func_vmcount
+
+Multiple mailboxes may now be specified instead of just one.
diff --git a/funcs/func_vmcount.c b/funcs/func_vmcount.c
index cb75110..b3fe88e 100644
--- a/funcs/func_vmcount.c
+++ b/funcs/func_vmcount.c
@@ -44,10 +44,12 @@
 /*** DOCUMENTATION
 	<function name="VMCOUNT" language="en_US">
 		<synopsis>
-			Count the voicemails in a specified mailbox.
+			Count the voicemails in a specified mailbox or mailboxes.
 		</synopsis>
 		<syntax>
-			<parameter name="vmbox" required="true" />
+			<parameter name="vmbox" required="true" argsep="|">
+				<para>A mailbox or list of mailboxes</para>
+			</parameter>
 			<parameter name="folder" required="false">
 				<para>If not specified, defaults to <literal>INBOX</literal></para>
 			</parameter>
@@ -56,12 +58,16 @@
 			<para>Count the number of voicemails in a specified mailbox, you could also specify
 			the mailbox <replaceable>folder</replaceable>.</para>
 			<para>Example: <literal>exten => s,1,Set(foo=${VMCOUNT(125 at default)})</literal></para>
+			<para>A pipe-separated list of mailboxes may be specified to count voicemails in
+			multiple mailboxes. If a folder is specified, this will apply to all mailboxes specified.</para>
 		</description>
 	</function>
  ***/
 
 static int acf_vmcount_exec(struct ast_channel *chan, const char *cmd, char *argsstr, char *buf, size_t len)
 {
+	int total = 0;
+	char *mailbox = NULL;
 	AST_DECLARE_APP_ARGS(args,
 		AST_APP_ARG(vmbox);
 		AST_APP_ARG(folder);
@@ -82,7 +88,10 @@
 		args.folder = "INBOX";
 	}
 
-	snprintf(buf, len, "%d", ast_app_messagecount(args.vmbox, args.folder));
+	while ((mailbox = strsep(&args.vmbox, "|"))) {
+		total += ast_app_messagecount(mailbox, args.folder);
+	}
+	snprintf(buf, len, "%d", total);
 
 	return 0;
 }

-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/16490
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: I9108528300795fd5b607efa9d4dd7b74be031813
Gerrit-Change-Number: 16490
Gerrit-PatchSet: 1
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210919/3ffa503c/attachment.html>