[Home]

Summary:ASTERISK-06601: [patch] [needs testing] app_voicemail looks for VMBox in the [default] context only
Reporter:JR Richardson (hubguru)Labels:
Date Opened:2006-03-22 19:08:40.000-0600Date Closed:2006-03-24 22:11:20.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20060323__bug6783.diff.txt
( 1) app_voicemail.c.rej
Description:The situation:  When I dial into VoiceMailMain(@context), put in my VM # 1001 and Password 1001, no problem, but at the voicemail main audio prompt (Alison), when I press 3 for advanced options then press 5 to leave a message I put in a mailbox number 1002 within the same [context], but VoiceMailMain looks for the mailbox in the [default] context and will not recognize the mailbox in the current [context].

Error at the CLI:

Mar 21 14:11:18 WARNING[21294]: app_voicemail.c:2384 leave_voicemail: No entry in voicemail config file for '1002'

Voicemail.conf

[default]

[context]
1001 => 1001,line1
1002 => 1002,line2

extension.conf

exten => *155,1,Voicemailmain(@context)



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

I posted this to the asterisk-user group and others have experienced the smae issue with the latest 1.2.5 stable.
Comments:By: JR Richardson (hubguru) 2006-03-22 19:59:37.000-0600

I just checked an older version, CVS Head 11-24-05 and this feature worked fine.
By: twisted (twisted) 2006-03-22 20:07:30.000-0600

what about current SVN trunk?
By: Tilghman Lesher (tilghman) 2006-03-23 12:26:43.000-0600

If you set 'searchcontexts=yes' in voicemail.conf, does this issue go away?
By: Tilghman Lesher (tilghman) 2006-03-23 12:37:33.000-0600

You can also try the attached patch.  This is an either/or for the searchcontexts=yes, so try one or the other, but not both.
By: JR Richardson (hubguru) 2006-03-23 15:14:40.000-0600

I tried searchcontexts=yes and yes, it finds the VBBox ok.  I odn't think that is a good thing though.  If voicemail was setup like this:

[default]

[context1]
1001 =>
1002 =>

[context2]
1001 =>
1002 =>


If you were in vmbox 1001 @[context2] and tried to leave a vm for vmbox 1002, would searchcontexts=yes, search in default first, then [context1], then [context2]?

If this is the case, vm would be left in 1002 @[context1].

What is this patch supposed to do?
By: Tilghman Lesher (tilghman) 2006-03-23 18:37:58.000-0600

The patch will (hopefully) do what you need: to specify a specific context in the appropriate location.  Please test.
By: JR Richardson (hubguru) 2006-03-24 14:21:10.000-0600

The patch did not apply:

The text leading up to this was:
--------------------------
|Index: apps/app_voicemail.c
|===================================================================
|--- apps/app_voicemail.c       (revision 14417)
|+++ apps/app_voicemail.c       (working copy)
--------------------------
File to patch: apps/app_voicemail.c
patching file apps/app_voicemail.c
Hunk #1 succeeded at 3464 (offset -19 lines).
Hunk #2 FAILED at 6505.
1 out of 2 hunks FAILED -- saving rejects to file apps/app_voicemail.c.rej


The reject file is attatched.  I'm using 1.2.4 stable branch.
By: Tilghman Lesher (tilghman) 2006-03-24 15:26:48.000-0600

You'll need to download and patch against the latest 1.2 branch.  See the asterisk.org site for information on getting the latest source.
By: JR Richardson (hubguru) 2006-03-24 20:16:56.000-0600

I tested the patch against latest svn, it worked fine.  can you write the patch for 1.2.4 stable or 1.2.5 stable so I don't have to use the testing branch?

Thanks.

JR
By: Tilghman Lesher (tilghman) 2006-03-24 22:11:20.000-0600

Committed to 1.2; merged to trunk.