Closed Bug 270712 Opened 20 years ago Closed 20 years ago

last sent LDAP attribute name always displayed in AB cards

Categories

(MailNews Core :: LDAP Integration, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.8beta2

People

(Reporter: dmosedale, Assigned: dmosedale)

References

Details

In fact, we have a precedence table in nsAbLDAPProperties.cpp, but it turns that because of the way MozillaLDAPPropertyRelator:createCardPropertyFromLDAPMessage() is written, that precedence is ignored.
Blocks: 157928
Product: MailNews → Core
Turns out that the precedence table as a data structure isn't really good enough to handle this in an efficient way. I spent some time thinking about this this weekend, and have come up with a somewhat modified set of data structures which should make this fairly straightforward. I have a partial patch in one of my trees using this; I'll attach once it's in slightly better shape.
Status: NEW → ASSIGNED
Component: MailNews: LDAP Integration → Address Book
Product: Core → Thunderbird
Target Milestone: --- → Thunderbird1.1
Component: Address Book → MailNews: LDAP Integration
Product: Thunderbird → Core
Target Milestone: Thunderbird1.1 → mozilla1.8beta3
Priority: -- → P1
(In reply to comment #1) > Turns out that the precedence table as a data structure isn't really good enough > to handle this in an efficient way. I spent some time thinking about this this > weekend, and have come up with a somewhat modified set of data structures which > should make this fairly straightforward. I have a partial patch in one of my > trees using this; I'll attach once it's in slightly better shape. Has it taken a better shape?
(In reply to comment #2) > > Has it taken a better shape? Yeah; it's evolving over in bug 119291.
This bug should be fixed by the checkin to bug 119291.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Target Milestone: mozilla1.8beta3 → mozilla1.8beta2
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.