[asterisk-bugs] [Asterisk 0014949]: Can't delete temporary greeting when using IMAP storage

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14949 
====================================================================== 
Reported By:                noahisaac
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   14949
Category:                   Applications/app_voicemail/IMAP
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           1.6.2.0-beta1 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.2 
SVN Revision (number only!): 189816 
Request Review:              
====================================================================== 
Date Submitted:             2009-04-22 10:33 CDT
Last Modified:              2009-04-22 11:58 CDT
====================================================================== 
Summary:                    Can't delete temporary greeting when using IMAP
storage
Description: 
When using IMAP storage for voicemail, temporary greetings can't be erased.
 
====================================================================== 

---------------------------------------------------------------------- 
 (0103636) noahisaac (reporter) - 2009-04-22 11:58
 http://bugs.digium.com/view.php?id=14949#c103636 
---------------------------------------------------------------------- 
It looks like this issue only exists when using IMAP storage without using
IMAP greetings. (Actually, it looks like there's another bug to file for
IMAP greetings)

Just reviewing the relevant stuff in app_voicemail.c - In
vm_tempgreeting(), option 2 references DELETE(), which, when we're using
IMAP storage, is defined to use vm_imap_delete().  I think this is the
problem because vm_imap_delete() has no facility to delete normal files -
it only deletes imap messages.

Would it be OK to call delete_file() directly, instead of using the
DELETE() macro?  I'll go ahead and create a patch to do this, but maybe
there's a good reason to use the macro? 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-04-22 11:58 noahisaac      Note Added: 0103636                          
======================================================================