Differential D7042

Work around Nextcloud sometimes providing vcards without trailing newline
AbandonedPublic
Actions

Authored by vkrause on Aug 1 2017, 6:21 PM.

Details

Reviewers
dvratil
Group Reviewers
KDE PIM
Summary

That makes our vcard parser fail, resulting in the affected contacts not
being added to Akonadi.

Diff Detail

Repository
R44 KDE PIM Runtime
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
vkrause created this revision.Aug 1 2017, 6:21 PM
Restricted Application added a project: KDE PIM. · View Herald TranscriptAug 1 2017, 6:21 PM
dvratil accepted this revision.Aug 1 2017, 6:30 PM
This revision is now accepted and ready to land.Aug 1 2017, 6:30 PM
mlaurent added a subscriber: mlaurent.Aug 2 2017, 4:58 AM

Could you verify with last kcontact as there are a fix about a missing "\n"
I commited patch yesterday.
Perhaps it's not necessary (and there is an autotest etc).

Regards

vkrause added a comment.Aug 2 2017, 7:23 AM

Bad timing, I just missed that one :) Looks like it indeed should fix this, and in the proper place, unlike my dirty workaround. Testing now...

vkrause abandoned this revision.Aug 2 2017, 7:30 AM

Works :)

Revision Contents
Changeset List

PathPackages
Mresources/dav/resource/utils.cpp (11 lines)

Diff 17510

View Options

resources/dav/resource/utils.cpp

Loading...