nsLDAPAutoCompleteSession should process filterTemplate param

VERIFIED FIXED in mozilla0.9.2

Status

P1
major
VERIFIED FIXED
18 years ago
16 years ago

People

(Reporter: dmose, Assigned: dmose)

Tracking

(Blocks: 1 bug)

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
(Assignee)

Comment 5

18 years ago
Created attachment 38298 [details] [diff] [review]
Implement processing of filter templates patch, v1.
(Assignee)

Comment 6

18 years ago
Created attachment 38321 [details] [diff] [review]
patch v2, addressing various suggestions from leif

Comment 7

18 years ago
r=leif
(Assignee)

Comment 8

18 years ago
Created attachment 38331 [details] [diff] [review]
patch 2a, changes to mailnews JS
(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 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.