[Kde-pim] Review Request: Review for the filter changes (already in master), for inclusion to 4.9 branch

[Andras Mantia]

> On July 5, 2012, 10:46 a.m., Kevin Krammer wrote:
> >

Cleanup/renaming/other stuff done.


> On July 5, 2012, 10:46 a.m., Kevin Krammer wrote:
> > mailcommon/searchpattern.h, line 206
> > <http://git.reviewboard.kde.org/r/105445/diff/1/?file=71338#file71338line206>
> >
> >     Looking through the subclass implementations, it seems that a suitable default implementation could return Envelope

It can, but I deliberately made it pure virtual, so subclasses are forced to decide what they need.


> On July 5, 2012, 10:46 a.m., Kevin Krammer wrote:
> > mailcommon/filter/itemcontext.h, line 46
> > <http://git.reviewboard.kde.org/r/105445/diff/1/?file=71332#file71332line46>
> >
> >     requiredPart?

This one I named differently because this is the *requested* part by the filter. Hard to explain, there is a difference though. :)


- Andras


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


On July 4, 2012, 9:15 p.m., Andras Mantia wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105445/
> -----------------------------------------------------------
> 
> (Updated July 4, 2012, 9:15 p.m.)
> 
> 
> Review request for KDEPIM and Volker Krause.
> 
> 
> Description
> -------
> 
> Fixes the mail loss bug in filtering (by removing the setCacheOnly(true) from the change recorder) and rewrites filter handling, so only the needed payload is downloaded. Adds a check to avoid changing an item with incomplete payload.
> Touches lots of files because of the fine-tuned payload downloading, that needed internal API changes. 
> 
> 
> Diffs
> -----
> 
>   kmail/kmcommands.h 6dbc2a6 
>   kmail/kmcommands.cpp b6edc27 
>   kmail/kmmainwidget.cpp 3688e55 
>   mailcommon/filter/filteraction.h cd6e18c 
>   mailcommon/filter/filteraction.cpp b4a348a 
>   mailcommon/filter/filteractionaddheader.h 4690ca3 
>   mailcommon/filter/filteractionaddheader.cpp 53f4c2c 
>   mailcommon/filter/filteractionaddtag.h 7b75a8f 
>   mailcommon/filter/filteractionaddtag.cpp 55114c9 
>   mailcommon/filter/filteractionaddtoaddressbook.h 61f793f 
>   mailcommon/filter/filteractionaddtoaddressbook.cpp f2d56f6 
>   mailcommon/filter/filteractionbeep.h 5dcf5cc 
>   mailcommon/filter/filteractionbeep.cpp 0c76ac2 
>   mailcommon/filter/filteractioncopy.h 4291a97 
>   mailcommon/filter/filteractioncopy.cpp 699b9ce 
>   mailcommon/filter/filteractiondelete.h 76881cf 
>   mailcommon/filter/filteractiondelete.cpp e11978e 
>   mailcommon/filter/filteractionexec.h 5436fb0 
>   mailcommon/filter/filteractionexec.cpp d3d9653 
>   mailcommon/filter/filteractionforward.h 0051a68 
>   mailcommon/filter/filteractionforward.cpp 4da32ff 
>   mailcommon/filter/filteractionmove.h 22f4541 
>   mailcommon/filter/filteractionmove.cpp 3f7d715 
>   mailcommon/filter/filteractionpipethrough.h 5135fa9 
>   mailcommon/filter/filteractionpipethrough.cpp 0313a20 
>   mailcommon/filter/filteractionplaysound.h 6e12a83 
>   mailcommon/filter/filteractionplaysound.cpp f3ad74c 
>   mailcommon/filter/filteractionredirect.h 48a823c 
>   mailcommon/filter/filteractionredirect.cpp 5f6f586 
>   mailcommon/filter/filteractionremoveheader.h 77365fc 
>   mailcommon/filter/filteractionremoveheader.cpp 90315ba 
>   mailcommon/filter/filteractionreplyto.h f8655d9 
>   mailcommon/filter/filteractionreplyto.cpp 97502bd 
>   mailcommon/filter/filteractionrewriteheader.h 825e82d 
>   mailcommon/filter/filteractionrewriteheader.cpp 6d7c3e4 
>   mailcommon/filter/filteractionsendfakedisposition.h be25071 
>   mailcommon/filter/filteractionsendfakedisposition.cpp 4225f57 
>   mailcommon/filter/filteractionsendreceipt.h bab1294 
>   mailcommon/filter/filteractionsendreceipt.cpp 3728555 
>   mailcommon/filter/filteractionsetidentity.h c054a1e 
>   mailcommon/filter/filteractionsetidentity.cpp 8405964 
>   mailcommon/filter/filteractionsetstatus.h 53f64ba 
>   mailcommon/filter/filteractionsetstatus.cpp 3da6102 
>   mailcommon/filter/filteractionsettransport.h 5c8b063 
>   mailcommon/filter/filteractionsettransport.cpp 601db46 
>   mailcommon/filter/filteractionstatus.h 7867235 
>   mailcommon/filter/filteractionstatus.cpp f99b28a 
>   mailcommon/filter/filteractionunsetstatus.h df5651d 
>   mailcommon/filter/filteractionunsetstatus.cpp 7413aa0 
>   mailcommon/filter/filterimporter/filterimporterevolution.cpp ca996ab 
>   mailcommon/filter/filterimporter/filterimporterprocmail.cpp 67055b0 
>   mailcommon/filter/filterimporter/filterimportersylpheed.cpp d23e14f 
>   mailcommon/filter/filtermanager.h 4f069dd 
>   mailcommon/filter/filtermanager.cpp 0f4101b 
>   mailcommon/filter/itemcontext.h bb4f7cc 
>   mailcommon/filter/itemcontext.cpp 43df66a 
>   mailcommon/filter/kmfilterdialog.h 3247cb4 
>   mailcommon/filter/kmfilterdialog.cpp 20b8d91 
>   mailcommon/filter/mailfilter.h 1444393 
>   mailcommon/filter/mailfilter.cpp 594c38b 
>   mailcommon/searchpattern.h dcda954 
>   mailcommon/searchpattern.cpp e50f253 
>   mailcommon/tests/searchpatterntest.cpp 703a572 
>   mailfilteragent/filtermanager.h 4b65af0 
>   mailfilteragent/filtermanager.cpp 142d978 
>   mailfilteragent/mailfilteragent.h d44f068 
>   mailfilteragent/mailfilteragent.cpp 13bd323 
> 
> Diff: http://git.reviewboard.kde.org/r/105445/diff/
> 
> 
> Testing
> -------
> 
> Works for me. :)
> 
> 
> 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/