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

[Commit Hook]

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


This review has been submitted with commit 4f88eb47c9772b5621eb22df13cc657b22e59bce by Andras Mantia to branch master.

- Commit Hook


On Feb. 29, 2012, 8:54 p.m., Andras Mantia wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104117/
> -----------------------------------------------------------
> 
> (Updated Feb. 29, 2012, 8:54 p.m.)
> 
> 
> 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 
>   resources/imap/tests/testremovecollectiontask.cpp d4c6765 
> 
> 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/