[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-02-23 02:07 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...
====================================================================== 

---------------------------------------------------------------------- 
 (0118392) tilghman (administrator) - 2010-02-23 02:07
 https://issues.asterisk.org/view.php?id=15654#c118392 
---------------------------------------------------------------------- 
I have updated the patch against trunk, along with some changes to the
functions, such that they act in predictable (and documented) ways.  One
problem was that messagecount did not act consistently between file storage
methods.  Also, in certain circumstances, an Urgent message was counted
twice. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-02-23 02:07 tilghman       Note Added: 0118392                          
======================================================================