LDAP autocomplete display odd when pretty name not found

VERIFIED FIXED in mozilla1.0

Status

MailNews Core
LDAP Integration
--
minor
VERIFIED FIXED
17 years ago
10 years ago

People

(Reporter: dmose, Assigned: dmose)

Tracking

(Blocks: 1 bug)

Trunk
mozilla1.0

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

17 years ago
from nsILDAPAutoCompleteSession.idl:
 
     * A template string for formatting the autocompletion results.   
     *
     * Required attributes are delimited by curly braces, and optional 
     * attributes are determined by brackets.  Backslash escapes any 
     * character, including itself.  The default template is
     * "[cn] <{mail}>", without the quotes.  This will generate autocomplete
     * items of the form "John Doe <john.doe@foo.com>", or, if the cn is not
     * found, " <john.doe@foo.com>".  This latter form is suboptimal, in that
     * the space and angle brackets are a bit ugly, but should work. 
 
The above-mentioned ugliness should go away.  Either there could be some 
special-case coding for the default (ick!), the syntax could be made more
complex to allow for parts of the template to be conditionalized based on
whether specific attrs are found, or an outputFormatter attribute could be
added, which would be a JS function that would do whatever formatting logic desired.
(Assignee)

Updated

17 years ago
Blocks: 17880
(Assignee)

Updated

17 years ago
Target Milestone: --- → mozilla0.9.3
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.3 → mozilla1.0
(Assignee)

Comment 1

17 years ago
This was fixed for mail-format addresses when nsIAbLDAPAutoCompFormatter was
factored out of nsILDAPAutoCompleteSession.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 2

17 years ago
Yulian Chang wrote:
Dan,
I tried to add an entry without CN in my LDAP server, bologna to test this fix.
But CN is required field to create the entry. What else I can do? Or do you have
any entry without CN? If yes, please give me the server information and the entry.
Thank you for help.

I suspect you're right, and that it's not even possible to have an entry without
a cn.  So I wouldn't worry about it.

Dan

Comment 3

16 years ago
Verified.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.