Merged revisions 180380 via svnmerge from
authorMark Michelson <mmichelson@digium.com>
Thu, 5 Mar 2009 19:14:14 +0000 (19:14 +0000)
committerMark Michelson <mmichelson@digium.com>
Thu, 5 Mar 2009 19:14:14 +0000 (19:14 +0000)
commite69803a2be40f88e96efa0a2040fe75efd5096a6
tree7ed861d412cce9fdba04b4d0e1b7b5058506f936
parent9348bfd926c3fca19117af0b68aede4dd9ee35a0
Merged revisions 180380 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r180380 | mmichelson | 2009-03-05 12:58:48 -0600 (Thu, 05 Mar 2009) | 25 lines

  Fix broken mailbox parsing when searchcontexts option is enabled.

  When using the searchcontexts option in voicemail.conf, the code
  made the assumption that all mailbox names defined were unique across
  all contexts. However, the code did nothing to actually enforce this
  assumption, nor did it do anything to alert a user that he may have
  created an ambiguity in his voicemail.conf file by defining the same
  mailbox name in multiple contexts.

  With this change, we now will issue a nice long warning if searchcontexts
  is on and we encounter the same mailbox name in multiple contexts and ignore
  any duplicates after the first box. Whether searchcontexts is enabled or not,
  if we come across a duplicate mailbox in the same context, then we will issue
  a warning and ignore the duplicated mailbox. I have also added a small note
  to voicemail.conf.sample in the explanation for searchcontexts explaining
  that you cannot define the same mailbox in multiple contexts if you have
  enabled the option.

  (closes issue #14599)
  Reported by: lmadsen
  Patches:
        14599.patch uploaded by mmichelson (license 60) (with slight modification)
  Tested by: lmadsen
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@180383 65c4cc65-6c06-0410-ace0-fbb531ad65f3
apps/app_voicemail.c
configs/voicemail.conf.sample