116984 - don't hard code ldap query results limit, use maxhit per server pref

Status: VERIFIED FIXED

(SeaMonkey :: MailNews: Address Book & Contacts, defect, P2)

Description

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

don't hard code ldap query results limit, use maxhit per server pref

from nsAbLDAPDirectory.cpp

    // XXX todo, instead of 100, use the ldap_2.servers.xxx.maxHits pref
    // the problem is how to get that value here.
    //
    // I'm thinking that nsAbDirectories should know their key so that
    // they can do a lookup of server values from the key, when they need it
    // (as those values can change)
    rv = DoQuery(arguments, queryListener, 100, 0, &mContext);

Comment 1

[yulian chang]

Reassign QA contact to myself

Comment 2

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

I'm not sure what the right approach is yet, but this does need fixing.

Comment 3

[Peter Trudelle]

ADT triage: okay to fix by removing UI, if that is easier/safer.

Comment 4

[John Marmion]

*** Bug 128161 has been marked as a duplicate of this bug. ***

Comment 5

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

I got a fix for this, see bug #126134

Comment 6

[Daniel Zirin]

*** Bug 128336 has been marked as a duplicate of this bug. ***

Comment 7

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

fixed.

Comment 8

[yulian chang]

20020320 trunk builds.