221286 - Don't let user try to add LDAP server if built with --disable-ldap

Status: NEW

(MailNews Core :: LDAP Integration, defect)

Description

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6a) Gecko/20031004
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6a) Gecko/20031004

If moz is compiled with --disable-ldap, I think it should warn you when you try
to add an LDAP server.

Reproducible: Always

Steps to Reproduce:
Compile moz with --disable-ldap.  Try to add an LDAP server.

Actual Results:  
Nothing.  You click "Add", and then the server list remains empty.

Expected Results:  
Either add the server to the list (bad behavior since LDAP was disabled) or warn
the user that with LDAP disabled, you can't add a server.

Comment 1

[Stefan Borggraefe]

Confirming. The best way to fix this is probably fixing bug 84004.

Comment 2

[Sebastian Späth]

Even better would be to hide the ldap related UI in this case. However warning
the user is the next best thing. I was really puzzled when no server appeared
after clicking add.

Comment 3

[(not reading, please use [e-mail] instead)]

sorry for the spam.  making bugzilla reflect reality as I'm not working on these bugs.  filter on FOOBARCHEESE to remove these in bulk.

Comment 4

[:aceman]

We can hide/disable UI if there are some useful compile-times defines set according to this --disable-ldap option. Looks like there is MOZ_LDAP_XPCOM .