D12427: IMAP resource: fix data loss on connection lost, depending on the current task

[David Faure]

dfaure created this revision.
dfaure added a reviewer: dvratil.
Restricted Application added a project: KDE PIM.
dfaure requested review of this revision.

REVISION SUMMARY
  If the current task is e.g. uploading an item to the server, we don't
  want to cancelTask (which means give up and never try again), we want to
  deferTask, so we try again later. Otherwise, well, this is how we end up
  with items without a remote id, which makes syncing that folder much
  slower every time due to the item count mismatch.
  
  Abstracted with a new method abortTask, which calls cancel or defer
  depending on the type of task.

TEST PLAN
  Sync large folder in kmail, abort sync. But I didn't test any of
  the task types other than collection sync...

REPOSITORY
  R44 KDE PIM Runtime

BRANCH
  Applications/18.04

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

AFFECTED FILES
  resources/imap/resourcetask.cpp
  resources/imap/resourcetask.h

To: dfaure, dvratil
Cc: #kde_pim, dvasin, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20180421/308365d2/attachment.html>