[Kde-pim] [patch] kontact todo summary widget: fix invalid due days

Allen Winter winter at kde.org
Thu Apr 10 15:06:29 BST 2008


On Thursday 10 April 2008 09:58:56 Jarosław Staniek wrote:
> For review:
>
> QDate::currentDate().daysTo( todo->dtDue().date() ) returns negative number
> if the due date is invalid.
>
> Previous state:
> http://img515.imageshack.us/my.php?image=todonoduedateed3.png
>
> After the fix (todosummarywidget.patch), we do not display invalid dates
> but we still show the todo items:
>
> http://img209.imageshack.us/my.php?image=todonoduedatefixpp3.png
>
> Optionally (todosummarywidget2.patch), we can just skip todos not having
> the due date despite having 'hasDueDate' flag set. How about this?
>
> (initially reported as https://www.intevation.de/roundup/kolab/issue2602)

The first patch is fine; i.e. still show the todo items.


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


More information about the kde-pim mailing list