Bug 222306 - Contact information dropped when saving into LDAP and LDAP cache options cannot be set
Summary: Contact information dropped when saving into LDAP and LDAP cache options cann...
Status: RESOLVED UNMAINTAINED
Alias: None
Product: kdepimlibs
Classification: Applications
Component: kabc (show other bugs)
Version: 4.3
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-01-11 23:59 UTC by rdratlos
Modified: 2017-01-07 21:29 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Patch for kdepimlibs to fix this bug (21.95 KB, patch)
2010-01-12 00:47 UTC, rdratlos
Details
LDAP schema required for testing of the patch using OpenLDAP (1.41 KB, application/octet-stream)
2010-01-12 00:49 UTC, rdratlos
Details

Note You need to log in before you can comment on or make changes to this bug.
Description rdratlos 2010-01-11 23:59:38 UTC
Version:            (using KDE 4.3.2)
OS:                Linux
Installed from:    Ubuntu Packages

I tested kaddressbook (4.3) as front-end to manage contacts in a LDAP directory
(addressbook). The server runs openldap 2.4. 

Using kaddressbook important information like role, work address and home address can be specified for a contact. 

kaddressbook uses kdepimlibs (kabc) for LDAP server based addressbooks. kdepimlibs
does not support role and work address for LDAP based contacts (addressees) and discards this information without any information to the user. This applies also for less important information like blog, IM, custom fields, etc. 

Furthermore, the LDAP cache options cannot be set and saved using the edit cache dialog of kabc.
Comment 1 rdratlos 2010-01-12 00:44:46 UTC
There seems to be a rather simple fix for this problem. 

The kabc plugin ldapkio within kdepimlibs has a private AddresseeToLDIF and a data method that can be changed to support home and work address. They reside in the source file resourceldapkio.cpp. 

In addition, the AddresseeToLDIF and saveData methods can be adapted to
support an error notification to the user if role information or home and work address have been specified for a contact.

The LDAP cache configuration problem can be fixed by adapting the setCachePolicy() method in resourceldapkio.cpp, the OfflineDialog constructor in resourceldapkioconfig.cpp and the CachePolicy enum in resourceldapkio.h.

I've attached a patch below to outline the required changes to the source code files. 

I've added several comments to the address related part of the patch to provide
for a better understanding of the changed address handling strategy. In principle, the new code uses the LDAP <organization> attribute to differentiate between a business and a work contact. If this attribute contains a company name ldapkio will put the address information into an address of type Address:Work. If the LDAP attribute is empty, the address information will be stored as Address:Home. When storing the address information into LDAP, ldapkio will use either Address:HOME or Address:Work depending on the type of address the user has chosen. If a user has specified both, a work and a home address for one and the same contact, ldapkio will store the work address, if a company name has been specified in the organization field, or the home address, if organization is empty. In addition, an error notification on the GUI will inform the user that only one address type has been store in LDAP directory.

The attached patch includes the corrected cache options management. In addition, the setCachePolicy() method in resourceldapkio.cpp checks now that only the cache policies that are specified by the CachePolicy enum (in resourceldapkio.h) can be set. This avoids a crash of kaddressbook.

I've tested this solution under Ubuntu karmic and it works fine. 

I have not identified further side-effects of this patch. But I recommend some
further testing. The solution seems also to be compatible to the new kaddressbook 4.4, which now limits address information to home and work address but still supports the role information.

Note: As the patch is against the original source code, it contains also the
patch for fixing the bugs https://bugs.kde.org/show_bug.cgi?id=218353 and https://bugs.kde.org/show_bug.cgi?id=221447.

Please consider this, when including this patch into the KDE source code
branch(es).
Comment 2 rdratlos 2010-01-12 00:47:48 UTC
Created attachment 39800 [details]
Patch for kdepimlibs to fix this bug

Note: As the patch is against the original source code, it contains also the
patch for fixing the bugs https://bugs.kde.org/show_bug.cgi?id=218353 and
https://bugs.kde.org/show_bug.cgi?id=221447.
Comment 3 rdratlos 2010-01-12 00:49:57 UTC
Created attachment 39801 [details]
LDAP schema required for testing of the patch using OpenLDAP

This schema adds country, mail2 and homeurl attributes to the standard
inetOrgPerson schema.
Comment 4 Denis Kurz 2016-09-24 20:56:09 UTC
This bug has only been reported for versions before 4.14, which have been unsupported for at least two years now. Can anyone tell if this bug still present?

If noone confirms this bug for a Framework-based version of kdepim (version 5.0 or later, as part of KDE Applications 15.08 or later), it gets closed in about three months.
Comment 5 Denis Kurz 2017-01-07 21:29:57 UTC
Just as announced in my last comment, I close this bug. If you encounter it again in a recent version (at least 5.0 aka 15.08), please open a new one unless it already exists. Thank you for all your input.