[Kde-pim] Review Request: Enable contact filtering by email address availabillity

[Kevin Krammer]

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


Generally a good idea, but maybe we should consider making that an enum or flags, e.g. to specify whether to apply always apply this criteria or only when a filter string is set, etc.

- Kevin


On Sept. 15, 2011, 2:58 p.m., Martin Bednar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102619/
> -----------------------------------------------------------
> 
> (Updated Sept. 15, 2011, 2:58 p.m.)
> 
> 
> Review request for KDEPIM-Libraries.
> 
> 
> Summary
> -------
> 
> Adds an option to ContactsFilterProxyModel to filter out contacts with no email addresses. This is useful in the EmailAddressSelectionWidget which, as its name suggests, should only show contacts with email addresses.
> 
> 
> This addresses bug 250242.
>     http://bugs.kde.org/show_bug.cgi?id=250242
> 
> 
> Diffs
> -----
> 
>   akonadi/contact/contactsfilterproxymodel.h 416869b 
>   akonadi/contact/contactsfilterproxymodel.cpp eecc220 
>   akonadi/contact/emailaddressselectionwidget.cpp fc7ec44 
> 
> Diff: http://git.reviewboard.kde.org/r/102619/diff
> 
> 
> Testing
> -------
> 
> created contacts with and without email addresses. As expected, those without didn't appear in the messagecomposer address selection widget.
> 
> 
> Thanks,
> 
> Martin
> 
>

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