[asterisk-bugs] [Asterisk 0015879]: [patch] voicemail should mark messages as read by doing an UPDATE instead of an INSERT followed by a DELETE

[Asterisk Bug Tracker]

The following issue has been CLOSED 
====================================================================== 
https://issues.asterisk.org/view.php?id=15879 
====================================================================== 
Reported By:                KyleK
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   15879
Category:                   Applications/app_voicemail/ODBC
Reproducibility:            always
Severity:                   tweak
Priority:                   normal
Status:                     closed
Target Version:             1.6.1.14
Asterisk Version:           SVN 
JIRA:                       SWP-585 
Regression:                 No 
Reviewboard Link:            
SVN Branch (only for SVN checkouts, not tarball releases): 1.6.1 
SVN Revision (number only!): 218049 
Request Review:              
Resolution:                 suspended
Fixed in Version:           
====================================================================== 
Date Submitted:             2009-09-10 21:54 CDT
Last Modified:              2010-01-12 17:09 CST
====================================================================== 
Summary:                    [patch] voicemail should mark messages as read by
doing an UPDATE instead of an INSERT followed by a DELETE
Description: 
I sniffed out that app_voicemail was doing an INSERT to move a message from
Inbox to Old and decided to change it to an UPDATE. I'm running asterisk
1.6.1 svn but I'll try this against trunk in a bit.
====================================================================== 

---------------------------------------------------------------------- 
 (0116547) tilghman (administrator) - 2010-01-12 17:09
 https://issues.asterisk.org/view.php?id=15879#c116547 
---------------------------------------------------------------------- 
This is wholly not going to work without a major restructuring of the code.
 The main problem here is that when you remove a voicemail out of sequence,
the code then fails to find messages beyond where that message used to be. 
In other words, we depend upon messages being contiguous in the code, so
when you create a hole by renaming a file, the rest of the code fails.

I'll suspend the issue for now, and you're welcome to reopen if/when you
have a patch that has been tested against the above scenario. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2010-01-12 17:09 tilghman       Note Added: 0116547                          
2010-01-12 17:09 tilghman       Status                   feedback => closed  
2010-01-12 17:09 tilghman       Resolution               open => suspended   
======================================================================