Akonadi contact resource wrong defaults

Bug #966913 reported by Alex Fiestas
32
This bug affects 5 people
Affects Status Importance Assigned to Milestone
KDE PIM
Fix Released
Critical
kdepim (Ubuntu)
Invalid
High
BVBISE_B6

Bug Description

By default Akonadi contact resource is configured to store contacts in .local/share/contacts but this last subfolder does not exist.

This will make akonadi launch a notification each time akonadi is started saying ".local/share/contacts folder does not exist" but what is more dangerous, akonadi will allow the creation of contacts since they will be stored in the cache (mysql), if something happens to that cache the data will be lost.

I already reported the akonadi_contact_resource bug in bugs.kde.org:
https://bugs.kde.org/show_bug.cgi?id=296944

But nevertheless the defaults for the resource must be fixed, either by changing the folder or by creating it.

Revision history for this message
In , Alex Fiestas (afiestas) wrote :

If for some reason (like what seems a bug in the distribution) the destination folder does not exists, KaddressBook (and the resource) will allow the creation of contacts and they will be stored in the akonadi cache.

Akonadi won't be able to synchronize the data since the folder does not exits in case that something happens to the mysql databse (or in case that we change to another database) we will suffer data loss :/

Maybe we should turn the resource offline in case that the folder does not exists or force the user to select another folder.

Revision history for this message
Alex Fiestas (afiestas) wrote :

Oh btw the same thing happens with .local/share/notes but in the case of notes the resource will create the folder when needed but we should create the folder ourselves too.

Jonathan Riddell (jr)
Changed in kdepim (Ubuntu):
milestone: none → ubuntu-12.04-beta-2
tags: added: kubuntu
Revision history for this message
Alex Fiestas (afiestas) wrote :

Same things happens with local/share/local-email folder, and like in the notes case it will be created by akonadi but still akonadi will popup a warning which is noisy.

Screeenshot with all warnings show first time you open Kontact:
http://wstaw.org/m/2012/03/28/plasma-windowedT19331.png

It is important that we fix this bug for making a good impression first time the user executes kontact.

Revision history for this message
In , E-jjm (e-jjm) wrote :

Is this the vcarddir Akonadi resource? If so, bug 289565 may be similar or even a duplicate.

Changed in kdepim (Ubuntu):
milestone: ubuntu-12.04-beta-2 → none
milestone: none → ubuntu-12.04
Changed in kdepim (Ubuntu):
status: New → Confirmed
Jonathan Riddell (jr)
Changed in kdepim (Ubuntu):
importance: Undecided → High
Revision history for this message
In , Harald Sitter (apachelogger) wrote :

Git commit 20f02b27dcc43948f3f062f9dfded338232a24af by Harald Sitter.
Committed on 04/06/2012 at 07:13.
Pushed by sitter into branch 'master'.

make sure the contacts resource creates its configured path

..to avoid silly errors
http://standards.freedesktop.org/basedir-spec/basedir-spec-latest.html

M +3 -0 resources/contacts/contactsresource.cpp

http://commits.kde.org/kdepim-runtime/20f02b27dcc43948f3f062f9dfded338232a24af

Changed in kdepim:
importance: Unknown → Critical
status: Unknown → Fix Released
tags: added: precise
Revision history for this message
Amr Ibrahim (amribrahim1987) wrote :

I can see that this bug is fixed upstream, is it going to be fixed in Kubuntu 12.04?

description: updated
Changed in kdepim (Ubuntu):
assignee: nobody → BVBISE_B6 (b6bvbisemp2013)
Revision history for this message
Rohan Garg (rohangarg) wrote :

Hi
This is already fixed upstream, packages for KDE SC 4.12.2 are available via Kubuntu Backports and should contain the fix for your release.

Changed in kdepim (Ubuntu):
milestone: ubuntu-12.04 → none
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.