[Home]

Summary:ASTERISK-15882: [patch] [regression] No message is actually prepended when sending a voicemail message to another user
Reporter:Martin Glazer (mglazer)Labels:
Date Opened:2010-03-27 03:37:00Date Closed:2010-11-16 15:29:30.000-0600
Priority:BlockerRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20100408__issue17103.diff.txt
Description:When in voicemail and listening to a message, one has the ability to press 8 and forward the message to another user. One can also prepend a message by then pressing 1.
this does not currently work - no message is actually prepended and only the original message is forwarded to the user.

****** ADDITIONAL INFORMATION ******

This works correctly in 1.4.29 and is probably caused by this patch
https://issues.asterisk.org/bug_view_page.php?bug_id=16415

I believe it is deleting or overwriting the prepended + original message and only sending the original message.
Comments:By: Martin Glazer (mglazer) 2010-03-27 03:46:47

Additionally, the original message files
msg0000.txt msg0000.wav msg0000.WAV are left in the /var/spool/asterisk/voicemail/defaul/<ext>/tmp directory and are not removed.

By: Tilghman Lesher (tilghman) 2010-04-08 15:54:32

It appears the issue exists in 1.4 alone.  Trunk and 1.6 are not affected.

Patch uploaded and ready for testing.

By: Tilghman Lesher (tilghman) 2010-04-13 14:26:13

mglazer: I really would like some testing feedback on this issue, please.

By: Leif Madsen (lmadsen) 2010-04-14 14:12:28

mglazer: this issue is now holding up the next Asterisk 1.4.x release. Please test as soon as possible.

By: Leif Madsen (lmadsen) 2010-04-14 14:12:54

Upgraded to blocker status for 1.4.31.

By: Digium Subversion (svnbot) 2010-04-14 18:08:12

Repository: asterisk
Revision: 257266

U   branches/1.4/apps/app_voicemail.c

------------------------------------------------------------------------
r257266 | tilghman | 2010-04-14 18:08:11 -0500 (Wed, 14 Apr 2010) | 10 lines

When forwarding a message, ensure that prepending works correctly.

This is a regression in 1.4, only.

(closes issue ASTERISK-15882)
Reported by: mglazer
Patches:
      20100408__issue17103.diff.txt uploaded by tilghman (license 14)
Tested by: tilghman

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=257266

By: Digium Subversion (svnbot) 2010-04-14 18:08:52

Repository: asterisk
Revision: 257267

_U  trunk/

------------------------------------------------------------------------
r257267 | tilghman | 2010-04-14 18:08:52 -0500 (Wed, 14 Apr 2010) | 16 lines

Blocked revisions 257266 via svnmerge

........
 r257266 | tilghman | 2010-04-14 18:08:11 -0500 (Wed, 14 Apr 2010) | 10 lines
 
 When forwarding a message, ensure that prepending works correctly.
 
 This is a regression in 1.4, only.
 
 (closes issue ASTERISK-15882)
  Reported by: mglazer
  Patches:
        20100408__issue17103.diff.txt uploaded by tilghman (license 14)
  Tested by: tilghman
........

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=257267

By: Digium Subversion (svnbot) 2010-11-16 15:29:30.000-0600

Repository: asterisk
Revision: 295200

U   branches/1.4/apps/app_voicemail.c

------------------------------------------------------------------------
r295200 | jpeeler | 2010-11-16 15:29:30 -0600 (Tue, 16 Nov 2010) | 9 lines

Ensure original message duration is preserved when prepending a message.

It seems the fix to issue 17103 was a little overzealous and removed the code
that backed up the textfile containing the original message duration. This
code has now been restored.

(related to issue ASTERISK-15882)
ABE-2654

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=295200