Bug 223073

Summary: messaging addresses Akonadi/KAddressbooks contacts not properly handled or stored, compatibility resource broken
Product: [Applications] kaddressbook Reporter: Patrick Callahan <Patrick.pxc.C>
Component: generalAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: tokoe
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: screenshot depicting the "raw payload" of the Akonadi contact entry in the old format
screenshot depicting the "raw payload" of the Akonadi contact entry in the new format
KAddressbook contact editor with old-format data present
KAddressbook contact editor with new-format data present

Description Patrick Callahan 2010-01-17 04:44:23 UTC
Version:           4.3.90 (using Devel)
OS:                Linux
Installed from:    Compiled sources

KAddressbook no longer properly supports storing IM Addresses.
-the "Edit IM Addresses" dialog has been removed, so support for multiple IM addresses is gone
-the display pane doesn't show IM addresses by protocol, or anything other than the plain text entered in the one-line field
-the field KAddressbook uses to store/retrieve IM addresses has changed from X-messaging/<protocol>:<DATA> to "X-KADDRESSBOOK-X-IMAddress:<DATA>", but the Akonadi compatibility resource still has "traditional" KDE applications, namely Kopete, write to the VCF files in the old format
-because of the above point, doing an "Import Contacts" from Kopete doesn't do anything; Kopete doesn't know that any IM information is stored
Comment 1 Patrick Callahan 2010-01-17 04:47:12 UTC
This is the case on both Arch Linux (kdemod-testing repository) and Ubuntu with the KDE 4.4 ppa.
Comment 2 Patrick Callahan 2010-01-17 04:50:00 UTC
Created attachment 39963 [details]
screenshot depicting the "raw payload" of the Akonadi contact entry in the old format

KAddressBook, though only storing/reading the new format for IM addresses, doesn't delete the old-style data.

Put up in PNG format to hopefully reduce spamming to my email address, and to show that this is in fact what Akonadi sees
Comment 3 Patrick Callahan 2010-01-17 04:51:03 UTC
Created attachment 39964 [details]
screenshot depicting the "raw payload" of the Akonadi contact entry in the new format

KAddressBook, though only storing/reading the new format for IM addresses, doesn't delete the old-style data.

Put up in PNG format to hopefully reduce spamming to my email (and IM) address, and to show that this is in fact what Akonadi sees
Comment 4 Patrick Callahan 2010-01-17 04:53:47 UTC
Created attachment 39965 [details]
KAddressbook contact editor with old-format data present
Comment 5 Patrick Callahan 2010-01-17 04:55:21 UTC
Created attachment 39966 [details]
KAddressbook contact editor with new-format data present

note that the data here was manually entered; Kopete, even with the Akonadi Compatibility Resource which allows it to see and link contacts from KAddressbook, does not enter the data in this format
Comment 6 Tobias Koenig 2010-03-07 13:33:20 UTC
SVN commit 1100381 by tokoe:

Bring back the edit fields for instant messaging

BUG: 223073
BUG: 223858
BUG: 226299



 M  +5 -0      CMakeLists.txt  
 A             editor/im (directory)  
 A             editor/im/CMakeLists.txt  
 A             editor/im/icons (directory)  
 A             editor/im/icons/CMakeLists.txt  
 AM            editor/im/icons/ox16-app-aim_protocol.png  
 AM            editor/im/icons/ox16-app-gadu_protocol.png  
 AM            editor/im/icons/ox16-app-groupwise_protocol.png  
 AM            editor/im/icons/ox16-app-icq_protocol.png  
 AM            editor/im/icons/ox16-app-irc_protocol.png  
 AM            editor/im/icons/ox16-app-jabber_protocol.png  
 AM            editor/im/icons/ox16-app-meanwhile_protocol.png  
 AM            editor/im/icons/ox16-app-msn_protocol.png  
 AM            editor/im/icons/ox16-app-skype_protocol.png  
 AM            editor/im/icons/ox16-app-sms_protocol.png  
 AM            editor/im/icons/ox16-app-yahoo_protocol.png  
 A             editor/im/imdelegate.cpp   [License: LGPL (v2+)]
 A             editor/im/imdelegate.h   [License: LGPL (v2+)]
 A             editor/im/imeditordialog.cpp   [License: GPL (v2+) (+Qt exception)]
 A             editor/im/imeditordialog.h   [License: GPL (v2+) (+Qt exception)]
 A             editor/im/immodel.cpp   [License: LGPL (v2+)]
 A             editor/im/immodel.h   [License: LGPL (v2+)]
 A             editor/im/improtocols.cpp   [License: LGPL (v2+)]
 A             editor/im/improtocols.h   [License: LGPL (v2+)]
 A             editor/im/kaddressbookimprotocol.desktop   [TRAILING SPACE]
 A             editor/im/protocols (directory)  
 A             editor/im/protocols/CMakeLists.txt  
 A             editor/im/protocols/aimprotocol.desktop   [TRAILING SPACE]
 A             editor/im/protocols/gaduprotocol.desktop   [TRAILING SPACE]
 A             editor/im/protocols/groupwiseprotocol.desktop   [TRAILING SPACE]
 A             editor/im/protocols/icqprotocol.desktop   [TRAILING SPACE]
 A             editor/im/protocols/ircprotocol.desktop  
 A             editor/im/protocols/jabberprotocol.desktop   [TRAILING SPACE]
 A             editor/im/protocols/meanwhileprotocol.desktop  
 A             editor/im/protocols/msnprotocol.desktop   [TRAILING SPACE]
 A             editor/im/protocols/skypeprotocol.desktop  
 A             editor/im/protocols/smsprotocol.desktop   [TRAILING SPACE]
 A             editor/im/protocols/yahooprotocol.desktop   [TRAILING SPACE]
 M  +84 -0     editor/imeditwidget.cpp  
 M  +9 -4      editor/imeditwidget.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1100381