| Summary: | Adding address book link and using name from address book causes name to be blank | ||
|---|---|---|---|
| Product: | [Unmaintained] kopete | Reporter: | Sean Lynch <techniq35> |
| Component: | libkopete | Assignee: | Kopete Developers <kopete-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | ||
| Priority: | NOR | ||
| Version First Reported In: | unspecified | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
Sean Lynch
2005-07-22 23:32:16 UTC
Would also maybe be a good idea to add an Apply button to this widget, especially until the bug is resolved.. that way you wouldn't have to reopen the dialog twice to use the workaround. I looked around and I guess all that has to do be done is change the OK|Cancel to OK|Apply|Cancel in both locations in kopetelviprops.cpp and connect the signal applyClicked() to the slotOkClicked slot ... but I have never coded on KDE and just started reading a QT book. SVN commit 453283 by ogoffart: fix Bug 109494: Adding address book link and using name from address book causes name to be blank BUG: 109494 CCMAIL: mattr@kde.org M +4 -2 kopetelviprops.cpp --- branches/KDE/3.5/kdenetwork/kopete/kopete/contactlist/kopetelviprops.cpp #453282:453283 @@ -414,6 +414,10 @@ void KopeteMetaLVIProps::slotOkClicked() { + // update meta contact's UID + item->metaContact()->setMetaContactId( mAddressBookUid ); + //this has to be done first, in the case something is synced with KABC (see bug 109494) + // set custom display name if( mainWidget->edtDisplayName->text() != item->metaContact()->customDisplayName() ) item->metaContact()->setDisplayName( mainWidget->edtDisplayName->text() ); @@ -450,8 +454,6 @@ item->metaContact()->setIcon( mainWidget->icnbUnknown->icon(), Kopete::ContactListElement::Unknown ); } - // update meta contact's UID - item->metaContact()->setMetaContactId( mAddressBookUid ); mNotificationProps->storeCurrentCustoms(); } |