[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/#review6706
-----------------------------------------------------------


Looks good now, but Tobias should have a look at it as well


akonadi/contact/contactsfilterproxymodel.cpp
<http://git.reviewboard.kde.org/r/102619/#comment5929>

    Minor nitpick: missing spaces before and after =
    


- Kevin


On Sept. 21, 2011, 12:09 p.m., Martin Bednar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102619/
> -----------------------------------------------------------
> 
> (Updated Sept. 21, 2011, 12:09 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/