nsLDAPAutoCompleteSession should process filterTemplate param

VERIFIED FIXED in mozilla0.9.2

Status

defect
P1
major
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: dmose, Assigned: dmose)

Tracking

other
mozilla0.9.2
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [PDT+] have r=, sr=, awaiting a=)

Attachments

(4 attachments)

(Assignee)

Description

18 years ago
And this should probably also be hooked up to a hidden pref in the JS in mailnew
that calls nsLDAPAutoCompleteSession, once it does.
(Assignee)

Updated

18 years ago
Blocks: 70933
Status: NEW → ASSIGNED
Depends on: 74896
(Assignee)

Updated

18 years ago
No longer blocks: 70933
(Assignee)

Updated

18 years ago
Blocks: 17880

Updated

18 years ago
Severity: normal → major
Priority: -- → P1
Target Milestone: --- → mozilla0.9.1

Comment 1

18 years ago
reassign to Olga as QA Contact
QA Contact: yulian → olgac

Updated

18 years ago
Target Milestone: mozilla0.9.1 → mozilla0.9.2
(Assignee)

Updated

18 years ago
No longer depends on: 74896
(Assignee)

Updated

18 years ago
Blocks: 85315
(Assignee)

Comment 2

18 years ago
See bug 85315 for a better explanation of what's going on here.  This bug
(77386) is for implementing the functionality of allowing the user to specify
the LDAP search filter template via hidden pref.  83515 is discussion about what
the default search filter template should be.  Only this patch has any actual
code associated with it.
(Assignee)

Comment 3

18 years ago
OK, I've got working code in my tree.  Need to sand the corners off a few edge
cases; then I'll attach a patch here.

Whiteboard: code in hand; patch forthcoming
PDT+ per 6/12 mtg.
Whiteboard: code in hand; patch forthcoming → [PDT+] code in hand; patch forthcoming

Comment 7

18 years ago
r=leif
(Assignee)

Comment 9

18 years ago
Forgot to attach the JS changes earlier; here they are.
Keywords: patch, review
Whiteboard: [PDT+] code in hand; patch forthcoming → [PDT+] have partial r=

Comment 10

18 years ago
r=srilatha on patch 2a
(Assignee)

Updated

18 years ago
Whiteboard: [PDT+] have partial r= → [PDT+] have r=, awaiting sr=

Comment 11

18 years ago
sr=bienvenu
(Assignee)

Comment 12

18 years ago
Posted patch patch v3Splinter Review
(Assignee)

Comment 13

18 years ago
OK, this latest patch has two tiny changes in it:

* used some temporaries to appease Mac compiler lameness

* updated to the CVS tip after the SearchExt API changes
(Assignee)

Updated

18 years ago
Whiteboard: [PDT+] have r=, awaiting sr= → [PDT+] have r=, sr=, awaiting a=
a= asa@mozilla.org for checkin to the trunk.
(on behalf of drivers)
Blocks: 83989
(Assignee)

Comment 15

18 years ago
Fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 16

18 years ago
verified on WinNT 4.0, MacOS 9.1, Linux RH 7.1 with  2001062004, 2001062008, 
2001062111 builds
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.