code cleanup of LDAP autocomplete pref panel js, addressbook properties js

RESOLVED DUPLICATE of bug 305434

Status

SeaMonkey
MailNews: Address Book & Contacts
RESOLVED DUPLICATE of bug 305434
16 years ago
9 years ago

People

(Reporter: (not reading, please use seth@sspitzer.org instead), Unassigned)

Tracking

Trunk
x86
Windows 2000
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

code cleanup of LDAP autocomplete pref js

there is some autocomplete pref js that is doing stuff that the back end code 
already does.

spefically, we have js to enumerate all ldap servers, for dynamically 
determining the unique "prefname",  for poking the prefs when adding or 
deleting or modifying ldap servers.

instead, we should be using the existing nsIAddressBook interface, to which we 
need to add a modify()

some bugs that relate to this:

124057  nor -- PC srilatha@netscape.com NEW  mozilla0.9.9 nab-ldap when I 
delete addressbooks from the prefs (addressing panel)  
124059  nor -- PC srilatha@netscape.com NEW  --- nab-ldap when I add LDAP 
directories from the prefs (addressing panel  
124061  nor -- PC srilatha@netscape.com NEW  --- nab-ldap when I delete a LDAP 
directory from the addressbook, there a  
124064  nor -- PC srilatha@netscape.com NEW  --- nab-ldap when I add an LDAP 
directory from the addressbook (not imple  
124553  nor -- PC srilatha@netscape.com NEW  --- nab-ldap Changes to LDAP 
properties in Address Book window not reflec
one more bug, #125823 (add modify() to nsIAddressBook.idl and use it.)

there's a lot of code to clean up, we shouldn't be writing or reading from 
prefs directly.

I don't think we need to re-write all the code before 0.9.9 or 1.0, but we 
should start with the code that writes prefs, and later fix the does the 
reading.

the reason we need to fix the write-pref code first, is that it poking prefs 
directly causes bugs that the user can see.
No longer depends on: 124553
Depends on: 125831

Updated

16 years ago
Depends on: 83114

Updated

16 years ago
Depends on: 127007

Updated

16 years ago
Depends on: 127110

Comment 2

16 years ago
*** Bug 83114 has been marked as a duplicate of this bug. ***

Updated

16 years ago
Depends on: 128340

Updated

16 years ago
Depends on: 128342

Updated

15 years ago
QA Contact: nbaca → gchan
cavin has fixed some of this already.
Assignee: srilatha → cavin
Product: Browser → Seamonkey
Depends on: 408613
Assignee: cavin → mail
QA Contact: grylchan → addressbook
The remaining work on this will be done in bug 305434.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 305434
You need to log in before you can comment on or make changes to this bug.