<p>Jenkins2 <strong>merged</strong> this change.</p><p><a href="https://gerrit.asterisk.org/7428">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Benjamin Keith Ford: Looks good to me, but someone else must approve
  Sean Bright: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, approved
  Jenkins2: Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">Add new object for VoicemailUserEntry<br><br>Currently, when the app_voicemail sending VoicemailUserEntry AMI event, there's<br>no OldMessageCount info for default.<br>To check the OldMessageCount info, it required IMAP_STORAGE define, but this is<br>not correct.<br>Added OldMessageCount item as a default.<br><br>ASTERISK-27456<br><br>Change-Id: I5c71521c2d1daf8b7b161e31c34d28cca6aea4c7<br>---<br>M apps/app_voicemail.c<br>1 file changed, 4 insertions(+), 7 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c<br>index ecd423f..d017301 100644<br>--- a/apps/app_voicemail.c<br>+++ b/apps/app_voicemail.c<br>@@ -13283,11 +13283,8 @@<br>  <br>      AST_LIST_TRAVERSE(&users, vmu, list) {<br>            char dirname[256];<br>-#ifdef IMAP_STORAGE<br>              int new, old;<br>-<br>              inboxcount(vmu->mailbox, &new, &old);<br>-#endif<br>             <br>              make_dir(dirname, sizeof(dirname), vmu->context, vmu->mailbox, "INBOX");<br>              astman_append(s,<br>@@ -13319,8 +13316,8 @@<br>                     "MaxMessageCount: %d\r\n"<br>                   "MaxMessageLength: %d\r\n"<br>                  "NewMessageCount: %d\r\n"<br>-#ifdef IMAP_STORAGE<br>                     "OldMessageCount: %d\r\n"<br>+#ifdef IMAP_STORAGE<br>                     "IMAPUser: %s\r\n"<br>                  "IMAPServer: %s\r\n"<br>                        "IMAPPort: %s\r\n"<br>@@ -13353,14 +13350,14 @@<br>                       ast_test_flag(vmu, VM_OPERATOR) ? "Yes" : "No",<br>                   vmu->maxmsg,<br>                       vmu->maxsecs,<br>+                     new,<br>+                 old<br> #ifdef IMAP_STORAGE<br>-                    new, old,<br>+                    ,<br>                     vmu->imapuser,<br>                     vmu->imapserver,<br>                   vmu->imapport,<br>                     vmu->imapflags<br>-#else<br>-                    count_messages(vmu, dirname)<br> #endif<br>                         );<br>            ++num_users;<br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/7428">change 7428</a>. To unsubscribe, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/7428"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: merged </div>
<div style="display:none"> Gerrit-Change-Id: I5c71521c2d1daf8b7b161e31c34d28cca6aea4c7 </div>
<div style="display:none"> Gerrit-Change-Number: 7428 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Benjamin Keith Ford <bford@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>