[asterisk-bugs] [Asterisk 0014599]: searchcontexts=yes causes voicemail boxes to be setup wrong

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14599 
====================================================================== 
Reported By:                lmadsen
Assigned To:                mmichelson
====================================================================== 
Project:                    Asterisk
Issue ID:                   14599
Category:                   Applications/app_voicemail
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
Target Version:             1.4.24
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 179056 
Request Review:              
====================================================================== 
Date Submitted:             2009-03-03 17:36 CST
Last Modified:              2009-03-04 13:11 CST
====================================================================== 
Summary:                    searchcontexts=yes causes voicemail boxes to be
setup wrong
Description: 
In issue 13853 I thought the following note was an issue with IMAP, but it
seems it is an issue regardless of whether you're using IMAP. The issue
comes from the searchcontext=yes option being enabled. The text in the note
I posted to issue http://bugs.digium.com/view.php?id=13853 is still relevant.
====================================================================== 

---------------------------------------------------------------------- 
 (0101209) mmichelson (administrator) - 2009-03-04 13:11
 http://bugs.digium.com/view.php?id=14599#c101209 
---------------------------------------------------------------------- 
I've uploaded 14599.patch.

There are a couple of provisions made in this patch.

1. If you have searchcontexts turned on and we read a duplicated mailbox
name from the config file, then we will issue a notice saying that we
ignored the option. Furthermore, if the duplicate mailbox names are in
different contexts, a larger warning message will be printed alerting them
that they have an ambiguous configuration file and that it needs to be
changed.

2. A similar ignore check was also added for the case where you do not
have searchcontexts turned on. The old behavior was to silently drop the
first of the duplicated entries. Now a notice is issued stating that the
second duplicate in the config is being ignored.

3. The voicemail.conf.sample file now has a clarification in the
searchcontexts documentation stating that a requirement of using the option
is to have unique mailbox names across all contexts.

Let me know what you think about these. As I'm writing this, I'm wondering
if the notices printed should actually be warnings instead. I think I may
also need to make a similar change for loading mailboxes from realtime. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-03-04 13:11 mmichelson     Note Added: 0101209                          
======================================================================