Closed Bug 270712 Opened 20 years ago Closed 19 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: 19 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.