[asterisk-dev] [Code Review] Voicemail check cleanup

Tilghman Lesher tlesher at digium.com
Thu Feb 25 18:29:08 CST 2010


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

(Updated 2010-02-25 18:29:08.119261)


Review request for Asterisk Developers.


Changes
-------

Addressed Mark's suggestions.


Summary
-------

A few cleanups to fix a few bugs.

- Urgent voicemails were not attached, because the attachment code looked in the wrong folder.
- Urgent voicemails were sometimes counted twice when displaying the count of new messages.
- Backends were inconsistent as to which voicemails each API counted.


This addresses bugs 15654 and 16448.
    https://issues.asterisk.org/view.php?id=15654
    https://issues.asterisk.org/view.php?id=16448


Diffs (updated)
-----

  /trunk/apps/app_voicemail.c 248859 

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


Testing
-------

Unit test written, tested, and included.


Thanks,

Tilghman




More information about the asterisk-dev mailing list