[Kde-pim] Review Request: Emit changeProcessed() when an invalid folder is asked to be removed

[Andras Mantia]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104117/
-----------------------------------------------------------

Review request for KDEPIM and Kevin Ottens.


Description
-------

In case the imap resource gets a request to remove a folder that doesn't exist, the resource will not signal that the operation was processed. This is very important if the request comes from the change recorder, as if it doesn't get the changeProcessed() signal, it will not process the next change. This can result in the resource being stopped, not syncing anymore.
The actual cause (why the change recorder had an entry that was not correct - a collection with remoteid "foo" instead of "/fpo") is not yet known, but neverthless the resource should deal with such case as well.

Please review asap, I'd like to put into 4.8 as well (with a different string, the same used in the other error cases).


Diffs
-----

  resources/imap/removecollectionrecursivetask.cpp 9e78cff 

Diff: http://git.reviewboard.kde.org/r/104117/diff/


Testing
-------

User confirmed that the change fixes the resource not syncing issue, make test still passes.


Thanks,

Andras Mantia

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/