[Kde-pim] Review Request 115846: kontact summary view

[Sergio Luis Martins]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/115846/#review52417
-----------------------------------------------------------


Please split this into separate reviews, one for each issue addressed.
The fix for 278956 can go into 4.13 but the new features (config options and i18n strings related to birthdays ) can only go into 4.14.

- Sergio Luis Martins


On Feb. 20, 2014, 10:12 a.m., Eugenio Accorsi wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115846/
> -----------------------------------------------------------
> 
> (Updated Feb. 20, 2014, 10:12 a.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Bugs: 278956
>     http://bugs.kde.org/show_bug.cgi?id=278956
> 
> 
> Repository: kdepim
> 
> 
> Description
> -------
> 
> since i use kontact very often i tried to make the summary view work as expected.
> the only problem with this patch is the special event plugin.
> maybe someone with better experience with akonadi can steer me in the right direction..
> i changed the old implementation because it doesn't work for every contact.
> 
> 
> Diffs
> -----
> 
>   kontact/plugins/specialdates/sdsummarywidget.h 9076c79 
>   kontact/plugins/specialdates/sdsummarywidget.cpp 67990c3 
>   kontact/plugins/specialdates/kcmsdsummary.h 778c389 
>   kontact/plugins/specialdates/kcmsdsummary.cpp 3b9c5a4 
>   kontact/plugins/korganizer/apptsummaryconfig_base.ui 93e72bd 
>   kontact/plugins/korganizer/apptsummarywidget.h 95bfe3b 
>   kontact/plugins/korganizer/apptsummarywidget.cpp 33fff04 
>   kontact/plugins/korganizer/kcmapptsummary.h bddab14 
>   kontact/plugins/korganizer/kcmapptsummary.cpp 6a661a4 
>   kontact/plugins/korganizer/summaryeventinfo.h a9bb775 
>   kontact/plugins/korganizer/summaryeventinfo.cpp 00c9906 
> 
> Diff: https://git.reviewboard.kde.org/r/115846/diff/
> 
> 
> Testing
> -------
> 
> in the special-occasion plugin enabling the search for special events in contacts will result in high cpu usage by nepomuk.
> i tried also with master but it does not find any contacts.
> the summary event plugin works as expected.
> 
> 
> Thanks,
> 
> Eugenio Accorsi
> 
>

_______________________________________________
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/