Closed Bug 81961 Opened 23 years ago Closed 23 years ago

LDAP autocomplete display odd when pretty name not found

Categories

(MailNews Core :: LDAP Integration, defect)

defect
Not set
minor

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.0

People

(Reporter: dmosedale, Assigned: dmosedale)

References

Details

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.
Blocks: 17880
Target Milestone: --- → mozilla0.9.3
Target Milestone: mozilla0.9.3 → mozilla1.0
This was fixed for mail-format addresses when nsIAbLDAPAutoCompFormatter was
factored out of nsILDAPAutoCompleteSession.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
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

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.