[asterisk-dev] [Code Review] Add aliases for Directory

[Mark Michelson]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2244/#review7533
-----------------------------------------------------------


I like this feature. The big thing that's missing here is updated documentation for voicemail.conf and realtime voicemail config.


/trunk/apps/app_directory.c
<https://reviewboard.asterisk.org/r/2244/#comment14319>

    The strcmp here should be strcasecmp since ast_variable_retrieve() does a case-insensitive comparison.



/trunk/apps/app_directory.c
<https://reviewboard.asterisk.org/r/2244/#comment14321>

    Since you use strcasestr, you should use strcasecmp.



/trunk/apps/app_directory.c
<https://reviewboard.asterisk.org/r/2244/#comment14329>

    IMO, this should only be done if OPT_LISTBYFIRSTNAME is set.


- Mark


On Dec. 12, 2012, 6:19 p.m., Tilghman Lesher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2244/
> -----------------------------------------------------------
> 
> (Updated Dec. 12, 2012, 6:19 p.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This patch adds extra strings for comparison in the Directory application.  The purpose of this is to enable nicknames that otherwise would not be compared.  For example, Bob "Bongo Boy" Berman might have an alias of "Bongo", for which he would be searchable, even though his name would not normally include it.
> 
> 
> Diffs
> -----
> 
>   /trunk/apps/app_directory.c 377511 
> 
> Diff: https://reviewboard.asterisk.org/r/2244/diff
> 
> 
> Testing
> -------
> 
> Deployed and working on a production machine.
> 
> 
> Thanks,
> 
> Tilghman
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20121213/d22dcec0/attachment-0001.htm>