D28179: [contacts] Populate remoteId of newly created contact

[Daniel Vrátil]

dvratil requested changes to this revision.
dvratil added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> contactsresource.cpp:527
> +        item.setPayload<KContacts::Addressee>(*contact.dynamicCast<KContacts::Addressee>());
> +        new ItemModifyJob(item);
>      } else if (collection.isValid()) {

This should not be needed: the call to `Akonadi::ResourceBase::changeCommitted()` does internally call `ItemModifyJob` for the item in order to update the remoteid...

REPOSITORY
  R44 KDE PIM Runtime

REVISION DETAIL
  https://phabricator.kde.org/D28179

To: poboiko, dvratil
Cc: kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20200321/c15f1dda/attachment.html>