Closed
Bug 899958
Opened 11 years ago
Closed 11 years ago
Lightning should stop using the LDAP interfaces in XPFE autocomplete now that they are going away
Categories
(Calendar :: Lightning Only, defect)
Calendar
Lightning Only
Tracking
(Not tracked)
RESOLVED
FIXED
2.8
People
(Reporter: philip.chee, Assigned: Fallen)
References
Details
Attachments
(1 file, 1 obsolete file)
17.77 KB,
patch
|
Details | Diff | Splinter Review |
This is unnecessary now that LDAP is using toolkit interfaces. See Bug 899822 Comment 0 > We're still trying to sync sessions between elements in the addressing > widget, but this is unnecessary now that LDAP is using toolkit interfaces. > I want to remove the API in bug 896123, so we should stop calling it! http://mxr.mozilla.org/comm-central/search?string=syncSessions http://mxr.mozilla.org/comm-central/search?string=addSession http://mxr.mozilla.org/comm-central/search?string=removeSession syncSessions/addSession/removeSession should all go.
Assignee | ||
Comment 1•11 years ago
|
||
This should take care. Looking at attachment 777047 [details] [diff] [review], all that needed to be done is remove all that xpfe code (yay!) and add "ldap" to the textbox's autocompletesearch attribute, which we already had. I've tested this with a global ldap directory defined.
Comment 2•11 years ago
|
||
Comment on attachment 785195 [details] [diff] [review] Fix - v1 Review of attachment 785195 [details] [diff] [review]: ----------------------------------------------------------------- Untested, looks good. r=mmecca
Attachment #785195 -
Flags: review?(matthew.mecca) → review+
Assignee | ||
Comment 3•11 years ago
|
||
Since I have no idea how RyanVM pushed the other patches without adding CLOSED TREE, I'm just going to upload this one and set checkin-needed.
Attachment #785195 -
Attachment is obsolete: true
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 4•11 years ago
|
||
(In reply to Philipp Kewisch from comment #3) > Since I have no idea how RyanVM pushed the other patches without adding > CLOSED TREE, I'm just going to upload this one and set checkin-needed. I don't know either, but I cheated. http://hg.mozilla.org/comm-central/rev/7ab5532fd624
Updated•11 years ago
|
Target Milestone: --- → 2.8
You need to log in
before you can comment on or make changes to this bug.
Description
•