[asterisk-dev] [Code Review] 2539: Fix up VM snapshot, INBOX combining.

[svnbot]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2539/
-----------------------------------------------------------

(Updated May 15, 2013, 9:19 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and Matt Jordan.


Changes
-------

Committed in revision 388814


Repository: Asterisk


Description
-------

VM snapshots have a "combine" mode, that allows multiple folders to be merged into INBOX.

This should make it a little more clear about what's going on.  It was overly complicated, and handled unnecessary/invalid use cases, like requesting the Old folder while trying to combine (I assert that to the POV of the snapshot requester, Old does not exist, when combine is enabled).


Diffs
-----

  /branches/11/apps/app_voicemail.c 388696 

Diff: https://reviewboard.asterisk.org/r/2539/diff/


Testing
-------

The bug that this was written to address is fixed - Old messages no longer show up in other folders.


Thanks,

Jason Parker

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130515/d568c2ba/attachment.htm>