[asterisk-dev] [Code Review] 4595: Voicemail API: fix handling of mailbox full condition

[Scott Griepentrog]

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

(Updated April 7, 2015, 2:34 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 434260


Bugs: ASTERISK-24942
    https://issues.asterisk.org/jira/browse/ASTERISK-24942


Repository: Asterisk


Description
-------

In r115582 (2008), the ERROR_MAILBOX_FULL flag was removed, and a check of the save_to_folder result no longer handled the mailbox full condition.  This lead to the deletion of a new Inbox message, once played, that could not be relocated to the Old mailbox because of the maxmsg limit.  This patch restores the original functionality lost, which is to leave the message in the Inbox without deleting it.


Diffs
-----

  /branches/11/apps/app_voicemail.c 434135 

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


Testing
-------

Tested manually on my system under Asterisk 13.


Thanks,

Scott Griepentrog

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150407/8ad2a7fd/attachment.html>