[asterisk-bugs] [Asterisk 0015654]: [patch] Missing new-message notification for urgent messages

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
https://issues.asterisk.org/view.php?id=15654 
====================================================================== 
Reported By:                tomo1657
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   15654
Category:                   Applications/app_voicemail
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     ready for review
Asterisk Version:           1.6.1.2 
JIRA:                        
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.1 
SVN Revision (number only!): 210394 
Request Review:              
====================================================================== 
Date Submitted:             2009-08-04 13:23 CDT
Last Modified:              2010-01-28 11:40 CST
====================================================================== 
Summary:                    [patch] Missing new-message notification for urgent
messages
Description: 
notify_new_message() is not called when urgent, new messages are left in a
mailbox using non-IMAP file system storage.

Urgent messages are moved to the "Urgent" folder so the following
condition to check INBOX for the new message (line 5312) is always false,
and notify_new_message() is never called.

  if (ast_fileexists(fn, NULL, NULL)) {

A workaround would be to add an exception for urgent messages:

  if (ast_fileexists(fn, NULL, NULL) && !ast_strlen_zero(flag) &&
!strcmp(flag, "Urgent"))

which I have attacked a patch of.  There are other ways to approach this,
though.  (Run notify_new_message() within the urgent message section or
checking for dfn location instead of fn)
======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
parent of           0016448 Voicemail messages flagged as urgent do...
====================================================================== 

---------------------------------------------------------------------- 
 (0117327) russell (administrator) - 2010-01-28 11:40
 https://issues.asterisk.org/view.php?id=15654#c117327 
---------------------------------------------------------------------- 
Thanks a lot for the patch.  Hopefully we can get a committer on this soon
to do final review and commit.  I have suspended the review request in the
mean time, but we can open it back up if the committer decides it is needed
once they take this on. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-01-28 11:40 russell        Note Added: 0117327                          
======================================================================